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

hammer: mon sync does not copy config-key #7576

Merged
merged 1 commit into from Feb 24, 2016
Merged

hammer: mon sync does not copy config-key #7576

merged 1 commit into from Feb 24, 2016

Conversation

ghost
Copy link

@ghost ghost commented Feb 9, 2016

Signed-off-by: Xiaowei Chen <chen.xiaowei@h3c.com>
(cherry picked from commit b77cd31)
@ghost ghost self-assigned this Feb 9, 2016
@ghost ghost added this to the hammer milestone Feb 9, 2016
@ghost ghost added bug-fix core labels Feb 9, 2016
ghost pushed a commit that referenced this pull request Feb 9, 2016
Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Feb 15, 2016
Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Feb 16, 2016
Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Feb 22, 2016
Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost ghost assigned liewegas and unassigned ghost Feb 24, 2016
@ghost
Copy link
Author

ghost commented Feb 24, 2016

@liewegas does this backport look good to merge ? It passed a run of the hammer rados suite ( see http://tracker.ceph.com/issues/14692#note-9 for details ). The upgrade/hammer suite also passes ( http://tracker.ceph.com/issues/14692#note-11 ) after a few tries to get rid of environmental noise.

liewegas added a commit that referenced this pull request Feb 24, 2016
hammer: mon sync does not copy config-key
@liewegas liewegas merged commit b73a93d into ceph:hammer Feb 24, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant