- Engine-commits - oVirt List Archives

Change in ovirt-hosted-engine-ha[master]: packaging: spec: fix packaging issue
by sbonazzo＠redhat.com 04 Aug '14

04 Aug '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: spec: fix packaging issue ...................................................................... packaging: spec: fix packaging issue Fixes: error: File not found: /lib/systemd/systemd-ovirt-ha-agent error: File not found: /lib/systemd/systemd-ovirt-ha-broker Change-Id: I28ac68184f1f6519a4dddcd6c40a6f0285aa98b9 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-hosted-engine-ha.spec.in 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/31031 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I28ac68184f1f6519a4dddcd6c40a6f0285aa98b9 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-release[master]: packaging: spec: fix EL7 detection
by sbonazzo＠redhat.com 04 Aug '14

04 Aug '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: spec: fix EL7 detection ...................................................................... packaging: spec: fix EL7 detection Change-Id: I04146665776c8ddb223cf42b85f1de9f7e36d320 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-release-3.5/ovirt-release35.spec.in M ovirt-release-master/ovirt-release-master.spec.in 2 files changed, 4 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/31030 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I04146665776c8ddb223cf42b85f1de9f7e36d320 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5]: webadmin: I18N - fix misalignment in import domain
by tnisan＠redhat.com 04 Aug '14

04 Aug '14

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: I18N - fix misalignment in import domain ...................................................................... webadmin: I18N - fix misalignment in import domain Modify style of 'Activate Domain in Data Center' check-box in import storage domain dialog (to fix misalignment in French localization). Change-Id: Ic489c119d4cf5fa4cc8f169d76442a0d8143ff53 Bug-Url: https://bugzilla.redhat.com/1123701 Bug-Url: https://bugzilla.redhat.com/1123702 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/StoragePopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/StoragePopupView.ui.xml 2 files changed, 7 insertions(+), 0 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Daniel Erez: Verified -- To view, visit http://gerrit.ovirt.org/31027 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic489c119d4cf5fa4cc8f169d76442a0d8143ff53 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: frontend: Sorting columns in VM subtabs
by lvernia＠redhat.com 04 Aug '14

04 Aug '14

