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

Only restart LXD if version is less than 5.21 #286

Merged
merged 1 commit into from
Apr 24, 2024

Conversation

masnax
Copy link
Contributor

@masnax masnax commented Apr 17, 2024

With the content interfaces in places in LXD to detect MicroOVN and MicroCeph, we don't need to restart LXD to pick them up anymore.

However, if we don't restart, this breaks support for versions of LXD prior to 5.21 when the content interface was introduced. We can't directly fetch the snap version or the available interfaces of LXD due to the snap confinement, so the best I came up with was to hard-code 5.21 into MicroCloud, and compare it to the server version reported from /1.0 in LXD.

If LXD is not at least version 5.21, then we need to restart the LXD daemon to pick up MicroOVN and MicroCeph integration. Otherwise we can just expect it to be there.

@masnax masnax changed the title Only restart LXD if version is less than 5.12 Only restart LXD if version is less than 5.21 Apr 17, 2024
Copy link
Member

@tomponline tomponline left a comment

Choose a reason for hiding this comment

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

Ready for merge once the comments are improved.

@simondeziel
Copy link
Member

The test failures are addressed by #289

@tomponline
Copy link
Member

@masnax please can you rebase to get the tests passing. Ta

@tomponline tomponline merged commit a096a77 into canonical:main Apr 24, 2024
10 of 12 checks passed
simondeziel added a commit to simondeziel/microcloud that referenced this pull request Apr 24, 2024
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.

3 participants