[lvm driver] should use lvextend capabilities

[Tom Barron]

Currently, the lvm driver uses commands of the form 'lvextend -L <size> -n <device-name>' to extend the logical volume size, then calls 'resize2fs' to make a corresponding change to the filesystem in the logical volume.

We should not use the '-n' argument, which tells lvextend to skip file system checks. The checks are very fast if they are not needed and it is unsafe to skip them if they are.

Additionally, lvextend can take an option '-r' wherein it takes responsibility itself for resizing the filesystem built on the logical volume. We should use it since it is more atomic than and involves less external size judgment for the file system resize than processing the file system resize separately.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/ussuri
Review: https://review.opendev.org/741488

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/740492
Committed: https://git.openstack.org/cgit/openstack/manila/commit/?id=c94e1deff1bde93d4167ceeb9dafc494d12cd60c
Submitter: Zuul
Branch: master

commit c94e1deff1bde93d4167ceeb9dafc494d12cd60c
Author: Tom Barron <email address hidden>
Date: Fri Jul 10 11:15:22 2020 -0400

    Update LVM volume extend

    - do *not* skip fsck when extending
    - have extend resize the filesystem itself
      rather than making a separate execution
      of ``resize2fs``

    Closes-bug: #1887694
    Change-Id: I67ac549e790322c56d02db4424ec01a5449e6633

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/741488
Committed: https://git.openstack.org/cgit/openstack/manila/commit/?id=2a3c18edcd9c4f7def293f18a6cac286ec00bc49
Submitter: Zuul
Branch: stable/ussuri

commit 2a3c18edcd9c4f7def293f18a6cac286ec00bc49
Author: Tom Barron <email address hidden>
Date: Fri Jul 10 11:15:22 2020 -0400

    Update LVM volume extend

    - do *not* skip fsck when extending
    - have extend resize the filesystem itself
      rather than making a separate execution
      of ``resize2fs``

    Closes-bug: #1887694
    Change-Id: I67ac549e790322c56d02db4424ec01a5449e6633

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/train
Review: https://review.opendev.org/742289

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/train
Review: https://review.opendev.org/742290

[OpenStack Infra (hudson-openstack)]

Change abandoned by Tom Barron (<email address hidden>) on branch: stable/train
Review: https://review.opendev.org/742289
Reason: in favor of 742290

[OpenStack Infra (hudson-openstack)]

Change abandoned by Goutham Pacha Ravi (<email address hidden>) on branch: stable/train
Review: https://review.opendev.org/742290

[OpenStack Infra (hudson-openstack)]

Change abandoned by Tom Barron (<email address hidden>) on branch: stable/train
Review: https://review.opendev.org/742289

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/train
Review: https://review.opendev.org/743640

[OpenStack Infra (hudson-openstack)]

Change abandoned by Goutham Pacha Ravi (<email address hidden>) on branch: stable/train
Review: https://review.opendev.org/742289
Reason: We'll abandon this in favor of: https://review.opendev.org/#/c/743640/

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/743640
Committed: https://git.openstack.org/cgit/openstack/manila/commit/?id=2063c93c078c990f4865994dc0221662cbc4c5d5
Submitter: Zuul
Branch: stable/train

commit 2063c93c078c990f4865994dc0221662cbc4c5d5
Author: Tom Barron <email address hidden>
Date: Fri Jul 10 11:15:22 2020 -0400

    Update LVM volume extend

    - do *not* skip fsck when extending
    - have extend resize the filesystem itself
      rather than making a separate execution
      of ``resize2fs``

    Due to an issue with Gerrit, we're unable
    to preserve the Change-ID for this change.
    The change-id in stable/ussuri and in the
    main branch is I67ac549e790322c56d02db4424ec01a5449e6633

    Closes-bug: #1887694
    Change-Id: Ieebd69ec9671d48f2d9de0ae3e110518cba4f6f9
    (cherry picked from commit c94e1deff1bde93d4167ceeb9dafc494d12cd60c)
    (cherry picked from commit 2a3c18edcd9c4f7def293f18a6cac286ec00bc49)

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/stein
Review: https://review.opendev.org/743711

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/743711
Committed: https://git.openstack.org/cgit/openstack/manila/commit/?id=e378f44598a9a2560f35948e108bc03f2bbb0adb
Submitter: Zuul
Branch: stable/stein

