chore: update README.md remove cross compile #45
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
yimuniao
approved these changes
Jul 23, 2020
pxaws
approved these changes
Jul 23, 2020
sky333999
pushed a commit
that referenced
this pull request
Jul 21, 2023
* 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 <[email protected]> * 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 <[email protected]> Co-authored-by: Seth L <[email protected]> Co-authored-by: Khanh Nguyen <[email protected]> Co-authored-by: Andrew Huynh <[email protected]> Co-authored-by: Adam <[email protected]> Co-authored-by: Chena Lee <[email protected]> Co-authored-by: J H <[email protected]> Co-authored-by: jean hamman <[email protected]> Co-authored-by: Thomas Yang <[email protected]> Co-authored-by: Khanh Nguyen <[email protected]>
sky333999
added a commit
that referenced
this pull request
Jul 21, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of the issue
The cross compile line is not needed anymore
Description of changes
Removed the cross compile line from README.md
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
No test needed