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

mgr/DaemonServer.cc: add 'is_valid=false' when decode caps error #16978

Merged
merged 1 commit into from Aug 12, 2017

Conversation

gmayyyha
Copy link
Contributor

@gmayyyha gmayyyha commented Aug 11, 2017

Signed-off-by: Yanhu Cao <gmayyyha@gmail.com>
@gmayyyha gmayyyha changed the title mgr/DaemonServer.cc: add 'isvalid=false' when decode caps error mgr/DaemonServer.cc: add 'is_valid=false' when decode caps error Aug 11, 2017
Copy link
Member

@joscollin joscollin left a comment

Choose a reason for hiding this comment

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

LGTM.

Fixed from the comments of #16891

@jcsp
Copy link
Contributor

jcsp commented Aug 11, 2017

Since this is a bug fix we'd want to backport, could you please create a ticket on tracker.ceph.com, and update the commit message here to have a "Fixes: " line referring to it?

@joscollin
Copy link
Member

@jcsp @gmayyyha

You can create one tracker for all the related PRs: #16978, #16891. I think there is one more ?

@yuriw yuriw merged commit aaf32d8 into ceph:master Aug 12, 2017
@gmayyyha gmayyyha deleted the mgr-ms-verify-auth branch August 14, 2017 01:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants