Dan Kenigsberg has submitted this change and it was merged.
Change subject: jsonrpcvdscli: add storage verbs
..
jsonrpcvdscli: add storage verbs
Add verbs used by ovirt-hosted-engine-setup for
handling the storage setup.
gerrit-hooks has posted comments on this change.
Change subject: jsonrpcvdscli: add storage verbs
..
Patch Set 12:
* Update tracker: IGNORE, no Bug-Url found
* Set MODIFIED::IGNORE, no Bug-Url found.
--
To view, visit
Piotr Kliczewski has posted comments on this change.
Change subject: jsonrpcvdscli: add storage verbs
..
Patch Set 11: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/55252
To unsubscribe, visit
Simone Tiraboschi has posted comments on this change.
Change subject: jsonrpcvdscli: add storage verbs
..
Patch Set 11: Verified+1
Verified on iSCSI and NFS
--
To view, visit https://gerrit.ovirt.org/55252
To unsubscribe,
Sandro Bonazzola has posted comments on this change.
Change subject: jsonrpcvdscli: add storage verbs
..
Patch Set 11: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/55252
To unsubscribe, visit
gerrit-hooks has posted comments on this change.
Change subject: jsonrpcvdscli: add storage verbs
..
Patch Set 11:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches
gerrit-hooks has posted comments on this change.
Change subject: jsonrpcvdscli: add storage verbs
..
Patch Set 10:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches
gerrit-hooks has posted comments on this change.
Change subject: jsonrpcvdscli: add storage verbs
..
Patch Set 9:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches
gerrit-hooks has posted comments on this change.
Change subject: jsonrpcvdscli: add storage verbs
..
Patch Set 8:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches
gerrit-hooks has posted comments on this change.
Change subject: jsonrpcvdscli: add storage verbs
..
Patch Set 7:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches
gerrit-hooks has posted comments on this change.
Change subject: jsonrpcvdscli: add storage verbs
..
Patch Set 6:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches
gerrit-hooks has posted comments on this change.
Change subject: jsonrpcvdscli: add storage verbs
..
Patch Set 5:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches
gerrit-hooks has posted comments on this change.
Change subject: jsonrpcvdscli: add storage verbs
..
Patch Set 4:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches
Piotr Kliczewski has posted comments on this change.
Change subject: jsonrpcvdscli: add storage verbs
..
Patch Set 3:
The change looks good to me. Please let me know when this WIP is done and I
will ack it.
--
To view,
gerrit-hooks has posted comments on this change.
Change subject: jsonrpcvdscli: add storage verbs
..
Patch Set 3:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches
gerrit-hooks has posted comments on this change.
Change subject: jsonrpcvdscli: add storage verbs
..
Patch Set 2:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches
Piotr Kliczewski has posted comments on this change.
Change subject: jsonrpcvdscli: add storage verbs
..
Patch Set 1:
Looking at API.py and the schema we do not have it optional but on hsm level we
you None as default value.
Sandro Bonazzola has posted comments on this change.
Change subject: jsonrpcvdscli: add storage verbs
..
Patch Set 1:
> not sure why they are defined as required on API.py level.
@Piotr are you going to fix API to make them
Piotr Kliczewski has posted comments on this change.
Change subject: jsonrpcvdscli: add storage verbs
..
Patch Set 1:
It looks like hsm makes them optional:
@public
def createStoragePool(self, poolType, spUUID,
Piotr Kliczewski has posted comments on this change.
Change subject: jsonrpcvdscli: add storage verbs
..
Patch Set 1:
It looks like xmlrpc was providing some default values:
def poolCreate(self, poolType, spUUID,
Piotr Kliczewski has posted comments on this change.
Change subject: jsonrpcvdscli: add storage verbs
..
Patch Set 1:
Sandro checking API.py I can see following params needed by this call :
def __init__(self, UUID):
and
Sandro Bonazzola has posted comments on this change.
Change subject: jsonrpcvdscli: add storage verbs
..
Patch Set 1: Verified-1
[ ERROR ] Failed to execute stage 'Misc configuration': Attempt to call
function: > with
Piotr Kliczewski has posted comments on this change.
Change subject: jsonrpcvdscli: add storage verbs
..
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/55252
To unsubscribe, visit
Sandro Bonazzola has posted comments on this change.
Change subject: jsonrpcvdscli: add storage verbs
..
Patch Set 1:
Traceback (most recent call last):
File "/usr/lib/python2.7/site-packages/otopi/context.py", line 131, in
Sandro Bonazzola has uploaded a new change for review.
Change subject: jsonrpcvdscli: add storage verbs
..
jsonrpcvdscli: add storage verbs
Add verbs used by ovirt-hosted-engine-setup for
handling the storage setup.
Change-Id:
gerrit-hooks has posted comments on this change.
Change subject: jsonrpcvdscli: add storage verbs
..
Patch Set 1:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches
26 matches
Mail list logo
