Change in vdsm[master]: sos: change getConnectedStoragePoolsList result key
gerrit-hooks has posted comments on this change. Change subject: sos: change getConnectedStoragePoolsList result key .. Patch Set 3: * Update Tracker::IGNORE, no bug url/s found -- To view, visit https://gerrit.ovirt.org/63557 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I369d1392701d7b0eac2ce73613ba633a9a60d059 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Irit GoihmanGerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Irit Goihman Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer Gerrit-Reviewer: Piotr Kliczewski Gerrit-Reviewer: Yaniv Bronhaim Gerrit-Reviewer: gerrit-hooks Gerrit-HasComments: No ___ vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org
Change in vdsm[master]: sos: change getConnectedStoragePoolsList result key
Irit Goihman has abandoned this change. Change subject: sos: change getConnectedStoragePoolsList result key .. Abandoned -- To view, visit https://gerrit.ovirt.org/63557 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: abandon Gerrit-Change-Id: I369d1392701d7b0eac2ce73613ba633a9a60d059 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Irit GoihmanGerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Irit Goihman Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer Gerrit-Reviewer: Piotr Kliczewski Gerrit-Reviewer: Yaniv Bronhaim Gerrit-Reviewer: gerrit-hooks ___ vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org
Change in vdsm[master]: sos: change getConnectedStoragePoolsList result key
Irit Goihman has posted comments on this change. Change subject: sos: change getConnectedStoragePoolsList result key .. Patch Set 3: -Verified pending jsonrpc cli fix -- To view, visit https://gerrit.ovirt.org/63557 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I369d1392701d7b0eac2ce73613ba633a9a60d059 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Irit GoihmanGerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Irit Goihman Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer Gerrit-Reviewer: Piotr Kliczewski Gerrit-Reviewer: Yaniv Bronhaim Gerrit-Reviewer: gerrit-hooks Gerrit-HasComments: No ___ vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org
Change in vdsm[master]: sos: change getConnectedStoragePoolsList result key
Nir Soffer has posted comments on this change. Change subject: sos: change getConnectedStoragePoolsList result key .. Patch Set 3: Code-Review-1 We need working json rpc client that supports all verbs before we can migrated this to json rpc. -- To view, visit https://gerrit.ovirt.org/63557 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I369d1392701d7b0eac2ce73613ba633a9a60d059 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Irit GoihmanGerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Irit Goihman Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer Gerrit-Reviewer: Piotr Kliczewski Gerrit-Reviewer: Yaniv Bronhaim Gerrit-Reviewer: gerrit-hooks Gerrit-HasComments: No ___ vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org
Change in vdsm[master]: sos: change getConnectedStoragePoolsList result key
Piotr Kliczewski has posted comments on this change. Change subject: sos: change getConnectedStoragePoolsList result key .. Patch Set 3: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/63557 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I369d1392701d7b0eac2ce73613ba633a9a60d059 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Irit GoihmanGerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Irit Goihman Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski Gerrit-Reviewer: Yaniv Bronhaim Gerrit-Reviewer: gerrit-hooks Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org
Change in vdsm[master]: sos: change getConnectedStoragePoolsList result key
gerrit-hooks has posted comments on this change. Change subject: sos: change getConnectedStoragePoolsList result key .. Patch Set 3: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url. * Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.6', 'ovirt-4.0']) -- To view, visit https://gerrit.ovirt.org/63557 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I369d1392701d7b0eac2ce73613ba633a9a60d059 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Irit GoihmanGerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Irit Goihman Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski Gerrit-Reviewer: Yaniv Bronhaim Gerrit-Reviewer: gerrit-hooks Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org
Change in vdsm[master]: sos: change getConnectedStoragePoolsList result key
Yaniv Bronhaim has posted comments on this change. Change subject: sos: change getConnectedStoragePoolsList result key .. Patch Set 2: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/63557 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I369d1392701d7b0eac2ce73613ba633a9a60d059 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Irit GoihmanGerrit-Reviewer: Dan Kenigsberg Gerrit-Reviewer: Irit Goihman Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski Gerrit-Reviewer: Yaniv Bronhaim Gerrit-Reviewer: gerrit-hooks Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org
Change in vdsm[master]: sos: change getConnectedStoragePoolsList result key
Irit Goihman has posted comments on this change. Change subject: sos: change getConnectedStoragePoolsList result key .. Patch Set 2: Verified+1 -- To view, visit https://gerrit.ovirt.org/63557 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I369d1392701d7b0eac2ce73613ba633a9a60d059 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Irit GoihmanGerrit-Reviewer: Irit Goihman Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Yaniv Bronhaim Gerrit-Reviewer: gerrit-hooks Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org
Change in vdsm[master]: sos: change getConnectedStoragePoolsList result key
gerrit-hooks has posted comments on this change. Change subject: sos: change getConnectedStoragePoolsList result key .. Patch Set 2: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url. * Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.6', 'ovirt-4.0']) -- To view, visit https://gerrit.ovirt.org/63557 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I369d1392701d7b0eac2ce73613ba633a9a60d059 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Irit GoihmanGerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org
Change in vdsm[master]: sos: change getConnectedStoragePoolsList result key
gerrit-hooks has posted comments on this change. Change subject: sos: change getConnectedStoragePoolsList result key .. Patch Set 1: * Update tracker: IGNORE, no Bug-Url found * Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' and is a valid url. * Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.6', 'ovirt-4.0']) -- To view, visit https://gerrit.ovirt.org/63557 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I369d1392701d7b0eac2ce73613ba633a9a60d059 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Irit GoihmanGerrit-Reviewer: gerrit-hooks Gerrit-HasComments: No ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org
Change in vdsm[master]: sos: change getConnectedStoragePoolsList result key
Irit Goihman has uploaded a new change for review. Change subject: sos: change getConnectedStoragePoolsList result key .. sos: change getConnectedStoragePoolsList result key patch https://gerrit.ovirt.org/#/c/63483 fixes getConnectedStoragePoolsList result so it's needs to be updated in sos report which uses the old wrong 'items' key. Change-Id: I369d1392701d7b0eac2ce73613ba633a9a60d059 Signed-off-by: Irit Goihman--- M vdsm/sos/vdsm.py.in 1 file changed, 2 insertions(+), 2 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/57/63557/1 diff --git a/vdsm/sos/vdsm.py.in b/vdsm/sos/vdsm.py.in index dd3ae91..5939d58 100644 --- a/vdsm/sos/vdsm.py.in +++ b/vdsm/sos/vdsm.py.in @@ -143,8 +143,8 @@ self.addObjectAsFile( cli.getAllTasksStatuses(), "getAllTasksStatuses") -if "items" in cli.getConnectedStoragePoolsList(): -pools_list = cli.getConnectedStoragePoolsList()["items"] +if "poollist" in cli.getConnectedStoragePoolsList(): +pools_list = cli.getConnectedStoragePoolsList()["poollist"] for pool in pools_list: self.addObjectAsFile( cli.getSpmStatus(pool), "getSpmStatus " + pool) -- To view, visit https://gerrit.ovirt.org/63557 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I369d1392701d7b0eac2ce73613ba633a9a60d059 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Irit Goihman ___ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org