February 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: restapi: Document All-Content header for hosts
by juan.hernandez＠redhat.com 19 Feb '15

19 Feb '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Document All-Content header for hosts ...................................................................... restapi: Document All-Content header for hosts The "All-Content" header is used by the hosts collection and by the host resource, but it isn't documented in the RSDL metadata. As a result it isn't described in the RSDL and it isn't available in the SDKs or the CLI. This patch fixes the RSDL metadata. Change-Id: I880cd2f00a74813defd8366285fefc9c6c1bacf6 Bug-Url: https://bugzilla.redhat.com/1194185 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/37945 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I880cd2f00a74813defd8366285fefc9c6c1bacf6 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master 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

Change in ovirt-engine[master]: core: temp fix for missing dbFacade in Utils
by ofrenkel＠redhat.com 19 Feb '15

19 Feb '15

Omer Frenkel has submitted this change and it was merged. Change subject: core: temp fix for missing dbFacade in Utils ...................................................................... core: temp fix for missing dbFacade in Utils Change-Id: I14e7d23a0112cb6b5261e720d376319d0ee0c902 Signed-off-by: Roy Golan <rgolan(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/Backend.java 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Roy Golan: Verified Omer Frenkel: Looks good to me, approved Liran Zelkha: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/37927 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I14e7d23a0112cb6b5261e720d376319d0ee0c902 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Liran Zelkha <lzelkha(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-engine[master]: core: Add JPA infrastructure to oVirt
by oourfali＠redhat.com 19 Feb '15

19 Feb '15

Oved Ourfali has submitted this change and it was merged. Change subject: core: Add JPA infrastructure to oVirt ...................................................................... core: Add JPA infrastructure to oVirt Adding JPA infrastructure (dependencies, persistence.xml) to oVirt. Future patches will migrate entities to use JPA Change-Id: I2440162230449d0c335a2c2d418fb9ff7e854793 Bug-Url: https://bugzilla.redhat.com/1159621 Signed-off-by: lzelkha(a)redhat.com <lzelkha(a)redhat.com> --- M backend/manager/modules/bll/pom.xml A backend/manager/modules/bll/src/main/resources/META-INF/persistence.xml M backend/manager/modules/common/pom.xml M backend/manager/modules/common/src/main/modules/org/ovirt/engine/core/common/main/module.xml M backend/manager/modules/dal/pom.xml M backend/manager/modules/dal/src/main/modules/org/ovirt/engine/core/dal/main/module.xml A backend/manager/modules/dal/src/test/resources/META-INF/persistence.xml M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/AbstractBackendResourceLoggingTest.java M ear/src/main/application/META-INF/jboss-deployment-structure.xml M pom.xml 10 files changed, 69 insertions(+), 20 deletions(-) Approvals: Oved Ourfali: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/33832 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2440162230449d0c335a2c2d418fb9ff7e854793 Gerrit-PatchSet: 56 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core : Add user id to Command Entities
by yzaslavs＠redhat.com 18 Feb '15

18 Feb '15

Yair Zaslavsky has submitted this change and it was merged. Change subject: core : Add user id to Command Entities ...................................................................... core : Add user id to Command Entities Add user id to command entities table Change-Id: I7557ef5f082f15af6e9a1806af22e2aeb981645f Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/CommandEntity.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/CommandEntityDaoDbFacadeImpl.java M packaging/dbscripts/command_entities_sp.sql A packaging/dbscripts/upgrade/03_06_0890_add_user_id_to_command_entities.sql 5 files changed, 29 insertions(+), 11 deletions(-) Approvals: Ravi Nori: Verified Yair Zaslavsky: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/37247 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7557ef5f082f15af6e9a1806af22e2aeb981645f Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: aaa: AuthzUtils: Unify query size limit handling
by yzaslavs＠redhat.com 18 Feb '15

18 Feb '15

Yair Zaslavsky has submitted this change and it was merged. Change subject: aaa: AuthzUtils: Unify query size limit handling ...................................................................... aaa: AuthzUtils: Unify query size limit handling Unifies query size limit handling in AuthzUtils for principal records and groups Change-Id: I54cc5a164e007bb75cddca586a6c01f7538b6eb4 Topic: AAA Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/modules/aaa/src/main/java/org/ovirt/engine/core/aaa/AuthzUtils.java 1 file changed, 7 insertions(+), 29 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Martin Peřina: Verified Yair Zaslavsky: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/37890 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I54cc5a164e007bb75cddca586a6c01f7538b6eb4 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Upgrade message in general tab is missing
by emesika＠redhat.com 18 Feb '15

18 Feb '15

Eli Mesika has submitted this change and it was merged. Change subject: core: Upgrade message in general tab is missing ...................................................................... core: Upgrade message in general tab is missing Alert should distinguish if the host is already in maintenance or not. If the host is not in maintenance it should display : "A new version is available; an upgrade option will appear once the Host is moved to maintenance mode." If the host is in maintenance it should display : "If you wish to upgrade or reinstall it click here." Change-Id: I5af3adca237609dfa696828caa39919c7e4da82b Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1186196 Signed-off-by: emesika <emesika(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationMessages.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/presenter/tab/host/SubTabHostGeneralPresenter.java 2 files changed, 10 insertions(+), 1 deletion(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/37849 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5af3adca237609dfa696828caa39919c7e4da82b Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine-sdk[master]: codegen, sdk: Adjust format of URL replacements
by juan.hernandez＠redhat.com 18 Feb '15

18 Feb '15

Juan Hernandez has submitted this change and it was merged. Change subject: codegen, sdk: Adjust format of URL replacements ...................................................................... codegen, sdk: Adjust format of URL replacements This patch adjust the format of the code generated for URL replacement in order to reduce the diffs with the next version of the generator. Change-Id: I4ff988deb525464b6bfa2f7852216a22a4f43003 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M generator/src/main/java/org/ovirt/engine/sdk/generator/rsdl/SubCollection.java M generator/src/main/java/org/ovirt/engine/sdk/generator/rsdl/templates/ResourceActionTemplate M generator/src/main/java/org/ovirt/engine/sdk/generator/rsdl/templates/ResourceDeleteTemplate M generator/src/main/java/org/ovirt/engine/sdk/generator/rsdl/templates/ResourceDeleteWithBodyAndParamsTemplate M generator/src/main/java/org/ovirt/engine/sdk/generator/rsdl/templates/ResourceDeleteWithBodyTemplate M generator/src/main/java/org/ovirt/engine/sdk/generator/rsdl/templates/ResourceDeleteWithParamsTemplate M generator/src/main/java/org/ovirt/engine/sdk/generator/rsdl/templates/ResourceUpdateTemplate M generator/src/main/java/org/ovirt/engine/sdk/generator/rsdl/templates/SubResourceCollectionActionTemplate M generator/src/main/java/org/ovirt/engine/sdk/generator/utils/UrlUtils.java M src/ovirtsdk/infrastructure/brokers.py 10 files changed, 2,447 insertions(+), 1,115 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/37861 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4ff988deb525464b6bfa2f7852216a22a4f43003 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master 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

Change in ovirt-engine-sdk[master]: codegen, sdk: Format change to reduce diffs
by juan.hernandez＠redhat.com 18 Feb '15

18 Feb '15

Juan Hernandez has submitted this change and it was merged. Change subject: codegen, sdk: Format change to reduce diffs ...................................................................... codegen, sdk: Format change to reduce diffs This patch changes the format of part of the generated code in order to reduces diffs generated by later patches. There is no change in functionality. Change-Id: I570d87afbc6931eef40a64a31d648aecc073d666 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M generator/src/main/java/org/ovirt/engine/sdk/generator/rsdl/templates/ResourceActionTemplate M src/ovirtsdk/infrastructure/brokers.py 2 files changed, 137 insertions(+), 36 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/37860 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I570d87afbc6931eef40a64a31d648aecc073d666 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master 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

Change in ovirt-engine[ovirt-engine-3.5]: packaging: engine-backup: Require lc_messages = en_US.UTF-8
by piotr.kliczewski＠gmail.com 18 Feb '15

18 Feb '15

Piotr Kliczewski has submitted this change and it was merged. Change subject: packaging: engine-backup: Require lc_messages = en_US.UTF-8 ...................................................................... packaging: engine-backup: Require lc_messages = en_US.UTF-8 Verify on restore that lc_messages is set to en_US.UTF-8, or we will not be able to correctly filter error messages. Change-Id: Ica0c8e4b3bc7fd6b06f762fa0bbe00058f16943d Bug-Url: https://bugzilla.redhat.com/1192954 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> (cherry picked from commit 7a2526baa77db462e36141a9d1ebc22770894614) --- M packaging/bin/engine-backup.sh 1 file changed, 12 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved oVirt Jenkins CI Server: Looks good to me, but someone else must approve Yedidyah Bar David: Verified -- To view, visit http://gerrit.ovirt.org/37812 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ica0c8e4b3bc7fd6b06f762fa0bbe00058f16943d Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: gluster: DAO for volume snapshot scheduling
by emesika＠redhat.com 18 Feb '15

18 Feb '15

Eli Mesika has submitted this change and it was merged. Change subject: gluster: DAO for volume snapshot scheduling ...................................................................... gluster: DAO for volume snapshot scheduling Introduced DAO for gluster volume snapshot scheduling Change-Id: Ie44116df49acc302bfc1b745c00535c1fb0174ad Signed-off-by: Shubhendu Tripathi <shtripat(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/gluster/GlusterVolumeEntity.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/DbFacade.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/gluster/GlusterVolumeDaoDbFacadeImpl.java A backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/gluster/GlusterVolumeSnapshotScheduleDao.java A backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/gluster/GlusterVolumeSnapshotScheduleDaoDBFacadeImpl.java M backend/manager/modules/dal/src/main/jdbc-resources/engine-daos.properties A backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/gluster/GlusterVolumeSnapshotScheduleDaoTest.java M backend/manager/modules/dal/src/test/resources/fixtures.xml A packaging/dbscripts/gluster_volume_snapshot_schedule_sp.sql A packaging/dbscripts/upgrade/03_06_0880_add_snapshot_scheduled_flag_to_gluster_volume.sql 10 files changed, 342 insertions(+), 0 deletions(-) Approvals: Eli Mesika: Looks good to me, approved Shubhendu Tripathi: Verified -- To view, visit http://gerrit.ovirt.org/36486 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie44116df49acc302bfc1b745c00535c1fb0174ad Gerrit-PatchSet: 10 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0