Skip to content

Commit

Permalink
ceph: require of e2fsprogs package
Browse files Browse the repository at this point in the history
since we've switched to centos as default images it has revealed a bug
during ceph-disk prepare operation when deploying with dmcrypt
scenarios.

typical error:
```
......
get_dm_uuid: get_dm_uuid /dev/sda uuid path is /sys/dev/block/8:0/dm/uuid
get_dm_uuid: get_dm_uuid /dev/sda uuid path is /sys/dev/block/8:0/dm/uuid
get_dm_uuid: get_dm_uuid /dev/sda5 uuid path is /sys/dev/block/8:5/dm/uuid
populate: Creating lockbox fs on %s: mkfs -t ext4 /dev/sda5
command_check_call: Running command: /usr/sbin/mkfs -t ext4 /dev/sda5
mkfs.ext4: No such file or directory
Traceback (most recent call last):
  File "/usr/sbin/ceph-disk", line 9, in <module>
    load_entry_point('ceph-disk==1.0.0', 'console_scripts', 'ceph-disk')()
......
```

mkfs.ext4 is not found because the package `e2fsprogs` isn't installed.
It seems that ceph/ceph doesn't have a require for `e2fsprogs` so this
commit is a workaround until we can get this fixed in ceph/ceph.
(see: https://github.com/ceph/ceph/pull/21345/files)

Since this impacts several distributions and the package name is the
same for all, I've put this in `./src/daemon-base/__CEPH_BASE_PACKAGES__`.
We could remove this when it will have been fixed in ceph/ceph.

Signed-off-by: Guillaume Abrioux <[email protected]>
  • Loading branch information
guits authored and ErwanAliasr1 committed Apr 11, 2018
1 parent f3dc37c commit 2302241
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions src/daemon-base/__CEPH_BASE_PACKAGES__
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
\
ca-certificates \
e2fsprogs \
ceph-common__ENV_[CEPH_POINT_RELEASE]__ \
ceph-mon__ENV_[CEPH_POINT_RELEASE]__ \
ceph-osd__ENV_[CEPH_POINT_RELEASE]__ \
Expand Down

0 comments on commit 2302241

Please sign in to comment.