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

mimic: ceph-volume: ensure device lists are disjoint #30334

Merged
merged 2 commits into from Sep 19, 2019

Conversation

jan--f
Copy link
Contributor

@jan--f jan--f commented Sep 11, 2019

Jan Fajerski added 2 commits September 11, 2019 14:36
Fixes: https://tracker.ceph.com/issues/41018

Signed-off-by: Jan Fajerski <jfajerski@suse.com>
(cherry picked from commit 8fcdb4d)
This was introduced by ceph#27754. The explicit device lists were cast to
sets but other parts of the code where not updated accordingly. To avoid
touching all code places, only cast to sets for disjoint test and keep
lists otherwise.

Fixes: https://tracker.ceph.com/issues/41292

Signed-off-by: Jan Fajerski <jfajerski@suse.com>
(cherry picked from commit 0534cf1)
@jan--f jan--f force-pushed the wip-disjoint-device-lists-mimic branch from 258ebdb to 53cb311 Compare September 11, 2019 12:37
@jan--f
Copy link
Contributor Author

jan--f commented Sep 11, 2019

jenkins test ceph-volume all

@smithfarm
Copy link
Contributor

Added the backport tracker issue numbers to the PR description.

@jan--f
Copy link
Contributor Author

jan--f commented Sep 13, 2019

The failing tests are 4 failed clones and 1 failed cluster deployment, so nothing relating to this PR.

@alfredodeza
Copy link
Contributor

I've updated the PR to include the links to the master PRs so that it is easier to track down

@alfredodeza alfredodeza merged commit 4188f49 into ceph:mimic Sep 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants