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

lxd: Initialise server name and global config before storage patches are run #12421

Merged
merged 4 commits into from
Oct 19, 2023

Conversation

tomponline
Copy link
Member

@tomponline tomponline commented Oct 19, 2023

Fixes start up issue for ZFS volumes from #12390

Signed-off-by: Thomas Parrott <thomas.parrott@canonical.com>
@tomponline tomponline self-assigned this Oct 19, 2023
And then reload them after patches have run in case its been modified.

Signed-off-by: Thomas Parrott <thomas.parrott@canonical.com>
simondeziel
simondeziel previously approved these changes Oct 19, 2023
@monstermunchkin
Copy link
Contributor

LGTM, but static analysis is not happy.

@tomponline
Copy link
Member Author

@monstermunchkin thanks. Please can you take over this PR and also review the other recent patches that retrieve custom volumes to make them use the same pattern.

@monstermunchkin monstermunchkin self-assigned this Oct 19, 2023
tomponline and others added 2 commits October 19, 2023 20:14
…sUnsetInvalidBlockSettingsV2

Signed-off-by: Thomas Parrott <thomas.parrott@canonical.com>
Signed-off-by: Thomas Hipp <thomas.hipp@canonical.com>
…sUnsetInvalidBlockSettings

Signed-off-by: Thomas Hipp <thomas.hipp@canonical.com>
@tomponline
Copy link
Member Author

@simondeziel please can you approve?

@simondeziel simondeziel merged commit 699be01 into canonical:main Oct 19, 2023
25 of 26 checks passed
@tomponline tomponline deleted the tp-servername-patch branch October 24, 2023 22:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants