vdsm-patches October 2016

vdsm-patches@lists.fedorahosted.org

33 participants
468 discussions

Change in vdsm[master]: storage: RPC support for IPv6 iSCSI targets
by dholler＠redhat.com 31 Oct '16

31 Oct '16

Dominik Holler has uploaded a new change for review. Change subject: storage: RPC support for IPv6 iSCSI targets ...................................................................... storage: RPC support for IPv6 iSCSI targets HSM.discoverSendTargets uses iscsi.getTargetString, which already supports IPv6 iSCSI targets, to encapsulate the iSCSI targets in a string. Change-Id: Ia31b045237dc2defc00181ef8a08cdaf808aa635 Bug-Url: https://bugzilla.redhat.com/1308306 Signed-off-by: Dominik Holler <dholler(a)redhat.com> --- M vdsm/storage/hsm.py 1 file changed, 1 insertion(+), 2 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/36/65736/1 diff --git a/vdsm/storage/hsm.py b/vdsm/storage/hsm.py index e1d6d1e..50f7013 100644 --- a/vdsm/storage/hsm.py +++ b/vdsm/storage/hsm.py @@ -2937,8 +2937,7 @@ fullTargets = [] partialTargets = [] for target in targets: - fullTargets.append("%s:%d,%d %s" % - (target.portal.hostname, target.portal.port, + fullTargets.append(iscsi.getTargetString( target.portal, target.tpgt, target.iqn)) partialTargets.append(target.iqn) -- To view, visit https://gerrit.ovirt.org/65736 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ia31b045237dc2defc00181ef8a08cdaf808aa635 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Dominik Holler <dholler(a)redhat.com>

4 15

Change in vdsm[master]: net: switch type switch
by phoracek＠redhat.com 31 Oct '16

31 Oct '16

