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

vstart.sh: fix mgr vs restful command startup race #16564

Merged
merged 1 commit into from Jul 25, 2017

Conversation

Projects
None yet
3 participants
@liewegas
Member

liewegas commented Jul 25, 2017

If the mgr hasn't activated yet we won't have registered the python
commands. Use 'ceph tell mgr ...' to ensure we block. (This is only a
problem right after mkfs when the first mgr hasn't started yet.)

Signed-off-by: Sage Weil sage@redhat.com

vstart.sh: fix mgr vs restful command startup race
If the mgr hasn't activated yet we won't have registered the python
commands.  Use 'ceph tell mgr ...' to ensure we block.  (This is only a
problem right after mkfs when the first mgr hasn't started yet.)

Signed-off-by: Sage Weil <sage@redhat.com>
@jecluis

haven't got a chance to test this yet, but assuming it works fine, I approve.

@theanalyst

This comment has been minimized.

Member

theanalyst commented Jul 25, 2017

worked for me

@liewegas liewegas merged commit 65e0f2c into ceph:master Jul 25, 2017

2 of 4 checks passed

make check running make check
Details
make check (arm64) running make check
Details
Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details

@liewegas liewegas deleted the liewegas:wip-vstart-mgr branch Jul 25, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment