Cloudzero aws main pull v1.247357.0#10
Merged
muellerk22 merged 204 commits intocloudzero-optimizedfrom Feb 20, 2023
Merged
Conversation
Add the 'aws_sdk_log_level' option to Agent's JSON config.
Updated schema to describe the new config option.
Allow multiple log levels to be specified separated by a vertical bar and optionally white space.
Updated all existing uses of aws.Config{} to use the new the new SDKLogger{} struct and SDKLogLevel() function.
Added unit tests.
* Added support for implicit NetworkMode. Fixed aws#330.
* Fix missing Windows events by changing getRecords(). Add unit tests.
* fix duplicate retention check * remove formatting change * fix test case messages * check retentionSettings in 1 loop * remove comments * add comment and fix formatting * handle case where the same retention setting for a loggroup is set in different places * update test cases and error message * remove extra white space
* testing sleep periods in GitHub CI * tweak sleep duration * tweak sleep duration
Co-authored-by: Andrew Huynh <17056456+SaxyPandaBear@users.noreply.github.com>
* Add a feature flag to user agent for container insights * Update agentinfo.OutputPlugins directly instead of parsing. * Add TestUserAgent and fix test cases. * Add function to reset user agent string. * Reset the user agent string before constructing for container insights. Refactor and add test cases. * Move the regex for container insights to top level to reuse it. * Refactor to store userAgent string in map with group name as key. * Move test cases for container_insights flag to agentinfo. * Update test for container_insights and minior refacotr. Co-authored-by: Thomas Yang <yangth@amazon.com>
Drop original metrics when aggregating 1. Translate drop original flags config at metrics level 2. Drop metrics based on the metric names when the output plugin is configured 3. Unit tests for config translation and config reading *4. resolve comments and refactor the metrics.go adding output plugin method. * support wildcard to drop all metrics
* add concurrency for pr build and test * add concurrency for integration test * fix concurrency for pr build * fix concurrency for pr build * add paths ignores and reduce branches to use master only * change to use github workflow instead of literal string * change to use whole path instead of abs path * change order of path ignore * change order of path ignore * delete space * fix pattern for build test windows * change fail fast to false Co-authored-by: Nguyen <khanhntd@88665a0eaf54.ant.amazon.com>
* Breakout unit testing per OS. Issue: aws#353 * removed upload step from build-test-macos.yml * added on push main, to support future migration to this branch * removed strategy since we aren't using matrix for github * removed unused step for linux and macos * removed the 'restore-keys' to avoid caching old dependencies pkgs * added back the upload to Codecov in macos * removed useless 'id' in the setup * removed restore-keys in windows workflow, changed the testing by installing make, doing make test and make build, added upload to codecov in the windows pipeline * change jobs name in linux and macos to match the windows pipeline * removed runner cache, based on test done by khanhntd * added license on top of files, added the on push main branch in build-test-windows.yml * removed useless 'Debug go.mod' step since we do not use restore-keys anymore
* Add CodeQl analysis * Add license for security check
* Export Task ARN resource via Prometheus * Populate cluster name and task id labels
* add filtering docs to log filtering section
* debug logs * more log * attempt fix the decoding * Fix with a dynamic length calculation, remove debug logs, add unit tests. * unit test run on Windows 2019 only
* add posttrans scriptlet that uses conditional restart after RPM upgrade * fix scriptlet * create aoc user if not exists on the host
Author
|
Tested locally and everything looks good. |
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
20 participants
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
Pulled from the AWS forked report. Mainly to pull in the cGroups v2 when upgrading cadviser
Description of changes
New version usig cGroup v2 should work.
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
Buit and deployed to cluster ensure etrics made it into CloudZero
Requirements
Before commit the code, please do the following steps.
make fmtandmake fmt-shmake linter