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

tests: osd-crush.sh must retry crush dump #12016

Merged
merged 1 commit into from Nov 16, 2016
Merged

tests: osd-crush.sh must retry crush dump #12016

merged 1 commit into from Nov 16, 2016

Conversation

ghost
Copy link

@ghost ghost commented Nov 16, 2016

After ceph osd setcrushmap, the script must retry because ceph osd crush
dump races against the update of the crushmap.

Fixes: http://tracker.ceph.com/issues/17919

Signed-off-by: Loic Dachary loic@dachary.org

@ghost ghost added bug-fix tests labels Nov 16, 2016
@ghost ghost assigned tchaikov Nov 16, 2016
After ceph osd setcrushmap, the script must retry because ceph osd crush
dump races against the update of the crushmap.

Fixes: http://tracker.ceph.com/issues/17919

Signed-off-by: Loic Dachary <loic@dachary.org>
@ghost
Copy link
Author

ghost commented Nov 16, 2016

@tchaikov I've seen this failure twice in the past 10 make check failures: it looks quite frequent although not as frequent as the eio/scrub failures.

Copy link
Contributor

@tchaikov tchaikov left a comment

Choose a reason for hiding this comment

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

lgtm

@tchaikov tchaikov merged commit c1d74d2 into ceph:master Nov 16, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants