Bing Bu Cao has uploaded a new change for review.
Change subject: Add shutdown based on qemu-ga(qemu guest agent) in vdsm
......................................................................
Add shutdown based on qemu-ga(qemu guest agent) in vdsm
As previously what we agreed, the agent-assisted shutdown and fsfreeze would be handled
by the qemu guest agent while oVirt-specific functionality such as
Single-Sign-On would continue to be managed by the ovirt guest agent.
http://www.ovirt.org/wiki/Guest_agent_proposals
This patch enables the qemu guest agent channel in the libvirt domain XML and
changes the shutdown verb to pass the VIR_DOMAIN_SHUTDOWN_GUEST_AGENT flag to
the libvirt shutdown API.
Change-Id: If934c68552888c69c5a971c6f6140f59a9c6acc7
Signed-off-by: Cao,Bing Bu <mars(a)linux.vnet.ibm.com>
---
M vdsm/libvirtvm.py
M vdsm/vm.py
M vdsm_cli/vdsClient.py
3 files changed, 51 insertions(+), 9 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/40/5640/1
--
To view, visit http://gerrit.ovirt.org/5640
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: If934c68552888c69c5a971c6f6140f59a9c6acc7
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Bing Bu Cao <mars(a)linux.vnet.ibm.com>
Mark Wu has uploaded a new change for review.
Change subject: configNetworkTests: Fix wrong key for interface of bridgeless network
......................................................................
configNetworkTests: Fix wrong key for interface of bridgeless network
It breaks build with the following exception:
ERROR: testAddNetworkValidation (configNetworkTests.TestconfigNetwork)
----------------------------------------------------------------------
Traceback (most recent call last):
File "/home/mark/Work/vdsm/vdsm-src/tests/configNetworkTests.py", line 261, in testAddNetworkValidation
configNetwork.ne.ERR_USED_NIC)
File "/home/mark/Work/vdsm/vdsm-src/tests/configNetworkTests.py", line 185, in _addNetworkWithExc
configNetwork._addNetworkValidation(*parameters)
File "/home/mark/Work/vdsm/vdsm-src/vdsm/configNetwork.py", line 599, in _addNetworkValidation
_validateInterNetworkCompatibility(_netinfo, vlan, bonding, bridged)
File "/home/mark/Work/vdsm/vdsm-src/vdsm/configNetwork.py", line 534, in _validateInterNetworkCompatibility
ifaces_bridgeless = tuple(ni.getBridgelessNetworksAndVlansForIface(iface))
File "/home/mark/Work/vdsm/vdsm-src/vdsm/netinfo.py", line 359, in getBridgelessNetworksAndVlansForIface
if iface == netdict['iface']:
KeyError: 'iface'
Change-Id: Ib070e476f8fa059c2c181e52768d7623526cf8db
Signed-off-by: Mark Wu <wudxw(a)linux.vnet.ibm.com>
---
M tests/configNetworkTests.py
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/45/6845/1
--
To view, visit http://gerrit.ovirt.org/6845
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib070e476f8fa059c2c181e52768d7623526cf8db
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Mark Wu <wudxw(a)linux.vnet.ibm.com>
Saggi Mizrahi has uploaded a new change for review.
Change subject: Change exception thrown by validateQemuReadable
......................................................................
Change exception thrown by validateQemuReadable
Serializing obejcts that aren't canonical might cause issue with
pickling. In anycase, the gobal exceptions should not be used so low in
the stack.
Change-Id: I0c6a4fded368e89f057d244665be0e37d71f2d08
Signed-off-by: Saggi Mizrahi <smizrahi(a)redhat.com>
---
M vdsm/storage/fileSD.py
M vdsm/storage/fileUtils.py
2 files changed, 5 insertions(+), 2 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/63/6463/1
--
To view, visit http://gerrit.ovirt.org/6463
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I0c6a4fded368e89f057d244665be0e37d71f2d08
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi <smizrahi(a)redhat.com>
Dan Kenigsberg has uploaded a new change for review.
Change subject: configNetworkTests: test new "iface" API
......................................................................
configNetworkTests: test new "iface" API
commit 1cb43db326c8 has incurred a change in the reported attributes per
network. This needs to be translated to a compatible change in the test.
This issue has not been noticed due to a human race condition: the test
was verified before the API change was made, but committed after it.
Change-Id: I5f22f11dc593460a661b322e35422b0039306c19
Signed-off-by: Dan Kenigsberg <danken(a)redhat.com>
---
M tests/configNetworkTests.py
1 file changed, 5 insertions(+), 5 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/37/6837/1
--
To view, visit http://gerrit.ovirt.org/6837
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I5f22f11dc593460a661b322e35422b0039306c19
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <danken(a)redhat.com>
Jarod.w has uploaded a new change for review.
Change subject: deployUtil: avoid garbage character in auth key file
......................................................................
deployUtil: avoid garbage character in auth key file
If we fail to get file '/engine.ssh.key.txt', it will get the
content of url 'http://your_engine_ip:your_port/' which is not
what we want.
Change-Id: I93ca8b04aaac571afa567833360df9eed98f61ef
Signed-off-by: jarod.w <work.iec23801(a)gmail.com>
---
M vdsm_reg/deployUtil.py.in
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/55/6755/1
--
To view, visit http://gerrit.ovirt.org/6755
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I93ca8b04aaac571afa567833360df9eed98f61ef
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Jarod.w <work.iec23801(a)gmail.com>
Josef Pacula has uploaded a new change for review.
Change subject: with exceptions some variables could be uninitialised - now initialised to None
......................................................................
with exceptions some variables could be uninitialised - now initialised to None
Change-Id: Ie23c9b643b82fed2281a34e5c7b6cd1d935b1d19
Signed-off-by: Josef Pacula <josef.pacula(a)gmail.com>
---
M vdsm/mkimage.py
1 file changed, 2 insertions(+), 0 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/62/6662/1
--
To view, visit http://gerrit.ovirt.org/6662
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie23c9b643b82fed2281a34e5c7b6cd1d935b1d19
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Josef Pacula <josef.pacula(a)gmail.com>