- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: restapi: editing vm_init overrides all attrs
by tjelinek＠redhat.com 27 Apr '16

27 Apr '16

Tomas Jelinek has submitted this change and it was merged. Change subject: restapi: editing vm_init overrides all attrs ...................................................................... restapi: editing vm_init overrides all attrs The problem was that regardless of the fact if the vm_init was already set on VM or not, a new has been created. Change-Id: If1072e04f7a62d30589114020b072091c772aeb3 Bug-Url: https://bugzilla.redhat.com/1328737 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmResource.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/TemplateMapper.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java 3 files changed, 7 insertions(+), 6 deletions(-) Approvals: Tomas Jelinek: Verified Shahar Havivi: Looks good to me, approved Juan Hernandez: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/56615 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If1072e04f7a62d30589114020b072091c772aeb3 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: vdsm: add back fc22 build on 3.6
by sbonazzo＠redhat.com 27 Apr '16

27 Apr '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: vdsm: add back fc22 build on 3.6 ...................................................................... vdsm: add back fc22 build on 3.6 partially revert https://gerrit.ovirt.org/55685 re-adding fc22 builds for 3.6 branches. Change-Id: Iea24c8a24c298b394976a8163bb79ce68331c713 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/vdsm/vdsm_standard.yaml 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/56681 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iea24c8a24c298b394976a8163bb79ce68331c713 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Add an option to search all available projects. Convert shel...
by eedri＠redhat.com 27 Apr '16

27 Apr '16

Eyal Edri has submitted this change and it was merged. Change subject: Add an option to search all available projects. Convert shell_command output string to list. Update notification email addresses. Removed space in print() Replaced .split('\n') with splitlines() Include project name in the email subject ...................................................................... Add an option to search all available projects. Convert shell_command output string to list. Update notification email addresses. Removed space in print() Replaced .split('\n') with splitlines() Include project name in the email subject Change-Id: I5bc3c0511e318e9d88c2360b5d09bb651828557e Signed-off-by: Vishnu Sreekumar <vishnu.srkmr(a)gmail.com> --- M scripts/alert_old_patches.py 1 file changed, 36 insertions(+), 7 deletions(-) Approvals: Eyal Edri: Verified; Looks good to me; Passed CI tests; Ready for merge -- To view, visit https://gerrit.ovirt.org/56243 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5bc3c0511e318e9d88c2360b5d09bb651828557e Gerrit-PatchSet: 7 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Vishnu Sreekumar <vishnu.srkmr(a)gmail.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Vishnu Sreekumar <vishnu.srkmr(a)gmail.com> Gerrit-Reviewer: Yaniv Kaul <ykaul(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: UI: add tooltip information for vCenter
by shavivi＠redhat.com 27 Apr '16

27 Apr '16

Shahar Havivi has submitted this change and it was merged. Change subject: UI: add tooltip information for vCenter ...................................................................... UI: add tooltip information for vCenter Change-Id: I1ac7e0aaae36a6ffaa3ee4101bc6456e8c29c0b0 Bug-Url: https://bugzilla.redhat.com/1316077 Signed-off-by: Shahar Havivi <shaharh(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/ImportVmsPopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/ImportVmsPopupView.ui.xml 3 files changed, 38 insertions(+), 6 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Shahar Havivi: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/56450 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1ac7e0aaae36a6ffaa3ee4101bc6456e8c29c0b0 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-scheduler-proxy[master]: automation: fix el6 and fc22 builds
by msivak＠redhat.com 27 Apr '16

27 Apr '16

