SUSE / DeepSea

A collection of Salt files for deploying, managing and automating Ceph.
GNU General Public License v3.0
161 stars 75 forks source link

dg: workaround ceph-volume infused bugs #1708

Closed jschmid1 closed 5 years ago

jschmid1 commented 5 years ago

Add workarounds for:

1) Do not accept disks < 5GB 2) Do not accept /dev/mapper disks (this needs more thinking as you actually may want to deploy /dev/mapper devices)

Tests for various cases added.

The workarounds need to be adapted with the addition of #1690. With this PR we source the disks list from the ceph_volume library and not from the CLI. Also, we moved the logic in cephdisks.py.

suse_internal: bsc#1141368, bsc#1141367, bsc#1141503


Checklist:

jschmid1 commented 5 years ago

@susebot run teuthology

susebot commented 5 years ago

Commit e4747d358194674f05e1685a3e20fd8910ba55a7 is NOT OK for suite deepsea:tier2. Check tests results in the Jenkins job: http://ci.ses.suse.de:8080/job/pr-deepsea/250/

jschmid1 commented 5 years ago

I'm still seeing unrelated test failures @kshtsk @smithfarm. Are the artifacts up to date?