Change in vdsm[master]: ceph: Initial support for network disk

danken at redhat.com danken at redhat.com
Thu Jun 11 14:46:58 UTC 2015


Dan Kenigsberg has submitted this change and it was merged.

Change subject: ceph: Initial support for network disk
......................................................................


ceph: Initial support for network disk

The previous code supported network disk by abusing Drive.volumeInfo
dict to send network drive parameters. Instead, we are using the new
diskType attribute, and adding network disk related attributes. This is
more consistent with other types of disks.

Example conf for network disk:

    {
        'diskType': 'network',
        'hosts': [
             {'name': '1.2.1.1', 'port': '6789', 'transport': 'tcp'},
             {'name': '1.2.2.1', 'port': '6789', 'transport': 'tcp'},
         ],
         'path': 'poolname/volumename',
         'protocol': 'rbd',
         ...
    }

Change-Id: I1c5792bb6206a227ee0c2df9490b771a69c0093c
Signed-off-by: Nir Soffer <nsoffer at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/38766
Reviewed-by: Francesco Romani <fromani at redhat.com>
Reviewed-by: Adam Litke <alitke at redhat.com>
Continuous-Integration: Jenkins CI
Reviewed-by: Federico Simoncelli <fsimonce at redhat.com>
---
M tests/vmStorageTests.py
M vdsm/rpc/vdsmapi-schema.json
M vdsm/virt/vmdevices/storage.py
3 files changed, 79 insertions(+), 20 deletions(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I1c5792bb6206a227ee0c2df9490b771a69c0093c
Gerrit-PatchSet: 17
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Sahina Bose <sabose at redhat.com>
Gerrit-Reviewer: Timothy Asir <tjeyasin at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list