-
Notifications
You must be signed in to change notification settings - Fork 23
Change default Ceph release to Quincy #480
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My main concern here is the potential for surprise upgrades.
Do you prefer to wait until we have a zed branch? |
Let's discuss in today's infra meeting, but that might be safer. Deployments can always make these changes in their branches. |
I think we agreed to stay on pacific for now |
Pacific is EOL-ish - https://github.com/ceph/ceph/blob/main/doc/releases/releases.yml |
Yes. I'm not saying we shouldn't update clients to Quincy, but should we make such changes on a stable branch? |
In this case it probably is not problematic (you would need to run bootstrap against a running cluster to make some serious problems), but I understand where you're going - I don't know even if we should have a default in s-k-c. |
If we don't have a default in s-k-c then we depend on the default in the collection, which is less visible. We do specify the container image tag in s-k-c too, which is tied to the release |
So let's just assume we just update the default once, in the early stages of a branch - and that's it. |
Perhaps we should better document how to change the release in SKC docs? |
Is it time to close this PR @priteau @markgoddard? |
Should I target it to zed instead? |
It's already done: https://github.com/stackhpc/stackhpc-kayobe-config/blob/stackhpc/zed/etc/kayobe/cephadm.yml |
Not required anymore. |
No description provided.