Cleanup #15990
closed
ceph-disk: expected systemd unit failures are confusing
Added by Loïc Dachary almost 8 years ago.
Updated over 7 years ago.
Description
When activating a journal fails and later succeed (or the other way around), one of the two systemd unit is permanently marked as failed. This is confusing to the sysadmin who think there is a problem. This should be fixed.
Boris Ranto writes:
This could be fixed if we added 1 as an acceptable exit code for ceph-disk command, e.g. adding this line to the service file for ceph-disk:
SuccessExitStatus=1
We'd be just ignoring the error, though.
That would be fine with me (forcing the exit code to always be a success). The problem is that it will also hide real errors. Some would say that there are so many level of indirections when / if ceph-disk prepare fails that it won't really make a difference (ceph-disk prepare creates a partition -> fires a udev event -> it triggers a systemd unit -> it runs ceph-disk activate fails because the journal is not there yet -> ceph-disk prepare creates the journal partition -> fires a udev event -> it triggers the systemd unit -> it runs ceph-disk activate does the job).
It's a judgement call really and I'm not sure what's best.
Yeah, I was not too thrilled about it either. We might want to reserve a special exit code for saying 'not all devices ready' and ignore just that one?
btw: If journal and data are on the same drive, this will be ok after reboot as both devices show up at pretty much the same time. However, it is still reproducible when installing the cluster as you first create and activate the journal partition and only then, few seconds later, you create the data partition. It would help a bit to create both partitions at the same time (or tell kernel to activate them at the same time, not one by one).
- Priority changed from Normal to High
- Assignee set to Boris Ranto
- Status changed from New to Resolved
- Status changed from Resolved to Pending Backport
- Backport set to jewel
- Copied to Backport #17149: jewel: ceph-disk: expected systemd unit failures are confusing added
- Status changed from Pending Backport to Resolved
Also available in: Atom
PDF