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

Merge release 2.5.0.0 into master #2448

Merged
merged 50 commits into from
Dec 20, 2021
Merged

Merge release 2.5.0.0 into master #2448

merged 50 commits into from
Dec 20, 2021

Conversation

narrieta
Copy link
Member

No description provided.

kevinclark19a and others added 30 commits June 30, 2021 11:47
* 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>
* 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>
* Update hostgaplugin header before fetching vmsettings

* Remove retry loop

* Report empty etag as error

Co-authored-by: narrieta <narrieta>
larohra and others added 19 commits August 13, 2021 10:43
* 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: Laveesh Rohra <larohra@microsoft.com>
#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>
@codecov
Copy link

codecov bot commented Dec 20, 2021

Codecov Report

Merging #2448 (40f750c) into master (cf8a893) will increase coverage by 0.25%.
The diff coverage is 84.33%.

Impacted file tree graph

@@            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     
Impacted Files Coverage Δ
azurelinuxagent/common/logcollector_manifests.py 100.00% <ø> (ø)
azurelinuxagent/common/utils/archive.py 84.35% <ø> (+0.68%) ⬆️
azurelinuxagent/pa/rdma/factory.py 56.25% <0.00%> (ø)
azurelinuxagent/common/persist_firewall_rules.py 76.92% <50.00%> (-0.14%) ⬇️
azurelinuxagent/common/utils/textutil.py 62.50% <63.63%> (+0.05%) ⬆️
azurelinuxagent/common/cgroupconfigurator.py 70.27% <70.96%> (-1.00%) ⬇️
azurelinuxagent/common/protocol/hostplugin.py 85.99% <75.00%> (-0.51%) ⬇️
azurelinuxagent/ga/collect_logs.py 82.26% <80.00%> (-2.35%) ⬇️
azurelinuxagent/agent.py 57.43% <81.25%> (+3.28%) ⬆️
azurelinuxagent/common/protocol/wire.py 78.15% <81.81%> (-0.58%) ⬇️
... and 23 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cf8a893...40f750c. Read the comment docs.

@narrieta narrieta merged commit d2f61b0 into master Dec 20, 2021
@narrieta narrieta deleted the release-2.5.0.0 branch June 8, 2022 16:42
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.

6 participants