Build squid Microceph #345
Conversation
UtkarshBhatthere
force-pushed
the
devel/squid
branch
4 times, most recently
from
3aa64d2 to
93661d5
Compare
UtkarshBhatthere
force-pushed
the
devel/squid
branch
from
93661d5 to
1dfd81f
Compare
| - 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.).
| @@ -12,7 +12,7 @@ confinement: strict | |||
|
|
|||
| 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 <[email protected]>
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 <[email protected]>
RGW requires liblmbdb but this is not staged. Ensure that this is staged into the snap. Signed-off-by: Billy Olsen <[email protected]>
Signed-off-by: Utkarsh Bhatt <[email protected]>
Signed-off-by: Utkarsh Bhatt <[email protected]>
UtkarshBhatthere
force-pushed
the
devel/squid
branch
from
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: