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: osd: osd/PG.cc: 3837: FAILED assert(0 == "Running incompatible OSD") #7206

Merged
merged 2 commits into from Jan 31, 2016

Conversation

dzafman
Copy link
Contributor

@dzafman dzafman commented Jan 12, 2016

Feature present since at least the Dumpling release.
A later commit will add it to the osd_required mask

Fixes: ceph#11661

Signed-off-by: David Zafman <dzafman@redhat.com>
(cherry picked from commit 7a10a7e)
…uired mask

Signed-off-by: David Zafman <dzafman@redhat.com>
(cherry picked from commit 0b2bab4)
@yuriw
Copy link
Contributor

yuriw commented Jan 12, 2016

@dzafman You don't want me review this PR, do you?

@yuriw yuriw removed their assignment Jan 12, 2016
@yuriw
Copy link
Contributor

yuriw commented Jan 12, 2016

@dachary FYI

@dzafman
Copy link
Contributor Author

dzafman commented Jan 12, 2016

@dachary I wasn't sure if these need to be expedited, so I assigned to Yuri. It consists of 2 straight cherry-picks. I didn't backport a bunch of mostly code clean-up that was went in together with these 2 changes, so that is the only risk.

@ghost ghost changed the title Backports to fix 14292 osd/PG.cc: 3837: FAILED assert(0 == "Running incompatible OSD") Jan 13, 2016
ghost pushed a commit that referenced this pull request Jan 15, 2016
…g incompatible OSD")

Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost ghost self-assigned this Jan 29, 2016
@ghost ghost removed the needs-integration-qa label Jan 29, 2016
@ghost
Copy link

ghost commented Jan 29, 2016

@dzafman it will be in the next batch of tests.

ghost pushed a commit that referenced this pull request Jan 29, 2016
…g incompatible OSD")

Reviewed-by: Loic Dachary <ldachary@redhat.com>
liewegas added a commit that referenced this pull request Jan 31, 2016
osd/PG.cc: 3837: FAILED assert(0 == "Running incompatible OSD")

http://pulpito.ceph.com/sage-2016-01-30_09:58:32-rados-wip-sage-testing-hammer---basic-mira/

Reviewed-by: David Zafman <dzafman@redhat.com>
@liewegas liewegas merged commit 211a093 into ceph:hammer Jan 31, 2016
@ghost ghost changed the title osd/PG.cc: 3837: FAILED assert(0 == "Running incompatible OSD") hammer: osd: osd/PG.cc: 3837: FAILED assert(0 == "Running incompatible OSD") Feb 18, 2016
@dzafman dzafman deleted the wip-14292 branch May 2, 2018 22:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants