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.
Chang
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 https:/
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 https://gerrit.ovirt.o
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, vis
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 https://gerrit.ovirt.o
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 'Bu
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 'Bu
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 'Bug
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 'Bug
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 'Bug
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 'Bug
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 'Bug
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 'Bug
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, visit
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 'Bug
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 'Bug
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 op
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, poolNam
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, poolName
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 argument
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 https://gerrit.ovirt.or
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 'Bug
26 matches
Mail list logo