-
Notifications
You must be signed in to change notification settings - Fork 522
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
defaults, migrations: admin container version bump to v0.6.0 #1347
defaults, migrations: admin container version bump to v0.6.0 #1347
Conversation
sources/api/migration/migrations/v1.0.6/admin-container-v0-6-0/src/main.rs
Outdated
Show resolved
Hide resolved
Set default admin container version to v0.6.0
9ca000f
to
7dfc6d4
Compare
|
sources/api/migration/migrations/v1.0.6/admin-container-v0-6-0/src/main.rs
Outdated
Show resolved
Hide resolved
Migrates default version of admin container from v0.5.2 to v0.6.0
7dfc6d4
to
fb450cc
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.
Looks good to me, once the testing is complete.
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.
💸
Once testing is complete I'm happy with this.
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.
Tested the migrations via upgrade/downgrade on a Bottlerocket host and it works as expected (source template gets replaced, admin container source points to v0.6.0)
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.
Nice work!
Issue number:
N/A
Description of changes:
Migrates default version of admin container from v0.5.2 to v0.6.0
Testing done:
Terms of contribution:
By submitting this pull request, I agree that this contribution is dual-licensed under the terms of both the Apache License, version 2.0, and the MIT license.