Martin Sivák has submitted this change and it was merged. Change subject: automation: fix el6 and fc22 builds ...................................................................... automation: fix el6 and fc22 builds master branch is used for both 3.6 and 4.0. In 4.0 el6 and fc22 are not supported anymore so for building this package on 3.6 we need 3.6 repos. Change-Id: I5ac7e45d542d9c829ac9c190360dca9afdb3b9fb Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- A automation/build-artifacts.repos.el6 A automation/build-artifacts.repos.fc22 2 files changed, 4 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified Nadav Goldin: Looks good to me, but someone else must approve Martin Sivák: Looks good to me, approved Roman Mohr: Looks good to me, but someone else must approve Eyal Edri: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/56663 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5ac7e45d542d9c829ac9c190360dca9afdb3b9fb Gerrit-PatchSet: 1 Gerrit-Project: ovirt-scheduler-proxy Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Nadav Goldin <ngoldin(a)redhat.com> Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[ovirt-4.0]: epel: python-ecdsa has been renamed
by sbonazzo＠redhat.com 27 Apr '16

27 Apr '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: epel: python-ecdsa has been renamed ...................................................................... epel: python-ecdsa has been renamed Updated python-ecdsa to python2-ecdsa for epel7. Change-Id: Ib35305aae9b19b7ac1a3009e32cb5aaff97de485 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> (cherry picked from commit 0069d6e7400e0f2b513e02444cc1b99d095c4a2d) --- M ovirt-el7-deps.repo.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/56674 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib35305aae9b19b7ac1a3009e32cb5aaff97de485 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.0 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[ovirt-4.0]: pull python34-* from epel7
by sbonazzo＠redhat.com 27 Apr '16

27 Apr '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: pull python34-* from epel7 ...................................................................... pull python34-* from epel7 A recent Vdsm commit required these packages. It seems that they have to be explictly included here as well. Change-Id: If3a3598f70c54bc481734b7050e3fb167830257b Signed-off-by: Dan Kenigsberg <danken(a)redhat.com> (cherry picked from commit 9d1a64e7bde9ba58b90b19591b6e6d43d953dd20) --- M ovirt-el7-deps.repo.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/56670 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If3a3598f70c54bc481734b7050e3fb167830257b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.0 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[ovirt-4.0]: [Node]: Add requires for gluster pkgs
by sbonazzo＠redhat.com 27 Apr '16

27 Apr '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: [Node]: Add requires for gluster pkgs ...................................................................... [Node]: Add requires for gluster pkgs Gluster packages are required to be included in the node distro. Change-Id: I3cf5f91a904535eb0adc3efabe92c89ead5cd12e Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> (cherry picked from commit f727c15acbbdea6ff22728e5e792205231de511f) --- M ovirt-release40.spec.in 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/56671 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3cf5f91a904535eb0adc3efabe92c89ead5cd12e Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.0 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[ovirt-4.0]: node: Don't touch vdsm
by sbonazzo＠redhat.com 27 Apr '16

27 Apr '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: node: Don't touch vdsm ...................................................................... node: Don't touch vdsm vdsm has it's own presets for it's own services, thus we leave it untouched. Change-Id: I582f0333cf115eb67d9e4ed734a0fdbb0683c058 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> (cherry picked from commit bd0b2535bf30f4ded10d65d925ebdb58268554cf) --- M host-node/systemd/system-preset/98-ovirt-host-node.preset 1 file changed, 0 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/56672 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I582f0333cf115eb67d9e4ed734a0fdbb0683c058 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.0 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[ovirt-4.0]: node: Enable sshd.service by default
by sbonazzo＠redhat.com 27 Apr '16

27 Apr '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: node: Enable sshd.service by default ...................................................................... node: Enable sshd.service by default Previously we enabled sshd.socket by default, this lead to bug https://bugzilla.redhat.com/show_bug.cgi?id=1327332#c5 Now we just enable the sshd.service, which should avoid this bug, and also be aligned with what anaconda does. Change-Id: I88a0703b1a4818c4817637ce1e6021e979602b45 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1323975 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> (cherry picked from commit bca272a85765ce3a4f8268670322b4f15279a310) --- M host-node/systemd/system-preset/98-ovirt-host-node.preset 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/56673 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I88a0703b1a4818c4817637ce1e6021e979602b45 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.0 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0