4:08 a.m.
Martin Polednik has uploaded a new change for review.
Change subject: ppc64le: vm.py support for running ppc64le systems
......................................................................
ppc64le: vm.py support for running ppc64le systems
Change-Id: I1ad44c79bfee11745cce530312f283c8d0b5ce9a
Signed-off-by: Martin Polednik <mpolednik(a)redhat.com>
---
M vdsm/virt/vm.py
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/16/39216/1
diff --git a/vdsm/virt/vm.py b/vdsm/virt/vm.py
index c5221ca..7a64b30 100644
--- a/vdsm/virt/vm.py
+++ b/vdsm/virt/vm.py
@@ -804,7 +804,7 @@
self._watchdogEvent = {}
self.arch = caps.getTargetArch()
- if (self.arch not in ['ppc64', 'x86_64']):
+ if self.arch not in caps.Architecture.POWER or self.arch != 'x86_64':
raise RuntimeError('Unsupported architecture: %s' % self.arch)
self._powerDownEvent = threading.Event()
--
To view, visit https://gerrit.ovirt.org/39216
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I1ad44c79bfee11745cce530312f283c8d0b5ce9a
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik(a)redhat.com>
4:08 a.m.
automation(a)ovirt.org has posted comments on this change.
Change subject: ppc64le: vm.py support for running ppc64le systems
......................................................................
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.5',
'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/39216
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1ad44c79bfee11745cce530312f283c8d0b5ce9a
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-HasComments: No
4:08 a.m.
oVirt Jenkins CI Server has posted comments on this change.
Change subject: ppc64le: vm.py support for running ppc64le systems
......................................................................
Patch Set 1:
Build Started (1/2) -> http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/17152/
--
To view, visit https://gerrit.ovirt.org/39216
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1ad44c79bfee11745cce530312f283c8d0b5ce9a
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
4:13 a.m.
automation(a)ovirt.org has posted comments on this change.
Change subject: ppc64le: vm.py support for running ppc64le systems
......................................................................
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.5',
'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/39216
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1ad44c79bfee11745cce530312f283c8d0b5ce9a
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
4:13 a.m.
oVirt Jenkins CI Server has posted comments on this change.
Change subject: ppc64le: vm.py support for running ppc64le systems
......................................................................
Patch Set 2:
Build Started (1/2) -> http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/17154/
--
To view, visit https://gerrit.ovirt.org/39216
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1ad44c79bfee11745cce530312f283c8d0b5ce9a
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
4:21 a.m.
Martin Polednik has posted comments on this change.
Change subject: ppc64le: vm.py support for running ppc64le systems
......................................................................
Patch Set 2: Verified+1
Verified by being able to start vdsm VM on ppc64le.
--
To view, visit https://gerrit.ovirt.org/39216
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1ad44c79bfee11745cce530312f283c8d0b5ce9a
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
4:27 a.m.
oVirt Jenkins CI Server has posted comments on this change.
Change subject: ppc64le: vm.py support for running ppc64le systems
......................................................................
Patch Set 1:
Build Started (2/2) ->
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/17326/
--
To view, visit https://gerrit.ovirt.org/39216
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1ad44c79bfee11745cce530312f283c8d0b5ce9a
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
4:27 a.m.
Francesco Romani has posted comments on this change.
Change subject: ppc64le: vm.py support for running ppc64le systems
......................................................................
Patch Set 2: Code-Review+1
(1 comment)
https://gerrit.ovirt.org/#/c/39216/2/vdsm/virt/vm.py
File vdsm/virt/vm.py:
Line 803: self.saveState()
Line 804: self._watchdogEvent = {}
Line 805: self.arch = caps.getTargetArch()
Line 806:
Line 807: if self.arch not in caps.Architecture.POWER or self.arch !=
'x86_64':
In a future patch: can we use nicer caps.Architecutre.SOMETHING also for x86_64 here?
Could it be nicer if we add a classmethod like supported() to caps.Architecture so
we'll be able to write
if not caps.Architecture.supported(self.arch):
...
Line 808: raise RuntimeError('Unsupported architecture: %s' %
self.arch)
Line 809:
Line 810: self._powerDownEvent = threading.Event()
Line 811: self._liveMergeCleanupThreads = {}
--
To view, visit https://gerrit.ovirt.org/39216
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1ad44c79bfee11745cce530312f283c8d0b5ce9a
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
4:29 a.m.
oVirt Jenkins CI Server has posted comments on this change.
Change subject: ppc64le: vm.py support for running ppc64le systems
......................................................................
Patch Set 1:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/17152/ : SUCCESS
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/17326/ : FAILURE
--
To view, visit https://gerrit.ovirt.org/39216
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1ad44c79bfee11745cce530312f283c8d0b5ce9a
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
4:49 a.m.
oVirt Jenkins CI Server has posted comments on this change.
Change subject: ppc64le: vm.py support for running ppc64le systems
......................................................................
Patch Set 2:
Build Started (2/2) ->
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/17328/
--
To view, visit https://gerrit.ovirt.org/39216
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1ad44c79bfee11745cce530312f283c8d0b5ce9a
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
4:51 a.m.
oVirt Jenkins CI Server has posted comments on this change.
Change subject: ppc64le: vm.py support for running ppc64le systems
......................................................................
Patch Set 2:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/17154/ : SUCCESS
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/17328/ : FAILURE
--
To view, visit https://gerrit.ovirt.org/39216
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1ad44c79bfee11745cce530312f283c8d0b5ce9a
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
5:11 a.m.
automation(a)ovirt.org has posted comments on this change.
Change subject: ppc64le: vm.py support for running ppc64le systems
......................................................................
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.5',
'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/39216
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1ad44c79bfee11745cce530312f283c8d0b5ce9a
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
5:11 a.m.
oVirt Jenkins CI Server has posted comments on this change.
Change subject: ppc64le: vm.py support for running ppc64le systems
......................................................................
Patch Set 3:
Build Started (1/2) ->
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/17332/
--
To view, visit https://gerrit.ovirt.org/39216
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1ad44c79bfee11745cce530312f283c8d0b5ce9a
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
5:11 a.m.
oVirt Jenkins CI Server has posted comments on this change.
Change subject: ppc64le: vm.py support for running ppc64le systems
......................................................................
Patch Set 3:
Build Started (2/2) -> http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/17158/
--
To view, visit https://gerrit.ovirt.org/39216
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1ad44c79bfee11745cce530312f283c8d0b5ce9a
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
5:13 a.m.
oVirt Jenkins CI Server has posted comments on this change.
Change subject: ppc64le: vm.py support for running ppc64le systems
......................................................................
Patch Set 3:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/17158/ : SUCCESS
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/17332/ : SUCCESS
--
To view, visit https://gerrit.ovirt.org/39216
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1ad44c79bfee11745cce530312f283c8d0b5ce9a
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
5:29 a.m.
Martin Polednik has posted comments on this change.
Change subject: ppc64le: vm.py support for running ppc64le systems
......................................................................
Patch Set 3: Verified+1
Manually tried to start VM on power8 LE, got as far as generating XML.
--
To view, visit https://gerrit.ovirt.org/39216
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1ad44c79bfee11745cce530312f283c8d0b5ce9a
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
5:37 a.m.
Francesco Romani has posted comments on this change.
Change subject: ppc64le: vm.py support for running ppc64le systems
......................................................................
Patch Set 3: Code-Review+1
(2 comments)
https://gerrit.ovirt.org/#/c/39216/3//COMMIT_MSG
Commit Message:
Line 3: AuthorDate: 2015-03-23 17:36:12 +0100
Line 4: Commit: Martin Polednik <mpolednik(a)redhat.com>
Line 5: CommitDate: 2015-03-26 11:10:32 +0100
Line 6:
Line 7: ppc64le: vm.py support for running ppc64le systems
s/vm.py/virt/
my rule is:
vm: if the patch affects vm only (and it is quite often the case, e.g. the Vm class)
virt: if the patch affects more than a module of the virt package, or something in the
'virt' vertical
Line 8:
Line 9: Our architecture check lacked ppc64le. This patch fixes it by adding it
Line 10: as a supported architecture and adding default machine type for it.
Line 11:
https://gerrit.ovirt.org/#/c/39216/3/vdsm/virt/vm.py
File vdsm/virt/vm.py:
Line 804: self._watchdogEvent = {}
Line 805: self.arch = caps.getTargetArch()
Line 806:
Line 807: if self.arch not in caps.Architecture.POWER and \
Line 808: self.arch != caps.Architecture.X86_64:
thanks :)
Line 809: raise RuntimeError('Unsupported architecture: %s' %
self.arch)
Line 810:
Line 811: self._powerDownEvent = threading.Event()
Line 812: self._liveMergeCleanupThreads = {}
--
To view, visit https://gerrit.ovirt.org/39216
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1ad44c79bfee11745cce530312f283c8d0b5ce9a
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
6:35 a.m.
New subject: Change in vdsm[master]: ppc64le: virt: support for running ppc64le systems
automation(a)ovirt.org has posted comments on this change.
Change subject: ppc64le: virt: support for running ppc64le systems
......................................................................
Patch Set 4:
* 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.5',
'ovirt-3.4', 'ovirt-3.3'])
--
To view, visit https://gerrit.ovirt.org/39216
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1ad44c79bfee11745cce530312f283c8d0b5ce9a
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
6:35 a.m.
New subject: Change in vdsm[master]: ppc64le: virt: support for running ppc64le systems
oVirt Jenkins CI Server has posted comments on this change.
Change subject: ppc64le: virt: support for running ppc64le systems
......................................................................
Patch Set 4:
Build Started (1/2) -> http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/17164/
--
To view, visit https://gerrit.ovirt.org/39216
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1ad44c79bfee11745cce530312f283c8d0b5ce9a
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
6:39 a.m.
New subject: Change in vdsm[master]: ppc64le: virt: support for running ppc64le systems
oVirt Jenkins CI Server has posted comments on this change.
Change subject: ppc64le: virt: support for running ppc64le systems
......................................................................
Patch Set 4:
Build Started (2/2) ->
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/17338/
--
To view, visit https://gerrit.ovirt.org/39216
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1ad44c79bfee11745cce530312f283c8d0b5ce9a
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
6:41 a.m.
New subject: Change in vdsm[master]: ppc64le: virt: support for running ppc64le systems
oVirt Jenkins CI Server has posted comments on this change.
Change subject: ppc64le: virt: support for running ppc64le systems
......................................................................
Patch Set 4:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/17164/ : SUCCESS
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/17338/ : SUCCESS
--
To view, visit https://gerrit.ovirt.org/39216
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1ad44c79bfee11745cce530312f283c8d0b5ce9a
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
9:57 a.m.
New subject: Change in vdsm[master]: ppc64le: virt: support for running ppc64le systems
Dan Kenigsberg has posted comments on this change.
Change subject: ppc64le: virt: support for running ppc64le systems
......................................................................
Patch Set 4: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/39216
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1ad44c79bfee11745cce530312f283c8d0b5ce9a
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
9:58 a.m.
New subject: Change in vdsm[master]: ppc64le: virt: support for running ppc64le systems
Dan Kenigsberg has submitted this change and it was merged.
Change subject: ppc64le: virt: support for running ppc64le systems
......................................................................
ppc64le: virt: support for running ppc64le systems
Our architecture check lacked ppc64le. This patch fixes it by adding it
as a supported architecture and adding default machine type for it.
Change-Id: I1ad44c79bfee11745cce530312f283c8d0b5ce9a
Signed-off-by: Martin Polednik <mpolednik(a)redhat.com>
Reviewed-on: https://gerrit.ovirt.org/39216
Reviewed-by: Francesco Romani <fromani(a)redhat.com>
Reviewed-by: Dan Kenigsberg <danken(a)redhat.com>
---
M vdsm/virt/vm.py
M vdsm/virt/vmxml.py
2 files changed, 4 insertions(+), 2 deletions(-)
Approvals:
Dan Kenigsberg: Looks good to me, approved
Francesco Romani: Looks good to me, but someone else must approve
Martin Polednik: Verified
--
To view, visit https://gerrit.ovirt.org/39216
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I1ad44c79bfee11745cce530312f283c8d0b5ce9a
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
9:58 a.m.
New subject: Change in vdsm[master]: ppc64le: virt: support for running ppc64le systems
automation(a)ovirt.org has posted comments on this change.
Change subject: ppc64le: virt: support for running ppc64le systems
......................................................................
Patch Set 5:
* Update tracker::IGNORE, no Bug-Url found
* Set MODIFIED::IGNORE, no Bug-Url found.
--
To view, visit https://gerrit.ovirt.org/39216
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1ad44c79bfee11745cce530312f283c8d0b5ce9a
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
10:56 a.m.
New subject: Change in vdsm[master]: ppc64le: virt: support for running ppc64le systems
oVirt Jenkins CI Server has posted comments on this change.
Change subject: ppc64le: virt: support for running ppc64le systems
......................................................................
Patch Set 5:
Build Started (1/9) ->
http://jenkins.ovirt.org/job/vdsm_master_create-rpms-el6-x86_64_merged/806/
--
To view, visit https://gerrit.ovirt.org/39216
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1ad44c79bfee11745cce530312f283c8d0b5ce9a
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
11:02 a.m.
New subject: Change in vdsm[master]: ppc64le: virt: support for running ppc64le systems
oVirt Jenkins CI Server has posted comments on this change.
Change subject: ppc64le: virt: support for running ppc64le systems
......................................................................
Patch Set 5:
Build Started (2/9) ->
http://jenkins.ovirt.org/job/vdsm_master-libgfapi_create-rpms-el6-x86_64_...
--
To view, visit https://gerrit.ovirt.org/39216
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1ad44c79bfee11745cce530312f283c8d0b5ce9a
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
11:10 a.m.
New subject: Change in vdsm[master]: ppc64le: virt: support for running ppc64le systems
oVirt Jenkins CI Server has posted comments on this change.
Change subject: ppc64le: virt: support for running ppc64le systems
......................................................................
Patch Set 5:
Build Started (3/9) ->
http://jenkins.ovirt.org/job/vdsm_master_create-rpms-el7-x86_64_merged/811/
--
To view, visit https://gerrit.ovirt.org/39216
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1ad44c79bfee11745cce530312f283c8d0b5ce9a
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
11:16 a.m.
New subject: Change in vdsm[master]: ppc64le: virt: support for running ppc64le systems
oVirt Jenkins CI Server has posted comments on this change.
Change subject: ppc64le: virt: support for running ppc64le systems
......................................................................
Patch Set 5:
Build Started (4/9) ->
http://jenkins.ovirt.org/job/vdsm_master_create-rpms-fc20-x86_64_merged/812/
--
To view, visit https://gerrit.ovirt.org/39216
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1ad44c79bfee11745cce530312f283c8d0b5ce9a
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
11:16 a.m.
New subject: Change in vdsm[master]: ppc64le: virt: support for running ppc64le systems
oVirt Jenkins CI Server has posted comments on this change.
Change subject: ppc64le: virt: support for running ppc64le systems
......................................................................
Patch Set 5:
Build Started (5/9) ->
http://jenkins.ovirt.org/job/vdsm_master_create-rpms-fc21-x86_64_merged/778/
--
To view, visit https://gerrit.ovirt.org/39216
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1ad44c79bfee11745cce530312f283c8d0b5ce9a
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
11:20 a.m.
New subject: Change in vdsm[master]: ppc64le: virt: support for running ppc64le systems
oVirt Jenkins CI Server has posted comments on this change.
Change subject: ppc64le: virt: support for running ppc64le systems
......................................................................
Patch Set 5:
Build Started (6/9) ->
http://jenkins.ovirt.org/job/vdsm_master-libgfapi_create-rpms-fc21-x86_64...
--
To view, visit https://gerrit.ovirt.org/39216
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1ad44c79bfee11745cce530312f283c8d0b5ce9a
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
11:20 a.m.
New subject: Change in vdsm[master]: ppc64le: virt: support for running ppc64le systems
oVirt Jenkins CI Server has posted comments on this change.
Change subject: ppc64le: virt: support for running ppc64le systems
......................................................................
Patch Set 5:
Build Started (7/9) ->
http://jenkins.ovirt.org/job/vdsm_master-libgfapi_create-rpms-el7-x86_64_...
--
To view, visit https://gerrit.ovirt.org/39216
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1ad44c79bfee11745cce530312f283c8d0b5ce9a
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
11:23 a.m.
New subject: Change in vdsm[master]: ppc64le: virt: support for running ppc64le systems
oVirt Jenkins CI Server has posted comments on this change.
Change subject: ppc64le: virt: support for running ppc64le systems
......................................................................
Patch Set 5:
Build Started (8/9) ->
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_merged/4787/
--
To view, visit https://gerrit.ovirt.org/39216
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1ad44c79bfee11745cce530312f283c8d0b5ce9a
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
11:47 a.m.
New subject: Change in vdsm[master]: ppc64le: virt: support for running ppc64le systems
oVirt Jenkins CI Server has posted comments on this change.
Change subject: ppc64le: virt: support for running ppc64le systems
......................................................................
Patch Set 5:
Build Started (9/9) ->
http://jenkins.ovirt.org/job/vdsm_master-libgfapi_create-rpms-fc20-x86_64...
--
To view, visit https://gerrit.ovirt.org/39216
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1ad44c79bfee11745cce530312f283c8d0b5ce9a
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
12:06 p.m.
New subject: Change in vdsm[master]: ppc64le: virt: support for running ppc64le systems
oVirt Jenkins CI Server has posted comments on this change.
Change subject: ppc64le: virt: support for running ppc64le systems
......................................................................
Patch Set 5:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_create-rpms-el6-x86_64_merged/806/ : SUCCESS
http://jenkins.ovirt.org/job/vdsm_master-libgfapi_create-rpms-fc21-x86_64... :
SUCCESS
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_merged/4787/ : SUCCESS
http://jenkins.ovirt.org/job/vdsm_master_create-rpms-fc21-x86_64_merged/778/ : SUCCESS
http://jenkins.ovirt.org/job/vdsm_master-libgfapi_create-rpms-el6-x86_64_... :
SUCCESS
http://jenkins.ovirt.org/job/vdsm_master_create-rpms-el7-x86_64_merged/811/ : SUCCESS
http://jenkins.ovirt.org/job/vdsm_master-libgfapi_create-rpms-el7-x86_64_... :
SUCCESS
http://jenkins.ovirt.org/job/vdsm_master-libgfapi_create-rpms-fc20-x86_64... :
SUCCESS
http://jenkins.ovirt.org/job/vdsm_master_create-rpms-fc20-x86_64_merged/812/ : SUCCESS
--
To view, visit https://gerrit.ovirt.org/39216
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1ad44c79bfee11745cce530312f283c8d0b5ce9a
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
3344
days inactive
3344
days old
vdsm-patches@lists.fedorahosted.org
33 comments
5 participants
participants (5)
-
automation＠ovirt.org -
Dan Kenigsberg -
fromani＠redhat.com -
Jenkins CI RO -
Martin Polednik