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

[COS] Update cos-agent and libraries #141

Merged
merged 5 commits into from
Aug 30, 2024
Merged

Conversation

addyess
Copy link
Contributor

@addyess addyess commented Aug 9, 2024

Overview

Upgrade the cos_agent libraries to current without switching from pydanditc 2 since the libraries STILL support 1.*

Details

  • Update the cos_agent
  • Slightly the snap and containerd modules
  • Update the cosl library to current
  • Updated pydantic to current 1.x

@addyess addyess requested a review from a team as a code owner August 9, 2024 18:49
@addyess addyess force-pushed the KU-1264/cos-agent-update-0.10 branch 3 times, most recently from c9fbe44 to 409edb0 Compare August 27, 2024 23:16
Copy link
Contributor

@bschimke95 bschimke95 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, only questions

tests/integration/conftest.py Outdated Show resolved Hide resolved
charms/worker/k8s/lib/charms/k8s/v0/k8sd_api_manager.py Outdated Show resolved Hide resolved
@bschimke95
Copy link
Contributor

Also, seems like the integration tests are not happy for some reason.

* migrates pydantic 2 structures and validators
* removes pydantic 2 deprecation warnings
* updates cos_agent libraries
@addyess addyess force-pushed the KU-1264/cos-agent-update-0.10 branch from 12a93e3 to 88324e4 Compare August 30, 2024 02:25
@addyess addyess force-pushed the KU-1264/cos-agent-update-0.10 branch from 247d872 to b0bbfc0 Compare August 30, 2024 03:05
@addyess addyess changed the title [COS] Update to pydantic 2 [COS] Update pydantic Aug 30, 2024
@addyess addyess changed the title [COS] Update pydantic [COS] Update cos-agent and libraries Aug 30, 2024
@addyess addyess marked this pull request as ready for review August 30, 2024 03:09
@addyess addyess merged commit d307e17 into main Aug 30, 2024
34 checks passed
@addyess addyess deleted the KU-1264/cos-agent-update-0.10 branch August 30, 2024 12:36
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.

2 participants