Conversation
3aa64d2 to
93661d5
Compare
93661d5 to
1dfd81f
Compare
sabaini
left a comment
There was a problem hiding this comment.
Great to see the squid build works! -- think we might want to hold off before making S the default though
| - name: Print logs for failure | ||
| if: failure() | ||
| run: | | ||
| sudo snap logs microceph -n 1000 |
There was a problem hiding this comment.
Hm, I wish we could reduce the duplication of those print log stanzas here
There was a problem hiding this comment.
I am not sure if we can abstract this step out given, The actual step is a one-liner (and rest is step description in each run.).
| package-repositories: | ||
| - type: apt | ||
| cloud: bobcat | ||
| cloud: caracal |
There was a problem hiding this comment.
Not sure we want to build for Squid by default before having a stable upstream release.
In any case we should cut a reef branch before switching master
There was a problem hiding this comment.
I think these are independent. As soon as we switch the reef track to a stable branch we can and should also build/publish the squid snap (without a stable risk level maybe).
Signed-off-by: Utkarsh Bhatt <utkarsh.bhatt@canonical.com>
Ceph has a dependency on distutils, which was removed from the packaging since it was an undeclared package dependency. This change adds setuptools package to work around this dependency. Since distutils has been removed from python 12, this also reverts to core22. Signed-off-by: Billy Olsen <billy.olsen@canonical.com>
RGW requires liblmbdb but this is not staged. Ensure that this is staged into the snap. Signed-off-by: Billy Olsen <billy.olsen@canonical.com>
Signed-off-by: Utkarsh Bhatt <utkarsh.bhatt@canonical.com>
Signed-off-by: Utkarsh Bhatt <utkarsh.bhatt@canonical.com>
44f97f5 to
03fdcb2
Compare
|
@UtkarshBhatthere think we did this now elsewhere, guess this PR could be closed? |
Description
Build MicroCeph using Ceph Squid from the Jammy-Caracal pocket
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
The new build is expected to successfully comply with all existing CI tests.
Contributor's Checklist
Please check that you have: