May 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: restapi: Extend VM with provider id
by masayag＠redhat.com 18 May '15

18 May '15

Moti Asayag has submitted this change and it was merged. Change subject: restapi: Extend VM with provider id ...................................................................... restapi: Extend VM with provider id The provider id is a reference to the provider which is used for querying katello errata for the specific VM. In the future, the field will be used also to associate the VM with the server by which it was provisioned (Foreman/Satellite). <vm> ... <external_host_provider id="..." href="..." /> </vm> Change-Id: Ia923483e6ff8de8dc16dde84e61818472f59a944 Bug-Url: https://bugzilla.redhat.com/1157377 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java 3 files changed, 15 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Looks good to me, approved Jenkins CI: Looks good to me, but someone else must approve Moti Asayag: Verified Oved Ourfali: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/40719 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia923483e6ff8de8dc16dde84e61818472f59a944 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: engine: Introduce queries for vms errata
by masayag＠redhat.com 18 May '15

18 May '15

Moti Asayag has submitted this change and it was merged. Change subject: engine: Introduce queries for vms errata ...................................................................... engine: Introduce queries for vms errata Change-Id: Id3b2e149f64eb55d30bb093b5d82c68401edc5a3 Bug-Url: https://bugzilla.redhat.com/1157377 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetErrataForVmQuery.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetErratumByIdForVmQuery.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/VdcQueryType.java 3 files changed, 84 insertions(+), 0 deletions(-) Approvals: Moti Asayag: Verified Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/40720 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id3b2e149f64eb55d30bb093b5d82c68401edc5a3 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master 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: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: engine: Allow to associate a provider with a vm
by masayag＠redhat.com 18 May '15

18 May '15

Moti Asayag has submitted this change and it was merged. Change subject: engine: Allow to associate a provider with a vm ...................................................................... engine: Allow to associate a provider with a vm Change-Id: I9df76ab2a7858758a203eb45b416ae3c82718b88 Bug-Url: https://bugzilla.redhat.com/1157377 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmCommand.java 1 file changed, 19 insertions(+), 0 deletions(-) Approvals: Jenkins CI: Looks good to me, but someone else must approve Moti Asayag: Verified Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/40718 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9df76ab2a7858758a203eb45b416ae3c82718b88 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master 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: automation(a)ovirt.org

1 0

Change in ovirt-hosted-engine-ha[ovirt-hosted-engine-ha-1.2]: respect the timeout in EngineUpBadHealth state (better fix)
by msivak＠redhat.com 18 May '15

18 May '15

Martin Sivák has submitted this change and it was merged. Change subject: respect the timeout in EngineUpBadHealth state (better fix) ...................................................................... respect the timeout in EngineUpBadHealth state (better fix) Change-Id: Id86274fabe5f66ee42fc0235a4091205afedd79c Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1150600 Signed-off-by: Jiri Moskovcak <jmoskovc(a)redhat.com> --- M ovirt_hosted_engine_ha/agent/states.py 1 file changed, 0 insertions(+), 14 deletions(-) Approvals: Martin Sivák: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/40820 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id86274fabe5f66ee42fc0235a4091205afedd79c Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: ovirt-hosted-engine-ha-1.2 Gerrit-Owner: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: restapi: support creation of unattached volume provider
by derez＠redhat.com 18 May '15

18 May '15

Daniel Erez has submitted this change and it was merged. Change subject: restapi: support creation of unattached volume provider ...................................................................... restapi: support creation of unattached volume provider Removing the validation of a data_center element requirement upon volume provider creation (modified BackendOpenStackVolumeProvidersResource and rsdl_metadata.yaml accordingly). Change-Id: I288e9a463150b729d0551ed209171bb7df2652aa Bug-Url: https://bugzilla.redhat.com/1185826 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/provider/storage/OpenStackVolumeProviderProxy.java 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/resource/openstack/BackendOpenStackVolumeProvidersResource.java 3 files changed, 7 insertions(+), 5 deletions(-) Approvals: Juan Hernandez: Looks good to me, approved Jenkins CI: Looks good to me, but someone else must approve Daniel Erez: Verified Maor Lipchuk: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/40663 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I288e9a463150b729d0551ed209171bb7df2652aa Gerrit-PatchSet: 5 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: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: restapi: Adds disperse, redundancy info to glustervolumes res...
by juan.hernandez＠redhat.com 18 May '15

