August 2014 - Engine-commits - oVirt List Archives

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

Change in ovirt-engine[master]: engine: Add GetAllMacPoolsQuery
by lvernia＠redhat.com 04 Aug '14

04 Aug '14

Lior Vernia has submitted this change and it was merged. Change subject: engine: Add GetAllMacPoolsQuery ...................................................................... engine: Add GetAllMacPoolsQuery Introduce a query to retrieve all mac pools. Change-Id: I972ddd3a9731e34ef8d22fae83cad473d9e48afa Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetAllMacPoolsQuery.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/VdcQueryType.java 2 files changed, 16 insertions(+), 0 deletions(-) Approvals: Martin Mucha: Verified Lior Vernia: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/28984 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I972ddd3a9731e34ef8d22fae83cad473d9e48afa Gerrit-PatchSet: 26 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-ha[master]: 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/30925 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I349d8e6531dfbe063a65b5c0b50db09b92dddede Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master 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 Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

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

04 Aug '14

Oved Ourfali 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> --- 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: Ori Liel: Looks good to me, approved Ravi Nori: Verified Oved Ourfali: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/30813 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib38d9c0853bbc487a13057125df8bbb1e9cc77f0 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in jenkins[master]: enable el7 builds for hosted engine
by dcaroest＠redhat.com 04 Aug '14

04 Aug '14

David Caro has submitted this change and it was merged. Change subject: enable el7 builds for hosted engine ...................................................................... enable el7 builds for hosted engine Related to: https://fedorahosted.org/ovirt/ticket/234 Change-Id: Icb1b2410407837784c04e7e5cdc953c8d57b3ee9 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/yaml/jobs/ovirt-hosted-engine-ha/ovirt-hosted-engine-ha_create-rpms.yaml M jobs/confs/yaml/jobs/ovirt-hosted-engine-setup/ovirt-hosted-engine-setup_create-rpms.yaml 2 files changed, 66 insertions(+), 15 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve David Caro: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/30752 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icb1b2410407837784c04e7e5cdc953c8d57b3ee9 Gerrit-PatchSet: 3 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Ewoud Kohl van Wijngaarden <ewoud(a)kohlvanwijngaarden.nl> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Ohad Basan <obasan(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in jenkins[master]: jobs: ovirt-hosted-engine-setup
by dcaroest＠redhat.com 04 Aug '14

04 Aug '14

David Caro has submitted this change and it was merged. Change subject: jobs: ovirt-hosted-engine-setup ...................................................................... jobs: ovirt-hosted-engine-setup Add ovirt-hosted-engine-setup jobs. 1.1 / 3.4 targeted job requires otopi-devtools to be installed while running configure. Moved tarball and src.rpm generation inside mock for providing required dependencies. Change-Id: I10a908a2be067d38616c41a2c2e6cb3effc9c787 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- A jobs/confs/shell-scripts/build_mock_srcrpm.sh M jobs/confs/shell-scripts/build_mock_srcrpm_releng.sh M jobs/confs/yaml/builders/rpm_mock.yaml A jobs/confs/yaml/jobs/ovirt-hosted-engine-setup/ovirt-hosted-engine-setup_create-rpms.yaml A jobs/confs/yaml/scms/ovirt-hosted-engine-setup.yaml M jobs/confs/yaml/templates/create-rpms_mock.yaml 6 files changed, 236 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve David Caro: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/30510 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I10a908a2be067d38616c41a2c2e6cb3effc9c787 Gerrit-PatchSet: 11 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Ewoud Kohl van Wijngaarden <ewoud(a)kohlvanwijngaarden.nl> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Ohad Basan <obasan(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0