Automated sync with upstream - last commit 5ace5aa (#45)

* Readme (#601)

* Update testing README to reflect up-to-date requirements

- Background: Some process improvements were made to automate setup steps.
- Problem: README wasn't updated with the new steps and IAM policy requirements. Couldn't get the workflow to reach a stage where it runs ec2 integraion tests
- Fix: Updated README
- Verify: Ran on my personal fork and verified ec2 integ tests start running.
- Next: Some cleanup steps may be still failing, as well as other non-ec2 tests. Will continue to update README whenever new issues are encountered.

* Remove link to internal wiki

* Remove Windows Tests Until We Figure Out SSH Without Windows Passwrod (#602)

* Fix Broken Tests Not Being Run (#603)

* Add iam role policy in readme for test cleanup (#608)

* Initial support for ubuntu 22.04 (#589)

* Add ARM64 Darwin Support (#607)

* Test CPU metric values greater than zero (#609)

* Add cpu_usage_active test to verify >0 value

* Add all cpu metric test and refactoring for extensibility

* Linter

* Improve code quality based on reviews

* Found how to use composition for shared interface implementation

* Simplfy based on review comments

* Add Release Notes For Version 1.247355.0 (#612)

Update Cloud Watch Container Version

* Refactor benchmark test package for future tests (#611)

* Create suite for all benchmark tests

* Pull out agent runner function for reuse

* Print test suite result summary together

* Fix incorrect test result accumulation logic

* Fix suite assert misusage

* Separate out files per test group

* Fix undefined variable mistake

* Separate out placeholder test

* Move add test result function out of cpu file

* Make fmt changes

* fix syntax error

* Refactor using template method pattern

* make fmt result

* Fix swallowing error cause problem

* Cleanup config file after test

* Sanity test validation fix (#614)

* Change sanity validation from >0 to >=0 to match manual test validation process

* Fix quest typos to guest that causes test failure

* Fix more quest typos

* Change error message to reflect validation requirement change

* e2e mem sanity test (#617)

* add mem e2e test, clean up integ test workflow/docs

* fix placeholder runner, add mem test to benchmark suite

* implement mem fetcher, code cleanup

* implement mem fetcher, code cleanup

* remove bad metric dimension

* fix measured metrics

* fix fetcher names

* update README, formatting

* add license header

* remove placeholder test (#618)

* Fix flaky aggregator shutdown test. (#619)

* Fix flaky aggregator shutdown test.

* E2e test package separation (#622)

* Fix generateTestMatrix workflow step

* Fix startlocalstack step

* Fix ec2linuxintegrationtest

* Fix stoplocalstack

* Fix MakeMSIZip

* Fix checkout repository argument

* Fix MakeMacPkg

* Fix ECSFargateIntegrationTest

* Fix EC2NvidiaGPUIntegrationTest

* Fix PerformanceTrackingTest

* Fix matrix generation invalid file path

* Use env variables for test repo name and url

* Fix Env variable failure (#623)

* CVE-2022-29583 (#624)

Co-authored-by: jean hamman <jean@sharpspring.com>

* Comment out performance test that is buggy and no public release (#630)

DDB primary index shouldn't be YEAR+CommitDate as that makes the workflow fail if there were two commits on the same day

* Fix parsing win event log message (#627)

* Fix win event message to insert strings for placeholders

* Add test cases for insertion strings

* Add comment to insertPlaceholderValues

* Update test and test cases for insertPlaceholderValues

* Add UserData as a source of insertion strings

* Fix typo in comments. Update comments. Rename variable names

* Add test cases for EventData/UserData in windowsEventLogRecord

* Fix pointer issue in TestUnmarshalWinEvtRecord

* Delete integration directory after test package separation (#625)

* Delete integration directory after test package separation

* go mod tidy run changed go sum file

* go mod tidy after removing some dependencies

* go mod tidy with go 1.18.3 instead of 1.19

* go sum cleanup

* move daily cleanup code out from integration

* clean daily resources workflow fix

* add go.sum

* match go.mod to 1.18.3 results

* Remove internal/atomiccounter. Use sync/atomic from Golang 1.19. (#631)

* Remove internal/atomiccounter. Use sync/atomic from Golang 1.19.

* Fix github actions to use go 1.19. (#633)

* Disable nvidia gpu metrics on windows (it has never been released). (#632)

* Disable nvidia gpu metrics on windows (it has never been released).

Co-authored-by: Chena Lee <ChenaCLee@gmail.com>
Co-authored-by: Seth L <81644108+sethAmazon@users.noreply.github.com>
Co-authored-by: Khanh Nguyen <91758108+khanhntd@users.noreply.github.com>
Co-authored-by: Andrew Huynh <17056456+SaxyPandaBear@users.noreply.github.com>
Co-authored-by: Adam <90734270+adam-mateen@users.noreply.github.com>
Co-authored-by: Chena Lee <leechena@amazon.com>
Co-authored-by: J H <74546822+jshsp@users.noreply.github.com>
Co-authored-by: jean hamman <jean@sharpspring.com>
Co-authored-by: Thomas Yang <proty.ece96@g2.nctu.edu.tw>
Co-authored-by: Khanh Nguyen <khanhntd@amazon.com>

.github/workflows/build-test-linux.yml

@@ -23,12 +23,12 @@ concurrency:
 jobs:
   linux-unittest:
     runs-on: ubuntu-latest
-    steps:  
+    steps:
 
     - name: Set up Go 1.x
       uses: actions/setup-go@v2
       with:
-        go-version: ~1.18.3
+        go-version: ~1.19.2
 
     - name: Check out code
       uses: actions/checkout@v2
@@ -42,7 +42,7 @@ jobs:
 
     - uses: zencargo/github-action-go-mod-tidy@v1
       with:
-        go-version: 1.18.3
+        go-version: 1.19.2
 
     - name: Cache build output
       uses: actions/cache@v2

.github/workflows/build-test-macos.yml

@@ -10,7 +10,7 @@ on:
       - '**/*.md'
       - '.github/**'
       - '!.github/workflows/build-*'
-      
+
 
   pull_request:
     types: [opened, synchronize, reopened, ready_for_review]
@@ -23,12 +23,12 @@ concurrency:
 jobs:
   macos-unittest:
     runs-on: macos-latest
-    steps:  
+    steps:
 
     - name: Set up Go 1.x
       uses: actions/setup-go@v2
       with:
-        go-version: ~1.18.3
+        go-version: ~1.19.2
 
     - name: Check out code
       uses: actions/checkout@v2

.github/workflows/build-test-windows.yml

@@ -35,7 +35,7 @@ jobs:
       - name: Setup Go
         uses: actions/setup-go@v2
         with:
-          go-version: ~1.18.3
+          go-version: ~1.19.2
 
       - name: Cache Go
         uses: actions/cache@v2

.github/workflows/clean-aws-resources.yml

@@ -25,7 +25,7 @@ jobs:
           aws-region: us-west-2
 
       - name: Clean old ami
-        run: go run ./integration/clean/clean_ami/clean_ami.go --tags=clean
+        run: go run ./tool/clean/clean_ami/clean_ami.go --tags=clean
 
   clean-opensource-dedicated-hosts:
     runs-on: ubuntu-latest
@@ -43,7 +43,7 @@ jobs:
           aws-region: us-west-2
 
       - name: Clean old dedicated host
-        run: go run ./integration/clean/clean_dedicated_host/clean_dedicated_host.go --tags=clean
+        run: go run ./tool/clean/clean_dedicated_host/clean_dedicated_host.go --tags=clean
 
   clean-internal-dedicated-hosts:
     runs-on: ubuntu-latest
@@ -76,4 +76,4 @@ jobs:
           aws-region: ${{ matrix.region }}
 
       - name: Clean old dedicated host
-        run: go run ./integration/clean/clean_dedicated_host/clean_dedicated_host.go --tags=clean
+        run: go run ./tool/clean/clean_dedicated_host/clean_dedicated_host.go --tags=clean