Change in ovirt-engine[master]: restapi: rest support for MacPool
by lvernia@redhat.com
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
10 years, 4 months
Change in ovirt-engine[ovirt-engine-3.5]: restapi : Bad error message - no link
by tnisan@redhat.com
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
10 years, 4 months
Change in ovirt-engine[master]: webadmin: I18N - fix misalignment in import domain
by derez@redhat.com
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
10 years, 4 months
Change in ovirt-engine[master]: core: pass MaxNumberOfSockets to create verb
by rgolan@redhat.com
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
10 years, 4 months
Change in ovirt-hosted-engine-ha[ovirt-hosted-engine-ha-1.2]: use /usr/lib/instead of /lib
by sbonazzo@redhat.com
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
10 years, 4 months
Change in ovirt-engine[master]: engine: Add GetAllMacPoolsQuery
by lvernia@redhat.com
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
10 years, 4 months
Change in ovirt-hosted-engine-ha[master]: use /usr/lib/instead of /lib
by sbonazzo@redhat.com
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
10 years, 4 months
Change in ovirt-engine[master]: restapi : Bad error message - no link
by oourfali@redhat.com
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
10 years, 4 months
Change in jenkins[master]: enable el7 builds for hosted engine
by dcaroest@redhat.com
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
10 years, 4 months
Change in jenkins[master]: jobs: ovirt-hosted-engine-setup
by dcaroest@redhat.com
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
10 years, 4 months