-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
core: bump to ceph 16.2.7 #9343
Conversation
@@ -45,7 +45,7 @@ const ( | |||
// test with the latest octopus build | |||
octopusTestImage = "quay.io/ceph/ceph:v15" | |||
// test with the latest pacific build | |||
pacificTestImage = "quay.io/ceph/ceph:v16.2.6" | |||
pacificTestImage = "quay.io/ceph/ceph:v16.2.7" |
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.
Shall we just change this to the latest like with the octopus test tag?
pacificTestImage = "quay.io/ceph/ceph:v16.2.7" | |
pacificTestImage = "quay.io/ceph/ceph:v16" |
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.
Probably yes.
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.
And let's use 16.2 instead so we don't test on pre-release.
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.
v16 is expected to point to the latest v16 patch release, so currently is the same as v16.2.7. When would it be a prerelease?
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.
By pre-release I meant tags like 16.0 and 16.1, which are like alpha and beta tag IIRC.
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.
Sure, and since the v16 tag always points to the latest v16.2.x, the v16 and v16.2 tags are equivalent now.
ceph 16.2.7 is out so let's use it in our manifests. Signed-off-by: Sébastien Han <seb@redhat.com>
core: bump to ceph 16.2.7 (backport #9343)
Description of your changes:
ceph 16.2.7 is out so let's use it in our manifests.
Signed-off-by: Sébastien Han seb@redhat.com
Which issue is resolved by this Pull Request:
Resolves #
Checklist:
make codegen
) has been run to update object specifications, if necessary.