September 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-4.0.4]: packaging: setup: Handle the case when HE option is missing
by tnisan＠redhat.com 13 Sep '16

13 Sep '16

Tal Nisan has submitted this change and it was merged. Change subject: packaging: setup: Handle the case when HE option is missing ...................................................................... packaging: setup: Handle the case when HE option is missing This happens normally only in case of an upgrade from 3.5 Change-Id: I81319992643ed6763690ffc620577ff5e44a3c38 Bug-Url: https://bugzilla.redhat.com/1359844 Signed-off-by: Lev Veyde <lveyde(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-common/ovirt-engine/system/he.py 1 file changed, 11 insertions(+), 6 deletions(-) Approvals: Lev Veyde: Verified Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/63741 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I81319992643ed6763690ffc620577ff5e44a3c38 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0.4 Gerrit-Owner: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6.9]: packaging: setup: Handle the case when HE option is missing
by tnisan＠redhat.com 13 Sep '16

13 Sep '16

Tal Nisan has submitted this change and it was merged. Change subject: packaging: setup: Handle the case when HE option is missing ...................................................................... packaging: setup: Handle the case when HE option is missing This happens normally only in case of an upgrade from 3.5 Change-Id: I81319992643ed6763690ffc620577ff5e44a3c38 Bug-Url: https://bugzilla.redhat.com/1359844 Signed-off-by: Lev Veyde <lveyde(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-common/ovirt-engine/system/he.py 1 file changed, 11 insertions(+), 6 deletions(-) Approvals: Lev Veyde: Verified Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/63740 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I81319992643ed6763690ffc620577ff5e44a3c38 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6.9 Gerrit-Owner: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: core: Validate images existance for unregistered Template.
by tnisan＠redhat.com 13 Sep '16

13 Sep '16

Tal Nisan has submitted this change and it was merged. Change subject: core: Validate images existance for unregistered Template. ...................................................................... core: Validate images existance for unregistered Template. Once an unregister Template entity is being registered to the engine, the engine should verify all its disks existance in the Storage. The reason the engine should perform this validation is since the Template's OVF might not be synced with the disks in the Storage. If one of the disks doesn't exists, the engine should block the operation. This behavior should be compatible with the behavior being performed today once a Template is being imported from an export domain. Change-Id: I3b890bd831c34770d6fd43416d5b15888b45f85b Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/1367488 --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/exportimport/ImportVmTemplateFromConfigurationCommand.java 1 file changed, 5 insertions(+), 0 deletions(-) Approvals: Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, approved Maor Lipchuk: Verified -- To view, visit https://gerrit.ovirt.org/63720 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3b890bd831c34770d6fd43416d5b15888b45f85b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: packaging: setup: Handle the case when HE option is missing
by tnisan＠redhat.com 13 Sep '16

13 Sep '16

Tal Nisan has submitted this change and it was merged. Change subject: packaging: setup: Handle the case when HE option is missing ...................................................................... packaging: setup: Handle the case when HE option is missing This happens normally only in case of an upgrade from 3.5 Change-Id: I81319992643ed6763690ffc620577ff5e44a3c38 Bug-Url: https://bugzilla.redhat.com/1359844 Signed-off-by: Lev Veyde <lveyde(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-common/ovirt-engine/system/he.py 1 file changed, 11 insertions(+), 6 deletions(-) Approvals: Lev Veyde: Verified Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests Yedidyah Bar David: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/63725 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I81319992643ed6763690ffc620577ff5e44a3c38 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: packaging: setup: Handle the case when HE option is missing
by tnisan＠redhat.com 13 Sep '16

13 Sep '16

Tal Nisan has submitted this change and it was merged. Change subject: packaging: setup: Handle the case when HE option is missing ...................................................................... packaging: setup: Handle the case when HE option is missing This happens normally only in case of an upgrade from 3.5 Change-Id: I81319992643ed6763690ffc620577ff5e44a3c38 Bug-Url: https://bugzilla.redhat.com/1359844 Signed-off-by: Lev Veyde <lveyde(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-common/ovirt-engine/system/he.py 1 file changed, 11 insertions(+), 6 deletions(-) Approvals: Lev Veyde: Verified Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/63737 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I81319992643ed6763690ffc620577ff5e44a3c38 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-ruby[sdk_4.0]: Use XML schema naming exceptions
by juan.hernandez＠redhat.com 13 Sep '16

