-
Notifications
You must be signed in to change notification settings - Fork 510
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
Update host containers #3911
Update host containers #3911
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should not expose the user data in the PR description. This can leak sensitive data. For example, your user data actually contains ssh key details.
{"ssh":{"authorized-keys":["ssh-rsa AAAAB3NzaC1yc2EAAAADAQA... vyaghras-us-west-2"]}}%
technically speaking your key is compromised. It would be better if you revoke the key.
@monirul |
d2ac549
to
85d959d
Compare
⬆️ Fix the build error for |
85d959d
to
060a073
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Description of changes:
Update the admin and control host containers to bottlerocket-admin-container v0.11.7 and bottlerocket-control-container v0.7.11. Also add migrations move to these versions on upgrade.
Testing done:
cargo make check-migrations
v1.19.4
tov1.19.5
, host container versions are correctly picked up. (Same output as above)v1.19.5
fromv1.19.4
.