-
Notifications
You must be signed in to change notification settings - Fork 18
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
Only install and refresh lxd once #73
Conversation
addresses #72 as well by setting the default juju channel to |
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.
Supplying commentary
default: "latest/stable" | ||
default: "3/stable" |
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.
addresses the closing of the latest track of juju
if ([null, ""].includes(channel) == false) { | ||
yield snap(`refresh lxd --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.
This is no longer necessary with the above change
8ce545d
to
311e8de
Compare
@ca-scribner this is ready for a secondary review if you're okay w/ it |
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!
These changes only refresh the lxd snap to a single channel, rather than allowing it to flip to channels during setup
In the event the workflow is called like this:
lxd-channel defaults to
latest/stable
this causes the action to
Rather than going through jumps here, let's let the action install the correct one at the beginning.