-
Notifications
You must be signed in to change notification settings - Fork 0
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
Prevent automatic snap upgrades #8
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.
Couple of nits here and there... You'll make easy fixes
for snap in BASIC_SNAPS: | ||
install_snap( | ||
snap, | ||
channel=channel, | ||
classic=True, | ||
ignore_running=snap == "kubectl", | ||
) |
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.
❤️
for snap in CONTROL_PLANE_SNAPS: | ||
install_snap(snap, channel=channel) |
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.
❤️ ❤️
Co-authored-by: Adam Dyess <[email protected]>
Co-authored-by: Adam Dyess <[email protected]>
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
status.add( | ||
BlockedStatus("Snap channel version has changed. An upgrade is required.") | ||
) |
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.
@mateoflorido do you see any trouble changing this text to match the previous message in reactive:
Needs manual upgrade, run the upgrade action
Summary
This pull request implements a mechanism to prevent automatic upgrades.
Changes
channel
parameter is changed.BlockedStatus
if thechannel
parameter has changed and an upgrade is required.