September 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: restapi: Rename complex type "MAC" to "Mac"
by juan.hernandez＠redhat.com 24 Sep '15

24 Sep '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Rename complex type "MAC" to "Mac" ...................................................................... restapi: Rename complex type "MAC" to "Mac" This is required to make sure that there is only one rule for the names of complex types: all words, including acronyms, start with one upper case character and all the rest are lower case. Change-Id: I6502e7395eff19b97f5da6c68a957177be3f1e6d Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/README.md M backend/manager/modules/restapi/interface/definition/src/main/schema/api.xsd M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendVmNicResourceTest.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/HostNicMapper.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/NicMapper.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/ReportedDeviceMapper.java 6 files changed, 11 insertions(+), 10 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/46120 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6502e7395eff19b97f5da6c68a957177be3f1e6d Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-reports[ovirt-engine-reports-3.5]: build: post ovirt-engine-reports-3.5.5
by ydary＠redhat.com 24 Sep '15

24 Sep '15

Yaniv Dary has submitted this change and it was merged. Change subject: build: post ovirt-engine-reports-3.5.5 ...................................................................... build: post ovirt-engine-reports-3.5.5 Change-Id: I2c6a29e5bd6e2d708b8496406241bd669202b04d Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M version.mak 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Jenkins CI: Passed CI tests Yaniv Dary: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/46655 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2c6a29e5bd6e2d708b8496406241bd669202b04d Gerrit-PatchSet: 1 Gerrit-Project: ovirt-reports Gerrit-Branch: ovirt-engine-reports-3.5 Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-reports[ovirt-engine-reports-3.5]: build: ovirt-engine-reports-3.5.5
by ydary＠redhat.com 24 Sep '15

24 Sep '15

Yaniv Dary has submitted this change and it was merged. Change subject: build: ovirt-engine-reports-3.5.5 ...................................................................... build: ovirt-engine-reports-3.5.5 Change-Id: Ibb8c18a45bcfca580a2014bdb6132ba929288e6f Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M ovirt-engine-reports.spec.in M version.mak 2 files changed, 8 insertions(+), 3 deletions(-) Approvals: Jenkins CI: Passed CI tests Yaniv Dary: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/46654 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibb8c18a45bcfca580a2014bdb6132ba929288e6f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-reports Gerrit-Branch: ovirt-engine-reports-3.5 Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-scheduler-proxy[master]: automation: add standard build automation
by msivak＠redhat.com 24 Sep '15

24 Sep '15

Martin Sivák has submitted this change and it was merged. Change subject: automation: add standard build automation ...................................................................... automation: add standard build automation Change-Id: I8bd20509100fe9764cb951699b2273bcb0018c59 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- A automation/build-artifacts.packages A automation/build-artifacts.repos A automation/build-artifacts.sh A automation/check-patch.packages A automation/check-patch.repos A automation/check-patch.sh 6 files changed, 47 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified Martin Sivák: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/46631 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8bd20509100fe9764cb951699b2273bcb0018c59 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-scheduler-proxy Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(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: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.6]: webadmin: Hotfix of icon validation for IE
by piotr.kliczewski＠gmail.com 24 Sep '15

24 Sep '15

Piotr Kliczewski has submitted this change and it was merged. Change subject: webadmin: Hotfix of icon validation for IE ...................................................................... webadmin: Hotfix of icon validation for IE It disables part of client side icon validation on IE browsers. The disabled part relies on parsing of images using <img> tag. IE parses images asynchronously as opposed to Chrome or Firefox (after caching). Current frontend validation assumes synchronous behavior. There are still backend icon validations. Change-Id: I2766e993c3e03b5a040a178e26b8b71c93ac585c Bug-Url: https://bugzilla.redhat.com/1263994 Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/validation/IconValidation.java 1 file changed, 8 insertions(+), 1 deletion(-) Approvals: Piotr Kliczewski: Passed CI tests Jakub Niedermertl: Verified Vojtech Szocs: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/46534 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2766e993c3e03b5a040a178e26b8b71c93ac585c Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.6]: core: Permission based icon filtering: backend part
by piotr.kliczewski＠gmail.com 24 Sep '15

