Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Running PerformanceTest on Previous Releases (#546) #549

Merged
merged 1 commit into from
Aug 11, 2022

Conversation

sethAmazon
Copy link
Contributor

  • Basic functionality for backend transmitter

removed unit tests

playing around with github

changed package name

seperated the test case

reformatted

removed seperate test

changed package

new IntegrationTest Setup

fixed integrationTest.yml

applied pr feedback:
-added new stored data called "CommitDate"
-reorganized code
-updated code to go v2
-changed metrics struct

cleaned workflow init

auto run

typo auto test

cleanworkflow init

added performance test to integrationTest

changed epoch time to int
-after that became an issue in frontend

Delete fastPerformanceTest.yml

Added PR requested changes and comments

backend Fix

This reverts commit 89d1912.

  • remove log entirely

  • update release notes

  • Performance Benchmarking Log monitor and TPS tests (Performance Benchmarking Log monitor and TPS tests #513)

  • added concurrent log monitoring/writing to test and tests several combinations of # of logs monitored and different tps

  • fixed call to StartLogWrite

  • restructured json packet to send to database. CWA configs are now generated at runtime. log number is given as an environment variable so tests can run concurrently, so commented code is included for that change

  • rewrote how databse entry packets are crafted/create, defer log stream/group deletions, addressed other small pr comments

  • removed extra metrics from CWA config

  • separated delete log group and streams so they can be called separately, but kept the original function definition as to not break other code. removed bad calls to DeleteLogGroupAndStreams

  • testing with GAVIN'S code

  • debug print check

  • changed parser

  • testing with concurrency version

  • changeg setenv to string

  • added comments

Co-authored-by: Andrew Huynh 17056456+SaxyPandaBear@users.noreply.github.com
Co-authored-by: gmealy1 107051245+gmealy1@users.noreply.github.com

  • demo-0

  • test for pr

  • testing running performance test on prev. releases

  • test 1

  • test - 2 pRPT

  • genMatrix test

  • test

  • test

  • t

  • t

  • t

  • te

  • test

  • test

  • test

  • test

  • test

  • test

  • test

  • t

  • t

  • testing hard coded

  • full test w 3 old releases

  • test

  • testing with default

  • redid terraform for previousData.yml

  • debugging

  • test binary making

  • trying with new terraform

  • try

  • testing new terraform with git stash

  • forgot comma

  • one more comma :/

  • sha to old_sha

  • testing getting releases on workflow

  • same

  • test

  • t

  • no formatting

  • git show --tags

  • trying with sep shell script

  • trying .sh directly

  • log cat

  • :/

  • git

  • git show --tags

  • trying query tag

  • git fetch

  • git fetch w show

  • simplify

  • git tests

  • query

  • git describe tags

  • :(

  • not shallow

  • query t

  • testing

  • t

  • 3 test

  • testing update config

  • added concurrent update release tag

  • try2

  • Testing releases from 1627262978 to 1658798978

  • working release

  • Testing releases from 1627262978 to 1653934651

  • Testing releases from 1622398651 to 1653934651

  • Testing releases from to

  • Testing releases from Sun May 30 18:17:31 UTC 2021 to Mon May 30 18:17:31 UTC 2022

  • Testing releases from 20210530 to 20220530

  • updated shell

  • Testing releases from 2021/05/30 to 2022/05/30

  • updated previousData.yml

  • fixed stoplocal dep

  • generator duplicate fix

  • fixed the assume role in env

  • Testing releases from 2021/05/30 to 2022/05/30

  • testing

Co-authored-by: Andrew Huynh 17056456+SaxyPandaBear@users.noreply.github.com
Co-authored-by: gmealy1 107051245+gmealy1@users.noreply.github.com

Description of the issue

Describe the problem or feature in addition to a link to the issues.

Description of changes

How does this change address the problem?

License

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Tests

Describe what tests you have done.

Requirements

Before commit the code, please do the following steps.

  1. Run make fmt and make fmt-sh
  2. Run make linter

* Basic functionality for backend transmitter

removed unit tests

playing around with github

changed package name

seperated the test case

reformatted

removed seperate test

changed package

new IntegrationTest Setup

fixed integrationTest.yml

applied pr feedback:
-added new stored data called "CommitDate"
-reorganized code
-updated code to go v2
-changed metrics struct

cleaned workflow init

auto run

typo auto test

cleanworkflow init

added performance test to integrationTest

changed epoch time to int
-after that became an issue in frontend

Delete fastPerformanceTest.yml

Added PR requested changes and comments

backend Fix

* go mod tidy update

* Workflow (#5)

* concurrent workflow added
*update item added
*release detection added

* edited test

* added pt to master

* remove extra vars

* go mod tidy

* fixed imports

* import bug

* fixed value bug

* updated iam roles

* New workflow (#8)

* fixed sendItem nil map error

* added concurreny retrying

* fixed no value bug

* fixed value bug

* demo-4

* demo 1

* Fix debug logging and revert Windows ctl script PS1 changes (#516)

* change to use fmt

* Revert "Using cmd/c to avoid PowerShell ISE does not accept stderr (#473)"

This reverts commit 89d1912.

* remove log entirely

* update release notes

* Performance Benchmarking Log monitor and TPS tests (#513)

* added concurrent log monitoring/writing to test and tests several combinations of # of logs monitored and different tps

* fixed call to StartLogWrite

* restructured json packet to send to database. CWA configs are now generated at runtime. log number is given as an environment variable so tests can run concurrently, so commented code is included for that change

* rewrote how databse entry packets are crafted/create, defer log stream/group deletions, addressed other small pr comments

* removed extra metrics from CWA config

* separated delete log group and streams so they can be called separately, but kept the original function definition as to not break other code. removed bad calls to DeleteLogGroupAndStreams

* testing with _GAVIN'S_ code

* debug print check

* changed parser

* testing with concurrency version

* changeg setenv to string

* added comments

Co-authored-by: Andrew Huynh <17056456+SaxyPandaBear@users.noreply.github.com>
Co-authored-by: gmealy1 <107051245+gmealy1@users.noreply.github.com>

* demo-0

* test for pr

* testing running performance test on prev. releases

* test 1

* test - 2 pRPT

* genMatrix test

* test

* test

* t

* t

* t

* te

* test

* test

* test

* test

* test

* test

* test

* t

* t

* testing hard coded

* full test w 3 old releases

* test

* testing with default

* redid terraform for previousData.yml

* debugging

* test binary making

* trying with new terraform

* try

* testing new terraform with git stash

* forgot comma

* one more comma :/

* sha to old_sha

* testing getting releases on workflow

* same

* test

* t

* no formatting

* git show --tags

* trying with sep shell script

* trying .sh directly

* log cat

* :/

* git

* git show --tags

* trying query tag

* git fetch

* git fetch w show

* simplify

* git tests

* query

* git describe tags

* :(

* not shallow

* query t

* testing

* t

* 3 test

* testing update config

* added concurrent update release tag

* try2

* Testing releases from 1627262978 to 1658798978

* working release

* Testing releases from 1627262978 to 1653934651

* Testing releases from 1622398651 to 1653934651

* Testing releases from  to

* Testing releases from Sun May 30 18:17:31 UTC 2021 to Mon May 30 18:17:31 UTC 2022

* Testing releases from 20210530 to 20220530

* updated shell

* Testing releases from 2021/05/30 to 2022/05/30

* updated previousData.yml

* fixed stoplocal dep

* generator duplicate fix

* fixed the assume role in env

* Testing releases from 2021/05/30 to 2022/05/30

* testing

Co-authored-by: Andrew Huynh <17056456+SaxyPandaBear@users.noreply.github.com>
Co-authored-by: gmealy1 <107051245+gmealy1@users.noreply.github.com>
@sethAmazon sethAmazon requested a review from a team as a code owner August 11, 2022 14:27
@sethAmazon sethAmazon merged commit 1f0547d into previousReleases Aug 11, 2022
@lisguo lisguo deleted the performance-old-boy branch July 31, 2024 17:37
dchappa pushed a commit that referenced this pull request Sep 3, 2024
Co-authored-by: Seth L <81644108+sethAmazon@users.noreply.github.com>
Co-authored-by: Chad Patel <chadpatel@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Github Action <action@github.com>
Co-authored-by: Jeffrey Chien <chienjef@amazon.com>
Co-authored-by: Adam <90734270+adam-mateen@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants