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.

Sign up for GitHub

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

Jump to bottom

nautilus: mgr: synchronize ClusterState's health and mon_status. #34326

Merged
merged 1 commit into from
Apr 17, 2020
Merged

nautilus: mgr: synchronize ClusterState's health and mon_status. #34326

merged 1 commit into from
Apr 17, 2020

Conversation

rzarzynski
Copy link
Contributor

@rzarzynski rzarzynski commented Mar 31, 2020 •
edited by smithfarm
Loading

backport tracker: https://tracker.ceph.com/issues/44868

Fixes: https://tracker.ceph.com/issues/24995
Signed-off-by: Radoslaw Zarzynski rzarzyns@redhat.com
(cherry picked from commit da36267)

PR for master: #34245.

Checklist

  • References tracker ticket
  • Updates documentation if necessary
  • Includes tests for new functionality or reproducer for bug
Show available Jenkins commands
  • jenkins retest this please
  • jenkins test crimson perf
  • jenkins test signed
  • jenkins test make check
  • jenkins test make check arm64
  • jenkins test submodules
  • jenkins test dashboard
  • jenkins test dashboard backend
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox

@rzarzynski
mgr: synchronize ClusterState's health and mon_status.
1f87c58 
Fixes: https://tracker.ceph.com/issues/24995
Signed-off-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
(cherry picked from commit da36267)
@rzarzynski rzarzynski added this to the nautilus milestone Mar 31, 2020
smithfarm
smithfarm previously requested changes Apr 1, 2020
View reviewed changes
Copy link
Contributor

@smithfarm smithfarm left a comment

Choose a reason for hiding this comment

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

NOTE TO SELF:

master PR is still open

once it merges, double check the SHA1

@rzarzynski
Copy link
Contributor Author

Hello @smithfarm!
The commit is merged with master now. I made first check for commit ID and it looks there is no change since sending the backport:

$ git remote -v | grep origin
origin	https://github.com/ceph/ceph.git (fetch)
origin	https://github.com/ceph/ceph.git (push)
$ git branch --contains da362675da51de8b20a12e4cec479a77c7f5442b -a
  ...
  remotes/origin/master

@smithfarm smithfarm dismissed their stale review April 14, 2020 09:21

thanks, Radek

@smithfarm
Copy link
Contributor

jenkins test make check

@yuriw yuriw added wip-yuri2-testing nautilus-batch-1 nautilus point releases labels Apr 14, 2020
@yuriw
Copy link
Contributor

yuriw commented Apr 14, 2020

wip-yuri2-testing-2020-04-14-1817-nautilus

@yuriw yuriw merged commit 25bdfae into ceph:nautilus Apr 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@neha-ojha neha-ojha neha-ojha approved these changes

@smithfarm smithfarm smithfarm left review comments

Assignees
No one assigned
Labels
bug-fix mgr nautilus-batch-1 nautilus point releases needs-qa wip-yuri2-testing
Projects
None yet
Milestone
nautilus
Development

Successfully merging this pull request may close these issues.
4 participants
@rzarzynski @smithfarm @yuriw @neha-ojha