Lior Vernia has submitted this change and it was merged. Change subject: frontend: Sorting columns in VM subtabs ...................................................................... frontend: Sorting columns in VM subtabs This patch contains sorting columns in "Network Interface" VM subtab. Change-Id: I6b55d40f4978f994cc2c84f3a99d46294b7928fc Signed-off-by: Frantisek Kobzik <fkobzik(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/vm/VmInterfaceListModelTable.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/virtualMachine/SubTabVirtualMachineNetworkInterfaceView.java 2 files changed, 12 insertions(+), 3 deletions(-) Approvals: Frank Kobzik: Verified Lior Vernia: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/28360 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6b55d40f4978f994cc2c84f3a99d46294b7928fc Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Frank Kobzik <fkobzik(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Frank Kobzik <fkobzik(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: coverity issue fix.
by lvernia＠redhat.com 04 Aug '14

04 Aug '14

Lior Vernia has submitted this change and it was merged. Change subject: core: coverity issue fix. ...................................................................... core: coverity issue fix. - inlined variable, which can be null from static analysis point of view, but such scenario would be a bug in system. Inlining variable should convince coverity not to consider this as a problem. Change-Id: I6751881c8b72e400f898ddd631b34106742db3b8 Signed-off-by: Martin Mucha <mmucha(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/macpoolmanager/MacsStorage.java 1 file changed, 1 insertion(+), 2 deletions(-) Approvals: Martin Mucha: Verified Moti Asayag: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29155 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6751881c8b72e400f898ddd631b34106742db3b8 Gerrit-PatchSet: 27 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: restapi: rest support for MacPool
by lvernia＠redhat.com 04 Aug '14

04 Aug '14

Lior Vernia has submitted this change and it was merged. Change subject: restapi: rest support for MacPool ...................................................................... restapi: rest support for MacPool Change-Id: Id343433198dadd209933b69012303c51acb4c1e1 Bug-Url: https://bugzilla.redhat.com/1078844 Signed-off-by: Moti Asayag <masayag(a)redhat.com> Signed-off-by: Martin Mucha <mmucha(a)redhat.com> --- A backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/MacPoolResource.java A backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/MacPoolsResource.java M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/utils/ApiRootLinksCreator.java M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/utils/LinkHelper.java M backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd M backend/manager/modules/restapi/interface/definition/src/main/resources/jyaml.yml M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/BackendApplication.java A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendMacPoolResource.java A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendMacPoolsResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/utils/FeaturesHelper.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendApiResourceTest.java A backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendMacPoolResourceTest.java A backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendMacPoolsResourceTest.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/DataCenterMapper.java A backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/MacPoolMapper.java M backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/DataCenterMapperTest.java A backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/MacPoolMapperTest.java 18 files changed, 917 insertions(+), 5 deletions(-) Approvals: Lior Vernia: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/28706 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id343433198dadd209933b69012303c51acb4c1e1 Gerrit-PatchSet: 41 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.5]: restapi : Bad error message - no link
by tnisan＠redhat.com 04 Aug '14

04 Aug '14

Tal Nisan has submitted this change and it was merged. Change subject: restapi : Bad error message - no link ...................................................................... restapi : Bad error message - no link Incorrect xml posted to restapi results in a cryptic error message stating Request syntactically incorrect. See the link description below for the correct usage: The link refered to is an internal DetailedLink entity and should not be shown to the user. This patch changes the message dropping the reference to link Request syntactically incorrect. See the description below for the correct usage: Change-Id: Ib38d9c0853bbc487a13057125df8bbb1e9cc77f0 Bug-Url: https://bugzilla.redhat.com/1122591 Signed-off-by: Ravi Nori <rnori(a)redhat.com> (cherry picked from commit 1e67fd185825cf2af0abc9ff46528654756acc79) --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/validation/UsageFinder.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Oved Ourfali: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/31023 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib38d9c0853bbc487a13057125df8bbb1e9cc77f0 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: I18N - fix misalignment in import domain
by derez＠redhat.com 04 Aug '14

04 Aug '14

Daniel Erez has submitted this change and it was merged. Change subject: webadmin: I18N - fix misalignment in import domain ...................................................................... webadmin: I18N - fix misalignment in import domain Modify style of 'Activate Domain in Data Center' check-box in import storage domain dialog (to fix misalignment in French localization). Change-Id: Ic489c119d4cf5fa4cc8f169d76442a0d8143ff53 Bug-Url: https://bugzilla.redhat.com/1123701 Bug-Url: https://bugzilla.redhat.com/1123702 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/StoragePopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/StoragePopupView.ui.xml 2 files changed, 7 insertions(+), 0 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Daniel Erez: Verified -- To view, visit http://gerrit.ovirt.org/30976 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic489c119d4cf5fa4cc8f169d76442a0d8143ff53 Gerrit-PatchSet: 1 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: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: pass MaxNumberOfSockets to create verb
by rgolan＠redhat.com 04 Aug '14

04 Aug '14

Roy Golan has submitted this change and it was merged. Change subject: core: pass MaxNumberOfSockets to create verb ...................................................................... core: pass MaxNumberOfSockets to create verb in order to generate a correct topology xml-node for libvirt VDSM needs also the MaxNumberOfSockets Change-Id: I68368fdd40f0e43df18465c15faf2626789ee1c7 Bug-Url: https://bugzilla.redhat.com/1070890 Signed-off-by: Roy Golan <rgolan(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsProperties.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VmInfoBuilderBase.java 2 files changed, 17 insertions(+), 8 deletions(-) Approvals: Roy Golan: Verified Omer Frenkel: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/30877 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I68368fdd40f0e43df18465c15faf2626789ee1c7 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-ha[ovirt-hosted-engine-ha-1.2]: use /usr/lib/instead of /lib
by sbonazzo＠redhat.com 04 Aug '14

04 Aug '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: use /usr/lib/instead of /lib ...................................................................... use /usr/lib/instead of /lib Change-Id: I349d8e6531dfbe063a65b5c0b50db09b92dddede Bu-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1125909 Signed-off-by: Jiri Moskovcak <jmoskovc(a)redhat.com> --- M initscripts/ovirt-ha-agent.service M initscripts/ovirt-ha-broker.service M ovirt-hosted-engine-ha.spec.in 3 files changed, 6 insertions(+), 6 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Jiří Moskovčák: Verified -- To view, visit http://gerrit.ovirt.org/31025 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I349d8e6531dfbe063a65b5c0b50db09b92dddede Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: ovirt-hosted-engine-ha-1.2 Gerrit-Owner: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0