Change in vdsm[master]: storage: Introduction to transfer.py

nsoffer at redhat.com nsoffer at redhat.com
Sun Dec 13 18:50:22 UTC 2015


Nir Soffer has posted comments on this change.

Change subject: storage: Introduction to transfer.py
......................................................................


Patch Set 2:

(2 comments)

https://gerrit.ovirt.org/#/c/50014/2/lib/api/vdsmapi-schema.json
File lib/api/vdsmapi-schema.json:

Line 528: # @Host.startImageTransferSession:
Line 529: #
Line 530: # Start an image upload data transfer session.
Line 531: #
Line 532: # @ticketUUID:  Ticket UUID
> This should be decided in higher level than this API, as the engine, proxy,
Agree, hackday stuff.
Line 533: #
Line 534: # @ops:         Comma-delimited operation list (e.g. "get, put")
Line 535: #
Line 536: # @expiration:  Expiration epoch


Line 540: # @sdUUID:      Storage Domain UUID
Line 541: #
Line 542: # @imgUUID:     Image UUID
Line 543: #
Line 544: # @volUUID:     Volume UUID
> Not to vdsm-image, but to an ISO domain: the flow is that a user will use w
To upload to iso domain, vdsm will have to prepare a file where imaged will write the data.

We want to kill iso domain (and export domain), so this use case may not exits in 4.0.
Line 545: #
Line 546: # Since: 4.17.4
Line 547: ##
Line 548: {'command': {'class': 'Host', 'name': 'startImageTransferSession'},


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I6b9ded4bde73b1ab504cae50d2cea726d4f77e51
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Amit Aviram <aaviram at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Amit Aviram <aaviram at redhat.com>
Gerrit-Reviewer: Greg Padgett <gpadgett at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list