vdsm-patches August 2017

vdsm-patches@lists.fedorahosted.org

1 participants
209 discussions

Change in vdsm[master]: net: Update tags to reflect backport of getLldp command
by Code Review 24 Aug '17

24 Aug '17

From Dan Kenigsberg <danken(a)redhat.com>: Dan Kenigsberg has submitted this change and it was merged. ( https://gerrit.ovirt.org/80374 ) Change subject: net: Update tags to reflect backport of getLldp command ...................................................................... net: Update tags to reflect backport of getLldp command The API extension for Link Layer Discovery Protocol (LLDP) will be backported to oVirt 4.1.5. The 'added:' tags are updated to 4.1 to reflect this. Change-Id: I2dd21a7cd8aadeb33c333ac0733ae1903e96e21c Signed-off-by: Dominik Holler <dholler(a)redhat.com> --- M lib/vdsm/api/vdsm-api.yml 1 file changed, 5 insertions(+), 5 deletions(-) Approvals: Piotr Kliczewski: Looks good to me, approved Jenkins CI: Passed CI tests Irit Goihman: Looks good to me, but someone else must approve Dan Kenigsberg: Verified Edward Haas: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/80374 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2dd21a7cd8aadeb33c333ac0733ae1903e96e21c Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Dominik Holler <dholler(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Dominik Holler <dholler(a)redhat.com> Gerrit-Reviewer: Edward Haas <edwardh(a)redhat.com> Gerrit-Reviewer: Irit Goihman <igoihman(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in vdsm[master]: net: Update tags to reflect backport of getLldp command
by Code Review 24 Aug '17

24 Aug '17

From Dan Kenigsberg <danken(a)redhat.com>: Dan Kenigsberg has removed a vote on this change. Change subject: net: Update tags to reflect backport of getLldp command ...................................................................... Removed Verified-1 by Dominik Holler <dholler(a)redhat.com> -- To view, visit https://gerrit.ovirt.org/80374 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: deleteVote Gerrit-Change-Id: I2dd21a7cd8aadeb33c333ac0733ae1903e96e21c Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Dominik Holler <dholler(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Dominik Holler <dholler(a)redhat.com> Gerrit-Reviewer: Edward Haas <edwardh(a)redhat.com> Gerrit-Reviewer: Irit Goihman <igoihman(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in vdsm[master]: net: Update tags to reflect backport of getLldp command
by Code Review 24 Aug '17

24 Aug '17

From Dan Kenigsberg <danken(a)redhat.com>: Dan Kenigsberg has posted comments on this change. Change subject: net: Update tags to reflect backport of getLldp command ...................................................................... Patch Set 1: Verified+1 code merged to 4.1.6 -- To view, visit https://gerrit.ovirt.org/80374 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2dd21a7cd8aadeb33c333ac0733ae1903e96e21c Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Dominik Holler <dholler(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Dominik Holler <dholler(a)redhat.com> Gerrit-Reviewer: Edward Haas <edwardh(a)redhat.com> Gerrit-Reviewer: Irit Goihman <igoihman(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org> Gerrit-HasComments: No

1 0

Change in vdsm[master]: move vdsm.config to common
by Code Review 23 Aug '17

23 Aug '17

From Dan Kenigsberg <danken(a)redhat.com>: Dan Kenigsberg has uploaded a new change for review. Change subject: move vdsm.config to common ...................................................................... move vdsm.config to common vdsm.config is used by all subsystems. It should thus sit in common. This patch keeps a proxy in place, so that existing users of vdsm.config can be updated in separate patches. Change-Id: Ie84514aed56034c79044a94f7ce391b368cb99c4 Signed-off-by: Dan Kenigsberg <danken(a)redhat.com> --- M .gitignore M lib/vdsm/Makefile.am M lib/vdsm/common/Makefile.am R lib/vdsm/common/config.py.in 4 files changed, 5 insertions(+), 5 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/80/79980/1 diff --git a/.gitignore b/.gitignore index b96b310..7e8c362 100644 --- a/.gitignore +++ b/.gitignore @@ -28,7 +28,7 @@ init/vdsmd_init_common.sh lib/vdsm/api/vdsm-api.html lib/vdsm/common/constants.py -lib/vdsm/config.py +lib/vdsm/common/config.py lib/vdsm/dsaversion.py lib/sos/vdsm.py lib/vdsm/storage/lvm.env diff --git a/lib/vdsm/Makefile.am b/lib/vdsm/Makefile.am index cdc0247..8f9d4d2 100644 --- a/lib/vdsm/Makefile.am +++ b/lib/vdsm/Makefile.am @@ -31,6 +31,7 @@ cmdutils.py \ commands.py \ constants.py \ + config.py \ containersconnection.py \ cpuarch.py \ cpuinfo.py \ @@ -68,7 +69,6 @@ $(NULL) nodist_vdsmpylib_PYTHON = \ - config.py \ dsaversion.py \ $(NULL) @@ -83,13 +83,12 @@ EXTRA_DIST = \ - config.py.in \ dsaversion.py.in \ vdsm.conf.sample \ $(NULL) -vdsm.conf.sample: Makefile config.py - @echo " CONFIG $@"; $(PYTHON) $(builddir)/config.py > $@ +vdsm.conf.sample: Makefile common/config.py + @echo " CONFIG $@"; $(PYTHON) -m common.config > $@ install-data-local: \ install-data-sample-vdsm-conf diff --git a/lib/vdsm/common/Makefile.am b/lib/vdsm/common/Makefile.am index 0e8de2b..255b231 100644 --- a/lib/vdsm/common/Makefile.am +++ b/lib/vdsm/common/Makefile.am @@ -29,6 +29,7 @@ dist_vdsmcommon_PYTHON = *.py EXTRA_DIST = \ + config.py.in \ constants.py.in \ $(NULL) diff --git a/lib/vdsm/config.py.in b/lib/vdsm/common/config.py.in similarity index 100% rename from lib/vdsm/config.py.in rename to lib/vdsm/common/config.py.in -- To view, visit https://gerrit.ovirt.org/79980 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ie84514aed56034c79044a94f7ce391b368cb99c4 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Dan Kenigsberg <danken(a)redhat.com>

1 5

Change in vdsm[master]: hooking: move to lib
by Code Review 23 Aug '17

23 Aug '17

From Dan Kenigsberg <danken(a)redhat.com>: Dan Kenigsberg has submitted this change and it was merged. ( https://gerrit.ovirt.org/80842 ) Change subject: hooking: move to lib ...................................................................... hooking: move to lib As part of broader migration we move hooking.py away from vdsm dir. Change-Id: Ia8d7ac3d8730b3961e420936536f3ee5a055d336 Signed-off-by: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> --- M Makefile.am M configure.ac M lib/vdsm/Makefile.am A lib/vdsm/hook/Makefile.am A lib/vdsm/hook/__init__.py R lib/vdsm/hook/hooking.py M lib/vdsm/hooks.py M vdsm.spec.in M vdsm/Makefile.am 9 files changed, 33 insertions(+), 7 deletions(-) Approvals: Piotr Kliczewski: Verified Yaniv Bronhaim: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Dan Kenigsberg: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/80842 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia8d7ac3d8730b3961e420936536f3ee5a055d336 Gerrit-PatchSet: 3 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: Irit Goihman <igoihman(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Michael Burman <mburman(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]: hooking: move to lib
by Code Review 23 Aug '17

23 Aug '17

From Dan Kenigsberg <danken(a)redhat.com>: Dan Kenigsberg has posted comments on this change. Change subject: hooking: move to lib ...................................................................... Patch Set 2: Code-Review+2 -- To view, visit https://gerrit.ovirt.org/80842 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia8d7ac3d8730b3961e420936536f3ee5a055d336 Gerrit-PatchSet: 2 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: Irit Goihman <igoihman(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Michael Burman <mburman(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]: hooking: move to lib
by Code Review 23 Aug '17

23 Aug '17

From Yaniv Bronhaim <ybronhei(a)redhat.com>: Yaniv Bronhaim has posted comments on this change. Change subject: hooking: move to lib ...................................................................... Patch Set 2: Code-Review+1 -- To view, visit https://gerrit.ovirt.org/80842 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia8d7ac3d8730b3961e420936536f3ee5a055d336 Gerrit-PatchSet: 2 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: Irit Goihman <igoihman(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Michael Burman <mburman(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]: metrics: stop sending host metrics except storage
by Code Review 23 Aug '17

23 Aug '17

From Yaniv Bronhaim <ybronhei(a)redhat.com>: Yaniv Bronhaim has posted comments on this change. Change subject: metrics: stop sending host metrics except storage ...................................................................... Patch Set 2: Verified+1 -- To view, visit https://gerrit.ovirt.org/80050 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic39c3eee94e927446df0e7f93f53f54cfe96ff5e Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Irit Goihman <igoihman(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.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[ovirt-4.1]: stomp: stop using hard coded values for heartbeats
by Code Review 23 Aug '17

23 Aug '17

From Yaniv Bronhaim <ybronhei(a)redhat.com>: Yaniv Bronhaim has submitted this change and it was merged. ( https://gerrit.ovirt.org/78918 ) Change subject: stomp: stop using hard coded values for heartbeats ...................................................................... stomp: stop using hard coded values for heartbeats We wish to support incoming heartbeats in JSONRPC client in order to implement reconnect mechanism. AsyncClient connect method used to send HEARTBEAT header with hard coded values, which are now configurable and enable both ways heartbeats. Change-Id: I19cfec389e9d24d70e64617471df0d1dceb4229f Signed-off-by: Irit Goihman <igoihman(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/1376843 --- M lib/yajsonrpc/stomp.py M lib/yajsonrpc/stompreactor.py M tests/stompAsyncClientTests.py 3 files changed, 28 insertions(+), 9 deletions(-) Approvals: Piotr Kliczewski: Looks good to me, but someone else must approve Yaniv Bronhaim: Looks good to me, approved; Passed CI tests Irit Goihman: Verified -- To view, visit https://gerrit.ovirt.org/78918 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I19cfec389e9d24d70e64617471df0d1dceb4229f Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: ovirt-4.1 Gerrit-Owner: Irit Goihman <igoihman(a)redhat.com> Gerrit-Reviewer: Irit Goihman <igoihman(a)redhat.com> Gerrit-Reviewer: Jenkins CI 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[ovirt-4.1]: stomp: stop using hard coded values for heartbeats
by Code Review 23 Aug '17

23 Aug '17

From Yaniv Bronhaim <ybronhei(a)redhat.com>: Yaniv Bronhaim has posted comments on this change. Change subject: stomp: stop using hard coded values for heartbeats ...................................................................... Patch Set 5: Continuous-Integration+1 -- To view, visit https://gerrit.ovirt.org/78918 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I19cfec389e9d24d70e64617471df0d1dceb4229f Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: ovirt-4.1 Gerrit-Owner: Irit Goihman <igoihman(a)redhat.com> Gerrit-Reviewer: Irit Goihman <igoihman(a)redhat.com> Gerrit-Reviewer: Jenkins CI 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

← Newer
1
2
3
4
5
6
7
...
21
Older →

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

vdsm-patches August 2017