Petr Horáček has posted comments on this change. Change subject: net: switch type switch ...................................................................... Patch Set 13: Verified+1 Passed all network/*_test.py OK -- To view, visit https://gerrit.ovirt.org/62330 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia090277ee3c4995f0a313a032ae651ba55dc8989 Gerrit-PatchSet: 13 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Petr Horáček <phoracek(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Edward Haas <edwardh(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Petr Horáček <phoracek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org> Gerrit-HasComments: No

1 0

Change in vdsm[master]: net: switch type switch
by automation＠ovirt.org 31 Oct '16

31 Oct '16

gerrit-hooks has posted comments on this change. Change subject: net: switch type switch ...................................................................... Patch Set 13: * Update Tracker::#1362399::OK, status: POST * Check Bug-Url::OK * Check Public Bug::#1362399::OK, public bug * Check Product::IGNORE, not relevant for branch: master * Check TM::IGNORE, not relevant for branch: master * Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.6', 'ovirt-4.0']) -- To view, visit https://gerrit.ovirt.org/62330 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia090277ee3c4995f0a313a032ae651ba55dc8989 Gerrit-PatchSet: 13 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Petr Horáček <phoracek(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Edward Haas <edwardh(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org> Gerrit-HasComments: No

1 0

Change in vdsm[master]: net: switch type switch
by automation＠ovirt.org 31 Oct '16

31 Oct '16

gerrit-hooks has posted comments on this change. Change subject: net: switch type switch ...................................................................... Patch Set 12: * Update Tracker::#1362399::OK, status: NEW * Check Bug-Url::OK * Check Public Bug::#1362399::OK, public bug * Check Product::IGNORE, not relevant for branch: master * Check TM::IGNORE, not relevant for branch: master * Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.6', 'ovirt-4.0']) -- To view, visit https://gerrit.ovirt.org/62330 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia090277ee3c4995f0a313a032ae651ba55dc8989 Gerrit-PatchSet: 12 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Petr Horáček <phoracek(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Edward Haas <edwardh(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org> Gerrit-HasComments: No

1 0

Change in vdsm[master]: stomp: simplified client
by automation＠ovirt.org 31 Oct '16

31 Oct '16

gerrit-hooks has posted comments on this change. Change subject: stomp: simplified client ...................................................................... Patch Set 14: * update_tracker: OK * Set MODIFIED::IGNORE, no Bug-Url found. -- To view, visit https://gerrit.ovirt.org/59151 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2b9b33474e60ab349885a7de18eeacc8f648011f Gerrit-PatchSet: 14 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com> Gerrit-Reviewer: Irit Goihman <igoihman(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org> Gerrit-HasComments: No

1 0

Change in vdsm[master]: stomp: simplified client
by Dan Kenigsberg 31 Oct '16

31 Oct '16

Dan Kenigsberg has submitted this change and it was merged. Change subject: stomp: simplified client ...................................................................... stomp: simplified client Change-Id: I2b9b33474e60ab349885a7de18eeacc8f648011f Signed-off-by: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Signed-off-by: Irit Goihman <igoihman(a)redhat.com> Reviewed-on: https://gerrit.ovirt.org/59151 Continuous-Integration: Jenkins CI Reviewed-by: Yaniv Bronhaim <ybronhei(a)redhat.com> Reviewed-by: Dan Kenigsberg <danken(a)redhat.com> --- M lib/yajsonrpc/stompreactor.py 1 file changed, 24 insertions(+), 17 deletions(-) Approvals: Yaniv Bronhaim: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Irit Goihman: Verified Dan Kenigsberg: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/59151 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2b9b33474e60ab349885a7de18eeacc8f648011f Gerrit-PatchSet: 14 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com> Gerrit-Reviewer: Irit Goihman <igoihman(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in vdsm[master]: stomp: simplified client
by Dan Kenigsberg 31 Oct '16

31 Oct '16

Dan Kenigsberg has posted comments on this change. Change subject: stomp: simplified client ...................................................................... Patch Set 13: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/59151 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2b9b33474e60ab349885a7de18eeacc8f648011f Gerrit-PatchSet: 13 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com> Gerrit-Reviewer: Irit Goihman <igoihman(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org> Gerrit-HasComments: No

1 0

Change in vdsm[master]: utils: build cert paths in single place
by automation＠ovirt.org 31 Oct '16

31 Oct '16

gerrit-hooks has posted comments on this change. Change subject: utils: build cert paths in single place ...................................................................... Patch Set 15: * update_tracker: OK * Set MODIFIED::IGNORE, no Bug-Url found. -- To view, visit https://gerrit.ovirt.org/52354 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I58dd3a5f7c1503fc38b6c6a204c036c06d09941b Gerrit-PatchSet: 15 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com> Gerrit-Reviewer: Irit Goihman <igoihman(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org> Gerrit-HasComments: No

1 0

Change in vdsm[master]: utils: build cert paths in single place
by Dan Kenigsberg 31 Oct '16

31 Oct '16

Dan Kenigsberg has submitted this change and it was merged. Change subject: utils: build cert paths in single place ...................................................................... utils: build cert paths in single place We make sure that all the references to certificates location are in single place. As part of this fix we remove trust store path from configuration which is not needed anymore. Change-Id: I58dd3a5f7c1503fc38b6c6a204c036c06d09941b Signed-off-by: pkliczewski <piotr.kliczewski(a)gmail.com> Reviewed-on: https://gerrit.ovirt.org/52354 Continuous-Integration: Jenkins CI Reviewed-by: Yaniv Bronhaim <ybronhei(a)redhat.com> Tested-by: Irit Goihman <igoihman(a)redhat.com> Reviewed-by: Dan Kenigsberg <danken(a)redhat.com> --- M build-aux/Makefile.subs M lib/vdsm/config.py.in M lib/vdsm/constants.py.in M lib/vdsm/m2cutils.py M lib/vdsm/sslutils.py M lib/vdsm/tool/configurators/certificates.py M lib/vdsm/tool/configurators/libvirt.py M lib/vdsm/vdscli.py M vdsm/sos/vdsm.py.in 9 files changed, 26 insertions(+), 42 deletions(-) Approvals: Yaniv Bronhaim: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Irit Goihman: Verified Dan Kenigsberg: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/52354 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I58dd3a5f7c1503fc38b6c6a204c036c06d09941b Gerrit-PatchSet: 15 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com> Gerrit-Reviewer: Irit Goihman <igoihman(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in vdsm[master]: utils: build cert paths in single place
by Dan Kenigsberg 31 Oct '16

31 Oct '16

Dan Kenigsberg has posted comments on this change. Change subject: utils: build cert paths in single place ...................................................................... Patch Set 14: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/52354 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I58dd3a5f7c1503fc38b6c6a204c036c06d09941b Gerrit-PatchSet: 14 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com> Gerrit-Reviewer: Irit Goihman <igoihman(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org> Gerrit-HasComments: No

1 0

← Newer
1
...
5
6
7
8
9
10
11
...
47
Older →

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

vdsm-patches October 2016