commit e378f44598a9a2560f35948e108bc03f2bbb0adb
Author: Tom Barron <email address hidden>
Date: Fri Jul 10 11:15:22 2020 -0400

    Update LVM volume extend

    - do *not* skip fsck when extending
    - have extend resize the filesystem itself
      rather than making a separate execution
      of ``resize2fs``

    Due to an issue with Gerrit, we're unable
    to preserve the Change-ID for this change.
    The change-id in stable/ussuri and in the
    main branch is I67ac549e790322c56d02db4424ec01a5449e6633

    Closes-bug: #1887694
    Change-Id: Ieebd69ec9671d48f2d9de0ae3e110518cba4f6f9
    (cherry picked from commit c94e1deff1bde93d4167ceeb9dafc494d12cd60c)
    (cherry picked from commit 2a3c18edcd9c4f7def293f18a6cac286ec00bc49)
    (cherry picked from commit 2063c93c078c990f4865994dc0221662cbc4c5d5)

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/rocky
Review: https://review.opendev.org/743791

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/743791
Committed: https://git.openstack.org/cgit/openstack/manila/commit/?id=d8c433044b01ada78530838ac4467af6d39e27b7
Submitter: Zuul
Branch: stable/rocky

commit d8c433044b01ada78530838ac4467af6d39e27b7
Author: Tom Barron <email address hidden>
Date: Fri Jul 10 11:15:22 2020 -0400

    Update LVM volume extend

    - do *not* skip fsck when extending
    - have extend resize the filesystem itself
      rather than making a separate execution
      of ``resize2fs``

    Due to an issue with Gerrit, we're unable
    to preserve the Change-ID for this change.
    The change-id in stable/ussuri and in the
    main branch is I67ac549e790322c56d02db4424ec01a5449e6633

    Closes-bug: #1887694
    Change-Id: Ieebd69ec9671d48f2d9de0ae3e110518cba4f6f9
    (cherry picked from commit c94e1deff1bde93d4167ceeb9dafc494d12cd60c)
    (cherry picked from commit 2a3c18edcd9c4f7def293f18a6cac286ec00bc49)
    (cherry picked from commit 2063c93c078c990f4865994dc0221662cbc4c5d5)
    (cherry picked from commit e378f44598a9a2560f35948e108bc03f2bbb0adb)

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/queens
Review: https://review.opendev.org/743974

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/743974
Committed: https://git.openstack.org/cgit/openstack/manila/commit/?id=eee6c4218fe376f78c5fa5edd28e605e114b585b
Submitter: Zuul
Branch: stable/queens

commit eee6c4218fe376f78c5fa5edd28e605e114b585b
Author: Tom Barron <email address hidden>
Date: Fri Jul 10 11:15:22 2020 -0400

    Update LVM volume extend

    - do *not* skip fsck when extending
    - have extend resize the filesystem itself
      rather than making a separate execution
      of ``resize2fs``

    Due to an issue with Gerrit, we're unable
    to preserve the Change-ID for this change.
    The change-id in stable/ussuri and in the
    main branch is I67ac549e790322c56d02db4424ec01a5449e6633

    Closes-bug: #1887694
    Change-Id: Ieebd69ec9671d48f2d9de0ae3e110518cba4f6f9
    (cherry picked from commit c94e1deff1bde93d4167ceeb9dafc494d12cd60c)
    (cherry picked from commit 2a3c18edcd9c4f7def293f18a6cac286ec00bc49)
    (cherry picked from commit 2063c93c078c990f4865994dc0221662cbc4c5d5)
    (cherry picked from commit e378f44598a9a2560f35948e108bc03f2bbb0adb)
    (cherry picked from commit d8c433044b01ada78530838ac4467af6d39e27b7)

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/manila queens-eol release.

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/manila rocky-eol release.