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

README: Pass the --channel argument to snap refresh, not only --stable #534

Merged
merged 1 commit into from
Nov 15, 2023

Conversation

lorumic
Copy link
Contributor

@lorumic lorumic commented Nov 14, 2023

Done

On the README you can read the following:

Or refresh to ensure at least version 5.14 is installed

sudo snap refresh --stable lxd

But this doesn't work. If you are on 5.0.2, you'll get this output:

root@dear-dinosaur:~# snap refresh --stable lxd
lxd (5.0/stable) 5.0.2-838e1b2 from Canonical✓ refreshed

The latest/stable channel should be passed instead, to ensure that you refresh to at least the 5.14 version.

@lorumic lorumic added Documentation 📝 Documentation changes or updates Review: Code needed labels Nov 14, 2023
@lorumic lorumic requested a review from edlerd November 14, 2023 13:09
@webteam-app
Copy link

Demo starting at https://lxd-ui-534.demos.haus

@lorumic lorumic changed the title README: Pass the --channel argument to snap refresh, not only --stable README: Pass the --channel argument to snap refresh, not only --stable Nov 14, 2023
Copy link
Collaborator

@edlerd edlerd left a comment

Choose a reason for hiding this comment

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

good catch, thanks for the update!

Let's remember when the LTS is released to turn this back to stable -- or I guess then we can rely on the standard snap install lxd without channel or stable.

@edlerd edlerd merged commit fd02c0e into canonical:main Nov 15, 2023
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation 📝 Documentation changes or updates Review: Code +1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants