-
Notifications
You must be signed in to change notification settings - Fork 372
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
Merge release 2.5.0.0 into master #2448
Merged
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
* Allow systemd-run in the Agent's cgroup * Fix indentation * Fix indentation * Use /proc/comm instead of /proc/cmdline Co-authored-by: narrieta <narrieta>
Co-authored-by: narrieta <narrieta>
Co-authored-by: narrieta <narrieta>
* Added support for vmSettings' ETag * Added comment Co-authored-by: narrieta <narrieta>
Co-authored-by: narrieta <narrieta>
…he status file (#2270) * Code changes to include new files in history folder * Modified the Json Schema to load to waagent_status and fixed one test case * Fixed Pylint errors * Minor Fix * Code clean up * Updated name * Improved code * Addressed review comments * Addressed Pylint Error * Addressed Pylint Error * Addressed PR Comments * Fixed Pylint error * Fixed Pylint error * Fixed test cases * Modified write_ext_handlers_status_to_info_file method to consume from status_blob . Added test cases * Fixed pylint errors * Fixed pylint errors * Addressed PR reviews * Addressed Pylint error * Addressed a PR comment * Added comments * Addressed a PR comment * Addressed a PR comment * Fixed Pylint
* Adding the new file to log collector * Removed the old file
Merge hotfix-2.3.1 to develop
* Use If-None-Match for ETag header * Update goal state iff response includes etag Co-authored-by: narrieta <narrieta>
Co-authored-by: narrieta <narrieta>
* Created a helper function to format exceptions * Fixed Pylint error * Updated the format_exception function to accept exception as a parameter * Removed logger * Fixed Pylint error * Fixed logging and updated logic to process exceptions * Fixed logging and updated logic to process exceptions * Fixed minor PR comments * Fixed minor PR comments
Co-authored-by: narrieta <narrieta>
Merge fast-track into develop
Co-authored-by: narrieta <narrieta>
* add and remove extension slice * addressed PR comments * update constant
* Improve logging for vmSettings requests * Remove comment * Fix pylint warning Co-authored-by: narrieta <narrieta>
* Handle HTTP GONE in vmSettings request * Fix python 2 compatibility issue Co-authored-by: narrieta <narrieta>
Co-authored-by: narrieta <narrieta>
* Update hostgaplugin header before fetching vmsettings * Remove retry loop * Report empty etag as error Co-authored-by: narrieta <narrieta>
* Refactor write_ext_handlers_status_to_info_file function * Addressed code review comment
Co-authored-by: narrieta <narrieta>
…goal state processing (#2332) * Implement InitialGoalStatePeriod parameter + improvements in logging of goal state processing * re-add test * updated test comment * Review feedback; default InitialGoalStatePeriod to GoalStatePeriod; unit tests Co-authored-by: narrieta <narrieta>
* Fix operation name in InitializeHostPlugin event * fix unit test Co-authored-by: narrieta <narrieta>
Co-authored-by: narrieta <narrieta>
…it with the same value as the prod flag (#2339)
#2358) * Use handler status if extension status is None when computing the ExtensionsSummary * Add NotReady to convergence statuses Co-authored-by: narrieta <narrieta>
* define ExtensionsSummary.__eq__ * Fix test name * Fix test name * Fix test name Co-authored-by: narrieta <narrieta>
narrieta
requested review from
dhivyaganesan,
kevinclark19a,
larohra,
nagworld9 and
ZhidongPeng
as code owners
December 20, 2021 18:56
Codecov Report
@@ Coverage Diff @@
## master #2448 +/- ##
==========================================
+ Coverage 70.63% 70.88% +0.25%
==========================================
Files 96 97 +1
Lines 13889 14115 +226
Branches 1987 2251 +264
==========================================
+ Hits 9811 10006 +195
- Misses 3647 3662 +15
- Partials 431 447 +16
Continue to review full report at Codecov.
|
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.
No description provided.