June 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: gluster: Corrected geo-rep case of volume snapshot creation
by sabose＠redhat.com 23 Jun '15

23 Jun '15

Sahina Bose has submitted this change and it was merged. Change subject: gluster: Corrected geo-rep case of volume snapshot creation ...................................................................... gluster: Corrected geo-rep case of volume snapshot creation While creating gluster volume snapshot for a geo-replicated volume, we try to pause the geo-rep session and then create snapshot for the slave volume first. The pause of the slave volume should be done only if geo-rep session is not in one of the states from CREATED, PAUSED or STOPPED. Modified the code accordingly. Change-Id: Iaefc24c83168ce53960c9f90597a43d5030f7212 Bug-Url: https://bugzilla.redhat.com/1224902 Signed-off-by: Shubhendu Tripathi <shtripat(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/CreateGlusterVolumeSnapshotCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/PauseGlusterVolumeGeoRepSessionCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/RestoreGlusterVolumeSnapshotCommand.java 3 files changed, 85 insertions(+), 74 deletions(-) Approvals: Shubhendu Tripathi: Verified Sahina Bose: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/41491 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iaefc24c83168ce53960c9f90597a43d5030f7212 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: webadmin: change width of provider-url
by ahadas＠redhat.com 23 Jun '15

23 Jun '15

Arik Hadas has submitted this change and it was merged. Change subject: webadmin: change width of provider-url ...................................................................... webadmin: change width of provider-url Increase the width of provider-URL column in providers tab, so it could contain long URL such as needed for VMware. Change-Id: Ib52d4ac73769642d1a31dfbe67d6fda903aff571 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/MainTabProviderView.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Tomas Jelinek: Looks good to me, approved Martin Betak: Looks good to me, but someone else must approve Arik Hadas: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/42713 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib52d4ac73769642d1a31dfbe67d6fda903aff571 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(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: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: packaging: Point at conf.d directories in /etc
by sbonazzo＠redhat.com 23 Jun '15

23 Jun '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: Point at conf.d directories in /etc ...................................................................... packaging: Point at conf.d directories in /etc Change .conf files in /usr to point at conf.d directories in /etc. Change-Id: I20a1ad99151bfb492e70e032cf6de705fcb27c41 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/libexec/ovirt-vmconsole-proxy-helper/ovirt-vmconsole-proxy-helper.conf.in M packaging/services/ovirt-engine-notifier/ovirt-engine-notifier.conf.in M packaging/services/ovirt-engine/ovirt-engine.conf.in M packaging/services/ovirt-fence-kdump-listener/ovirt-fence-kdump-listener.conf.in M packaging/services/ovirt-websocket-proxy/ovirt-websocket-proxy.conf.in 5 files changed, 15 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests Francesco Romani: Verified -- To view, visit https://gerrit.ovirt.org/42725 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I20a1ad99151bfb492e70e032cf6de705fcb27c41 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: webadmin: utility method to construct vmware url
by ahadas＠redhat.com 23 Jun '15

23 Jun '15

Arik Hadas has submitted this change and it was merged. Change subject: webadmin: utility method to construct vmware url ...................................................................... webadmin: utility method to construct vmware url Add static method that constructs URL for VMware vCenter. This method is accessible as a utility method for other places that need to construct such URL, and it can construct the URL without username. Change-Id: Ie6973d680ccca3a097bfcee798141de300586449 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/ImportVmsModel.java 1 file changed, 12 insertions(+), 6 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Arik Hadas: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/42712 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie6973d680ccca3a097bfcee798141de300586449 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(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: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: webadmin: minor cleanup in ProviderModel
by ahadas＠redhat.com 23 Jun '15

23 Jun '15

Arik Hadas has submitted this change and it was merged. Change subject: webadmin: minor cleanup in ProviderModel ...................................................................... webadmin: minor cleanup in ProviderModel 1. Use daemon operator 2. Remove temp variables Change-Id: Ia728422647fcea1fbd2c1133b7d712a0f7bcf898 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/providers/ProviderModel.java 1 file changed, 11 insertions(+), 13 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Jenkins CI: Passed CI tests Martin Betak: Looks good to me, but someone else must approve Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/42711 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia728422647fcea1fbd2c1133b7d712a0f7bcf898 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(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: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: host-deploy: return http error if invalid command
by masayag＠redhat.com 23 Jun '15

23 Jun '15

Moti Asayag has submitted this change and it was merged. Change subject: host-deploy: return http error if invalid command ...................................................................... host-deploy: return http error if invalid command Change-Id: I79ded7365eb9e726a0f31b1b4bf4b725e88d6e53 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/modules/services/src/main/java/org/ovirt/engine/core/services/RegisterServlet.java 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Alon Bar-Lev: Verified Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/42737 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I79ded7365eb9e726a0f31b1b4bf4b725e88d6e53 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: support change quota for cinder disk
by derez＠redhat.com 23 Jun '15

23 Jun '15

Daniel Erez has submitted this change and it was merged. Change subject: core: support change quota for cinder disk ...................................................................... core: support change quota for cinder disk Supporting cinder on ChangeQuotaCommand. Change-Id: I99110eadbd092c7a2a30f65bd61dfae0c8533a53 Bug-Url: https://bugzilla.redhat.com/1185826 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ChangeQuotaForDiskCommand.java 1 file changed, 1 insertion(+), 2 deletions(-) Approvals: Daniel Erez: Verified; Passed CI tests Maor Lipchuk: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/42701 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I99110eadbd092c7a2a30f65bd61dfae0c8533a53 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: support quota on add cinder disk
by derez＠redhat.com 23 Jun '15

23 Jun '15

Daniel Erez has submitted this change and it was merged. Change subject: core: support quota on add cinder disk ...................................................................... core: support quota on add cinder disk Supporting quota for add cinder disk. Change-Id: I51b27f6104d6aeba8bebdbc13919bf41e511f4d3 Bug-Url: https://bugzilla.redhat.com/1185826 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddDiskCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/AddCinderDiskCommand.java 2 files changed, 5 insertions(+), 4 deletions(-) Approvals: Daniel Erez: Verified; Passed CI tests Maor Lipchuk: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/42700 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I51b27f6104d6aeba8bebdbc13919bf41e511f4d3 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: support quota on remove cinder disk
by derez＠redhat.com 23 Jun '15

23 Jun '15

Daniel Erez has submitted this change and it was merged. Change subject: core: support quota on remove cinder disk ...................................................................... core: support quota on remove cinder disk Supporting quota for remove cinder disk. Change-Id: I1000b8bdee1037730a072a63af12973d341905c7 Bug-Url: https://bugzilla.redhat.com/1185826 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveDiskCommand.java 1 file changed, 2 insertions(+), 3 deletions(-) Approvals: Daniel Erez: Verified; Passed CI tests Maor Lipchuk: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/42699 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1000b8bdee1037730a072a63af12973d341905c7 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine-cli[cli_3.5]: cli: Prepare for next release
by juan.hernandez＠redhat.com 23 Jun '15

23 Jun '15

Juan Hernandez has submitted this change and it was merged. Change subject: cli: Prepare for next release ...................................................................... cli: Prepare for next release In preparation for the next release this patch increases the version number and sets the RPM release number to 0.1. Change-Id: I745c400c3dd94857fa401a078c092b6bed6f2ea9 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M Makefile M setup.py 2 files changed, 3 insertions(+), 3 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/42740 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I745c400c3dd94857fa401a078c092b6bed6f2ea9 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: cli_3.5 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0