service: Support latest MicroCeph tentacle#1338
Draft
roosterfish wants to merge 6 commits intocanonical:mainfrom
Draft
service: Support latest MicroCeph tentacle#1338roosterfish wants to merge 6 commits intocanonical:mainfrom
roosterfish wants to merge 6 commits intocanonical:mainfrom
Conversation
roosterfish
changed the title
roosterfish
force-pushed
the
tentacle_support
branch
2 times, most recently
from
df57838 to
d75e7aa
Compare
There was a problem hiding this comment.
Pull request overview
Updates MicroCloud’s service version validation to accept the latest MicroCeph “tentacle” (newer major Ceph versions) rather than rejecting them.
Changes:
- Adjust MicroCeph version parsing/validation to use the shared
compareVersionlogic (allowing versions newer than the minimum). - Update version validation tests to treat a newer-major MicroCeph/Ceph version as supported.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| service/version.go | Switch MicroCeph validation to extract a semantic version from the daemon string and validate it via compareVersion. |
| service/version_test.go | Update test coverage to assert newer-major MicroCeph/Ceph versions are supported. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
roosterfish
force-pushed
the
tentacle_support
branch
from
d75e7aa to
e29e73b
Compare
roosterfish
force-pushed
the
tentacle_support
branch
from
877a156 to
a891c47
Compare
Signed-off-by: Julian Pelizäus <julian.pelizaeus@canonical.com>
Signed-off-by: Julian Pelizäus <julian.pelizaeus@canonical.com>
Signed-off-by: Julian Pelizäus <julian.pelizaeus@canonical.com>
Signed-off-by: Julian Pelizäus <julian.pelizaeus@canonical.com>
roosterfish
force-pushed
the
tentacle_support
branch
from
a891c47 to
fe98e21
Compare
MicroCeph tentacle uses Microcluster v3 which uses the control unix socket to serve the SQL endpoint. Signed-off-by: Julian Pelizäus <julian.pelizaeus@canonical.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
snap tends to perform a retry itself. Also due to current state of runners retrying might not anymore be required. The retry call is still left for apt operations. Signed-off-by: Julian Pelizäus <julian.pelizaeus@canonical.com>
Contributor
Author
|
The upgrade test on 22.04 fails due to MicroCeph tentacle. LXD cannot start the CephFS storage pool:
It works with 24.04 so likely kernel related. There also is a similar report here lxc/incus#2748. |
roosterfish
added a commit
that referenced
this pull request
Apr 21, 2026
MicroCeph tentacle doesn't play nicely on 22.04 which causes upgrade test failures in #1338. Therefore let's pin MicroCeph squid edge for now to unblock the many other PRs waiting to be included (also in preparation to release another version of MicroCloud 2 for future LXD 6.8 support). Also ensure latest version of snapd to not get into conflicts when trying to install `core26`.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Support latest MicroCeph tentacle.