April 2016 - Engine-commits - oVirt List Archives

Change in ovirt-hosted-engine-ha[master]: vdsm: switch from XML-RPC to JSON-RPC API
by msivak＠redhat.com 27 Apr '16

27 Apr '16

Martin Sivák has submitted this change and it was merged. Change subject: vdsm: switch from XML-RPC to JSON-RPC API ...................................................................... vdsm: switch from XML-RPC to JSON-RPC API Requires: https://gerrit.ovirt.org/55252 Change-Id: I0417f3a53caef4069bf5750aadb2bcbd3b2eb552 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M ovirt-hosted-engine-ha.spec.in M ovirt_hosted_engine_ha/agent/hosted_engine.py M ovirt_hosted_engine_ha/env/path.py M ovirt_hosted_engine_ha/lib/heconflib.py M ovirt_hosted_engine_ha/lib/image.py M ovirt_hosted_engine_ha/lib/ovf/ovf_store.py M ovirt_hosted_engine_ha/lib/storage_server.py M ovirt_hosted_engine_ha/lib/upgrade.py M ovirt_hosted_engine_ha/lib/util.py M ovirt_hosted_engine_ha/lib/vds_client.py 10 files changed, 247 insertions(+), 174 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Martin Sivák: Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/56217 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0417f3a53caef4069bf5750aadb2bcbd3b2eb552 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Fix typo in BaseDisk entity commment
by amureini＠redhat.com 27 Apr '16

27 Apr '16

Allon Mureinik has submitted this change and it was merged. Change subject: core: Fix typo in BaseDisk entity commment ...................................................................... core: Fix typo in BaseDisk entity commment Change-Id: Idabc8abfffe21b1dc4efffd9ed43ebfda859b0eb Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/storage/BaseDisk.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/56701 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idabc8abfffe21b1dc4efffd9ed43ebfda859b0eb Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: vm init not loaded for new vm from template
by mbetak＠redhat.com 27 Apr '16

27 Apr '16

Martin Betak has submitted this change and it was merged. Change subject: webadmin: vm init not loaded for new vm from template ...................................................................... webadmin: vm init not loaded for new vm from template The VM init is not loaded by default for all VMs and it always has to be loaded when the dialog is being populated. This has been forgotten for new Vm from template on the template tab Change-Id: I5e37f637b84d4a6f4e9e540715d12c18f9be61db Bug-Url: https://bugzilla.redhat.com/1328463 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/templates/TemplateListModel.java 1 file changed, 9 insertions(+), 3 deletions(-) Approvals: Tomas Jelinek: Verified Jenkins CI: Passed CI tests Martin Betak: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/56382 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5e37f637b84d4a6f4e9e540715d12c18f9be61db Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

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