13 Sep '16

Juan Hernandez has submitted this change and it was merged. Change subject: Use XML schema naming exceptions ...................................................................... Use XML schema naming exceptions Currently the SDK uses incorrect XML tag names for some types, in particular for the 'OpenStackImageProvider' type. For historical reasons the tag name corresponding to this type is 'openstack_image_provider', without a dash between the 'open' and 'stack' words. These naming exceptions are managed by the 'SchemaNames' class of the metamodel, but we aren't using it correctly. This patch fixes that error. Change-Id: Iedc3d7e7bd542ef3d304530588958d591675d9e0 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit 85fa34b9578b65a15cc1f21a4f0a074819254d9d) --- M generator/src/main/java/org/ovirt/sdk/ruby/WritersGenerator.java A sdk/spec/openstack_image_provider_writer_spec.rb 2 files changed, 55 insertions(+), 4 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/63739 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iedc3d7e7bd542ef3d304530588958d591675d9e0 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: sdk_4.0 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-ruby[master]: Add example of how to add OpenStack image provider
by juan.hernandez＠redhat.com 13 Sep '16

13 Sep '16

Juan Hernandez has submitted this change and it was merged. Change subject: Add example of how to add OpenStack image provider ...................................................................... Add example of how to add OpenStack image provider This patch adds an example that shows how to add an OpenStack image provider. Change-Id: I31af8e874d7a25da22d57d089bbdabf793bd3a1a Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- A sdk/examples/add_openstack_image_provider.py 1 file changed, 53 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/63735 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I31af8e874d7a25da22d57d089bbdabf793bd3a1a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-ruby 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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-ruby[master]: Use XML schema naming exceptions
by juan.hernandez＠redhat.com 13 Sep '16

13 Sep '16

Juan Hernandez has submitted this change and it was merged. Change subject: Use XML schema naming exceptions ...................................................................... Use XML schema naming exceptions Currently the SDK uses incorrect XML tag names for some types, in particular for the 'OpenStackImageProvider' type. For historical reasons the tag name corresponding to this type is 'openstack_image_provider', without a dash between the 'open' and 'stack' words. These naming exceptions are managed by the 'SchemaNames' class of the metamodel, but we aren't using it correctly. This patch fixes that error. Change-Id: Iedc3d7e7bd542ef3d304530588958d591675d9e0 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M generator/src/main/java/org/ovirt/sdk/ruby/WritersGenerator.java A sdk/spec/openstack_image_provider_writer_spec.rb 2 files changed, 55 insertions(+), 4 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests Objections: Jenkins CI: Failed CI tests -- To view, visit https://gerrit.ovirt.org/63734 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iedc3d7e7bd542ef3d304530588958d591675d9e0 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-ruby 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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: packaging: setup: Handle the case when HE option is missing
by sbonazzo＠redhat.com 13 Sep '16

13 Sep '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: Handle the case when HE option is missing ...................................................................... packaging: setup: Handle the case when HE option is missing This happens normally only in case of an upgrade from 3.5 Change-Id: I81319992643ed6763690ffc620577ff5e44a3c38 Bug-Url: https://bugzilla.redhat.com/1359844 Signed-off-by: Lev Veyde <lveyde(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-common/ovirt-engine/system/he.py 1 file changed, 11 insertions(+), 6 deletions(-) Approvals: Lev Veyde: Verified Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Yedidyah Bar David: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/63724 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I81319992643ed6763690ffc620577ff5e44a3c38 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> 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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Rephrase job description for register disk.
by mlipchuk＠redhat.com 13 Sep '16

13 Sep '16

Maor Lipchuk has submitted this change and it was merged. Change subject: core: Rephrase job description for register disk. ...................................................................... core: Rephrase job description for register disk. Register disk's job description should be compatible to the description which is being used for registering unregistered VMs and Templates Change-Id: I466fa338308acb4b57779153ddf50e26de9f6e65 Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com> --- M backend/manager/modules/dal/src/main/resources/bundles/ExecutionMessages.properties 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, approved Maor Lipchuk: Verified -- To view, visit https://gerrit.ovirt.org/62407 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I466fa338308acb4b57779153ddf50e26de9f6e65 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0