Comment 7 for bug 1807760

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (stable/queens)

Reviewed: https://review.openstack.org/634045
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=33c504888d655fb661895e30568da1fa150f7b26
Submitter: Zuul
Branch: stable/queens

commit 33c504888d655fb661895e30568da1fa150f7b26
Author: Alan Bishop <email address hidden>
Date: Mon Dec 10 15:18:56 2018 -0500

    Fix permissions with NFS-backed snapshots and backups

    Fix snapshot issues that occur if cinder does not have regular (non-root)
    write permission on an NFS share. This is done by following the volume
    driver's model of creating files as root, followed by calling
    _set_rw_permissions().

    Fix backup issues that occur if cinder's group ID changes, which can
    happen when migrating from bare metal to a containerized service. If the
    share's group ownership and permission needs to be modified, then do it
    recursively so that previously made backups remain accessible.

    Closes-Bug: #1807760
    Change-Id: I6c20c4825af0a365b6a20fb633c810c2f2fe48b0
    (cherry picked from commit 1fb342cba89e63d15dde9db2136bdf34b549e559)
    (cherry picked from commit 95f30ea6edc093d9d29a2197c79a7e0b7427728b)
    Conflicts:
     cinder/tests/unit/volume/drivers/test_remotefs.py