18 May '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Adds disperse,redundancy info to glustervolumes restapi ...................................................................... restapi: Adds disperse,redundancy info to glustervolumes restapi Disperse count, redundancy count informations of disperse volumes are added to glustervolumes restapi. Change-Id: I722b50e81c5118146d3cd4fb9c522a44aae17362 Bug-Url: https://bugzilla.redhat.com/1213379 Signed-off-by: Karnan TC <kchidamb(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/model/GlusterVolumeType.java M backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/GlusterVolumeMapper.java M backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/GlusterVolumeMapperTest.java 4 files changed, 29 insertions(+), 1 deletion(-) Approvals: Karnan t c: Verified Juan Hernandez: Looks good to me, approved Shubhendu Tripathi: Looks good to me, but someone else must approve Jenkins CI: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/40908 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I722b50e81c5118146d3cd4fb9c522a44aae17362 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Karnan t c <kchidamb(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Karnan t c <kchidamb(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mishka8520(a)yahoo.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine-cli[master]: cli: Support escape of double quote in strings
by juan.hernandez＠redhat.com 18 May '15

18 May '15

Juan Hernandez has submitted this change and it was merged. Change subject: cli: Support escape of double quote in strings ...................................................................... cli: Support escape of double quote in strings Currently strings support escape sequences, but they aren't replaced by the corresponding characters. For example, in the following command: # update vm myvv --name "My \"VM\"" The name of the VM will be set to exactly this: My \"VM\" Note that the backslash isn't removed. This makes it impossible to use strings that represent data that contain double quotes, in particular it makes it impossible to use XML documents, which are needed when setting the OVF representation of a VM. This patch changes the CLI so that the escape character preceding the double quote will be removed, thus the above command will result in the following name: My "VM" Change-Id: Ied53ac577efa3395438892de535c35cd068a12bd Bug-Url: https://bugzilla.redhat.com/1211670 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M src/cli/parser.py 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/40690 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ied53ac577efa3395438892de535c35cd068a12bd Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: Avoid using export validation for unregistred VMs
by mlipchuk＠redhat.com 18 May '15

18 May '15

Maor Lipchuk has submitted this change and it was merged. Change subject: core: Avoid using export validation for unregistred VMs ...................................................................... core: Avoid using export validation for unregistred VMs Move validation of ACTION_TYPE_FAILED_VM_NOT_FOUND_ON_EXPORT_DOMAIN to be used for VMs which are not an unregistered entities. Change-Id: I2df389156b156eadf6d4dd1244fa01b449d7d94e Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportVmCommand.java 1 file changed, 7 insertions(+), 9 deletions(-) Approvals: Jenkins CI: Looks good to me, but someone else must approve Maor Lipchuk: Verified Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/40965 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2df389156b156eadf6d4dd1244fa01b449d7d94e Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-hosted-engine-setup[master]: packaging: setup: configure firewall on additional host
by sbonazzo＠redhat.com 18 May '15

18 May '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: configure firewall on additional host ...................................................................... packaging: setup: configure firewall on additional host Bug-Url: https://bugzilla.redhat.com/1221148 Change-Id: I5a3dbd27d1669df6dff927f21eb38301c43f6b50 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M src/plugins/ovirt-hosted-engine-setup/network/firewall_manager.py 1 file changed, 18 insertions(+), 7 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Looks good to me, but someone else must approve Jenkins CI: Looks good to me, but someone else must approve Yedidyah Bar David: Verified -- To view, visit https://gerrit.ovirt.org/40914 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5a3dbd27d1669df6dff927f21eb38301c43f6b50 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5-gluster]: gluster: Remove volume snapshot schedule if job ended
by kmayilsa＠redhat.com 18 May '15

18 May '15

Kanagaraj M has submitted this change and it was merged. Change subject: gluster: Remove volume snapshot schedule if job ended ...................................................................... gluster: Remove volume snapshot schedule if job ended Added logic to remove the gluster volume snapshot schedule details if the schedule has ended. Change-Id: Ibbdb11f0d881339749dac8101ca9a01b04a334c6 Bug-Url: https://bugzilla.redhat.com/1219827 Signed-off-by: Shubhendu Tripathi <shtripat(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterSnapshotScheduleJob.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/RescheduleGlusterVolumeSnapshotCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/ScheduleGlusterVolumeSnapshotCommandBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/utils/GlusterUtil.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/LocalizedEnums.java M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/LocalizedEnums.properties 8 files changed, 74 insertions(+), 23 deletions(-) Approvals: Shubhendu Tripathi: Verified Kanagaraj M: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/41043 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibbdb11f0d881339749dac8101ca9a01b04a334c6 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5-gluster Gerrit-Owner: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(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

1 0