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

daemon-base: bump nfs-ganesha 4 > 5 (quincy/el9) #2213

Closed
wants to merge 1 commit into from

Conversation

guits
Copy link
Collaborator

@guits guits commented May 29, 2024

Quincy el9 based image should ship nfs-ganesha-5.
(nfs-ganesha-4 doesn't exist for CentOS 8)

Fixes: https://tracker.ceph.com/issues/66253

@guits
Copy link
Collaborator Author

guits commented May 29, 2024

@ljflores FYI

@guits guits force-pushed the fix-quincy-el9-ganesha branch 2 times, most recently from b66b16e to df7306c Compare May 29, 2024 15:37
Copy link
Contributor

@ljflores ljflores left a comment

Choose a reason for hiding this comment

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

Actually @guits it looks like the CI failures below only occur on c9 jobs:

CONTAINER ID   IMAGE     COMMAND   CREATED   STATUS    PORTS     NAMES
CONTAINER ID   IMAGE     COMMAND   CREATED   STATUS    PORTS     NAMES
Error response from daemon: No such container: ceph-demo
Error: Process completed with exit code 1.

I would recheck the "ceph-demo" references.

@guits
Copy link
Collaborator Author

guits commented May 30, 2024

Actually @guits it looks like the CI failures below only occur on c9 jobs:

CONTAINER ID   IMAGE     COMMAND   CREATED   STATUS    PORTS     NAMES
CONTAINER ID   IMAGE     COMMAND   CREATED   STATUS    PORTS     NAMES
Error response from daemon: No such container: ceph-demo
Error: Process completed with exit code 1.

I would recheck the "ceph-demo" references.

No. You are seeing this because the ceph-release RPM isn't available at download.ceph.com for both squid and quincy.
Those github workflow tests run a set of command, including a podman ps -a when a failure is caught. Obviously, if an error happens before the container is started, you won't see anything.

By the way, reef el9 test which has well its ceph-release RPM at download.ceph.com works, see [1]

[1] https://github.com/ceph/ceph-container/actions/runs/9289060252/job/25586978591?pr=2213

@yuriw
Copy link

yuriw commented Jun 1, 2024

@guits pls merge!

@guits guits force-pushed the fix-quincy-el9-ganesha branch 3 times, most recently from a2acda7 to 299f80a Compare June 3, 2024 22:15
This adds el9 based OS build testing.

Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
@guits
Copy link
Collaborator Author

guits commented Jun 3, 2024

closing in favor of #2218

@guits guits closed this Jun 3, 2024
@guits guits deleted the fix-quincy-el9-ganesha branch June 3, 2024 22:41
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