Dan Kenigsberg has submitted this change and it was merged.

Change subject: resize-lun: Add resizePV verb
......................................................................


resize-lun: Add resizePV verb

This series of patches add support resizing of a storage domain after
a LUN was resized on the storage server.

Main flow - the user initiates LUNs resize

- Engine will invoke getDeviceList verb on all the hosts, with the
  specific LUNs guid. getDeviceList will rescan and resize the multipath
  map if needed.
- If all the hosts are seeing the same LUN size, resizePV verb will be
  invoked on the SPM, resizing the PV to the underlying device size.

Secondary flow - a host recovers from maintenance/down state

- When a host connects to the storage server, a rescan and resize of
  multipath devices will be performed. This operation will make sure
  that the host is synced with the storage server.

This patch adds the resizePV verb. This verb calls pvresize on a device
and returns the new PV size. This verb will extend the PV after a LUN
was resized on the storage server. This is a SPM operation.

Example outputs:

    $ vdsClient -s 0 resizePV \
            d24ebee6-ec30-4270-88a2-a8f38a5a804a \
            5966cee6-ee28-454c-96bc-e318bed1291a \
            360014057fac79f7f7dc4c15b29405f4e
        size = '36104568832'

    $ vdsClient -s 0 resizePV
      Error using command: Wrong number of arguments provided, expecting 3
      (3 required) got 0

      resizePV
          <sdUUID> <spUUID> <GUID>
          Resize PV in Storage Domain - After a block device has been
          resized in storage server, this method will cause the PV to use
          the entire size of the block device

Change-Id: I38f9e603a66adbe2655173b8043766f6c04a7913
Bug-Url:https://bugzilla.redhat.com/609689
Signed-off-by: Fred Rolland <[email protected]>
Reviewed-on: https://gerrit.ovirt.org/40467
Continuous-Integration: Jenkins CI
Reviewed-by: Nir Soffer <[email protected]>
Reviewed-by: Adam Litke <[email protected]>
Reviewed-by: Federico Simoncelli <[email protected]>
---
M client/vdsClient.py
M vdsm/API.py
M vdsm/rpc/Bridge.py
M vdsm/rpc/bindingxmlrpc.py
M vdsm/rpc/vdsmapi-schema.json
M vdsm/storage/blockSD.py
M vdsm/storage/hsm.py
M vdsm/storage/lvm.py
M vdsm/storage/sd.py
M vdsm/storage/sp.py
M vdsm/storage/storage_exception.py
11 files changed, 114 insertions(+), 0 deletions(-)

Approvals:
  Adam Litke: Looks good to me, approved
  Nir Soffer: Looks good to me, but someone else must approve
  Federico Simoncelli: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Freddy Rolland: Verified



-- 
To view, visit https://gerrit.ovirt.org/40467
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I38f9e603a66adbe2655173b8043766f6c04a7913
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Freddy Rolland <[email protected]>
Gerrit-Reviewer: Adam Litke <[email protected]>
Gerrit-Reviewer: Ala Hino <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Fred Rolland <[email protected]>
Gerrit-Reviewer: Freddy Rolland <[email protected]>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <[email protected]>
Gerrit-Reviewer: [email protected]
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to