24 Sep '15

Piotr Kliczewski has submitted this change and it was merged. Change subject: core: Permission based icon filtering: backend part ...................................................................... core: Permission based icon filtering: backend part I allows to non-admin user to list icons using REST. Returned icons are union of predefined icons and icons that are associated with some entity the user has permission to (vm or tempate). Change-Id: I09aecac8c8265cf687ecf4edc2305421a2feb61b Bug-Url: https://bugzilla.redhat.com/1263758 Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetAllVmIconsQuery.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/VdcQueryType.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmIconDao.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmIconDaoImpl.java 4 files changed, 29 insertions(+), 7 deletions(-) Approvals: Piotr Kliczewski: Passed CI tests Jakub Niedermertl: Verified Omer Frenkel: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/46475 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I09aecac8c8265cf687ecf4edc2305421a2feb61b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.6]: core: Permission based icon filtering: database part
by piotr.kliczewski＠gmail.com 24 Sep '15

24 Sep '15

Piotr Kliczewski has submitted this change and it was merged. Change subject: core: Permission based icon filtering: database part ...................................................................... core: Permission based icon filtering: database part I allows to non-admin user to list icons using REST. Returned icons are union of predefined icons and icons that are associated with some entity the user has permission to (vm or tempate). Change-Id: I53d8cf17f91b67ae53ab90d7e929f0cf2a9810fb Bug-Url: https://bugzilla.redhat.com/1263758 Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com> --- M packaging/dbscripts/vm_icons_sp.sql 1 file changed, 35 insertions(+), 0 deletions(-) Approvals: Jakub Niedermertl: Verified Eli Mesika: Looks good to me, approved Jenkins CI: Passed CI tests Omer Frenkel: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/46474 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I53d8cf17f91b67ae53ab90d7e929f0cf2a9810fb Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-dwh[ovirt-engine-dwh-3.5]: build: post ovirt-engine-dwh-3.5.5
by ydary＠redhat.com 24 Sep '15

24 Sep '15

Yaniv Dary has submitted this change and it was merged. Change subject: build: post ovirt-engine-dwh-3.5.5 ...................................................................... build: post ovirt-engine-dwh-3.5.5 Change-Id: I1e7c89f25daa77025f822dbd1b43a9051522b49a Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M version.mak 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Jenkins CI: Passed CI tests Yaniv Dary: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/46653 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1e7c89f25daa77025f822dbd1b43a9051522b49a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-dwh Gerrit-Branch: ovirt-engine-dwh-3.5 Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-dwh[ovirt-engine-dwh-3.5]: build: ovirt-engine-dwh-3.5.5
by ydary＠redhat.com 24 Sep '15

24 Sep '15

Yaniv Dary has submitted this change and it was merged. Change subject: build: ovirt-engine-dwh-3.5.5 ...................................................................... build: ovirt-engine-dwh-3.5.5 Added released notes. Change-Id: I7e65d2b9ae25c2432a1254b4e5deb3c09a6193a0 Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M ovirt-engine-dwh.spec.in M version.mak 2 files changed, 6 insertions(+), 2 deletions(-) Approvals: Jenkins CI: Passed CI tests Yaniv Dary: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/46652 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7e65d2b9ae25c2432a1254b4e5deb3c09a6193a0 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-dwh Gerrit-Branch: ovirt-engine-dwh-3.5 Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.6]: core: Remove duplicated entry from AuditLogMessages
by tnisan＠redhat.com 24 Sep '15

24 Sep '15

Tal Nisan has submitted this change and it was merged. Change subject: core: Remove duplicated entry from AuditLogMessages ...................................................................... core: Remove duplicated entry from AuditLogMessages Change-Id: Ia3a01712432b112a17293baf488e6a7add34099b Bug-Url: https://bugzilla.redhat.com/1238594 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties 1 file changed, 0 insertions(+), 1 deletion(-) Approvals: Moti Asayag: Verified; Passed CI tests Oved Ourfali: Looks good to me, approved Objections: Jenkins CI: Failed CI tests -- To view, visit https://gerrit.ovirt.org/46645 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia3a01712432b112a17293baf488e6a7add34099b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0