Comment 2 for bug 1878227

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to charm-cinder (master)

Reviewed: https://review.opendev.org/727821
Committed: https://git.openstack.org/cgit/openstack/charm-cinder/commit/?id=fc8e95f8c712e8ec5f1d768a2ac3d5c7587421eb
Submitter: Zuul
Branch: master

commit fc8e95f8c712e8ec5f1d768a2ac3d5c7587421eb
Author: Alex Kavanagh <email address hidden>
Date: Wed May 13 17:07:34 2020 +0100

    Wrap lvm commands in try: except:

    Due to the additional of the cinder storage, it's possible that the
    log_lvm_info() commands gets called really early before lvm has been
    configured. This wraps the call to pvscan and logs if it doesn't
    complete -- the command is just to log the state of lvm at that point.

    Also, defensively wrap the reduce_lvm_volume_group_missing() calls for
    the same reason.

    Closes-Bug: #1878227
    Change-Id: I217f62d3f620992d1b18475ca5c3031da9428828