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
ceph: use next ceph v16.2.6 pacific version #8743
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.
LGTM, but adding changes requested until the cephfs issue is resolved
This pull request has merge conflicts that must be resolved before it can be merged. @leseb please rebase it. https://rook.io/docs/rook/master/development-flow.html#updating-your-fork |
Fixed in the third commit. |
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.
This new approach also works for pre-16.2.6, right? I guess we can see with the integration tests...
Yes it should, @batrick suggested that we should have had it like this for some time actually. |
9a5c043
to
2bbb420
Compare
db74ae7
to
3f4a47e
Compare
pkg/operator/ceph/file/mds/mds.go
Outdated
// In Pacific, standby-replay daemons are stopped automatically. Older versions of Ceph require us to stop these daemons manually. | ||
// In Pacific, standby-replay daemons are stopped automatically. Older versions of Ceph require | ||
// us to stop these daemons manually. | ||
// TODO: so why don't we have a version check? |
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.
This looks like it's always failing replay daemons whether it's pacific or not. @jshen28 Do we need to check for Pacific or was this comment outdated?
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.
@leseb Shall we open an issue for this and remove the TODO?
New version is out so let's use it. Signed-off-by: Sébastien Han <seb@redhat.com>
The new version v16.2.6 has a different behavior when it comes to the number of cephfs-mirror socket files. Previous version had exactly 3 and now has like way more... So let's just check for the presence of more sockets. Signed-off-by: Sébastien Han <seb@redhat.com>
The MDS core team suggested with deploy the MDS daemon first and then do the filesystem creation and configuration. Reversing the sequence lets us avoid spurious FS_DOWN warnings when creating the filesystem. Closes: rook#8745 Signed-off-by: Sébastien Han <seb@redhat.com>
ceph: use next ceph v16.2.6 pacific version
ceph: use next ceph v16.2.6 pacific version (backport #8743)
Description of your changes:
Which issue is resolved by this Pull Request:
Resolves #
Checklist:
make codegen
) has been run to update object specifications, if necessary.