January 2017 - Engine-commits - oVirt List Archives

Change in ovirt-engine-sdk-ruby[sdk_4.0]: [maven-release-plugin] prepare release 4.0.7
by Code Review 12 Jan '17

12 Jan '17

>From Juan Hernandez <juan.hernandez(a)redhat.com>: Juan Hernandez has submitted this change and it was merged. Change subject: [maven-release-plugin] prepare release 4.0.7 ...................................................................... [maven-release-plugin] prepare release 4.0.7 Change-Id: I28b1f56c872fb0da9e4a14144ef2056fb0c38971 --- M generator/pom.xml M pom.xml M sdk/pom.xml 3 files changed, 4 insertions(+), 4 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/70065 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I28b1f56c872fb0da9e4a14144ef2056fb0c38971 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[sdk_4.0]: Update CHANGES.adoc for release 4.0.7
by Code Review 12 Jan '17

12 Jan '17

>From Juan Hernandez <juan.hernandez(a)redhat.com>: Juan Hernandez has submitted this change and it was merged. Change subject: Update CHANGES.adoc for release 4.0.7 ...................................................................... Update CHANGES.adoc for release 4.0.7 This patch updates the CHANGES.adoc file, documenting the relevant changes in release 4.0.7. Change-Id: I87120ae8d56fb794f5afccae1294f987e38155a1 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M sdk/CHANGES.adoc 1 file changed, 16 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/70064 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I87120ae8d56fb794f5afccae1294f987e38155a1 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[sdk_4.0]: Enable installation in Vagrant with embedded Ruby
by Code Review 12 Jan '17

12 Jan '17

>From Juan Hernandez <juan.hernandez(a)redhat.com>: Juan Hernandez has submitted this change and it was merged. Change subject: Enable installation in Vagrant with embedded Ruby ...................................................................... Enable installation in Vagrant with embedded Ruby Currently when trying to install the SDK within as a Vagrant plugin the build fails in some versions of Vagrant because the directories of the embedded 'libxml2' and 'libcurl' libraries are incorrect. To avoid that issue this patch changes the build scripts of the SDK so that they explicitly fix those directories. Change-Id: I8743b3b89161618e18e954d108f30aa649ba3edd Bug-Url: https://bugzilla.redhat.com/1411594 Related-To: https://github.com/mitchellh/vagrant/issues/8184 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit 338a7b246ebc7bfeb0cdcfc34d1870e2ea1ff877) (cherry picked from commit c792adc64011f0a4e575b23ca49322f0f7ff9f71) --- M sdk/ext/ovirtsdk4c/extconf.rb 1 file changed, 10 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/70063 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8743b3b89161618e18e954d108f30aa649ba3edd 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: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-ruby[sdk_4.1]: Enable installation in Vagrant with embedded Ruby
by Code Review 12 Jan '17

12 Jan '17

>From Juan Hernandez <juan.hernandez(a)redhat.com>: Juan Hernandez has submitted this change and it was merged. Change subject: Enable installation in Vagrant with embedded Ruby ...................................................................... Enable installation in Vagrant with embedded Ruby Currently when trying to install the SDK within as a Vagrant plugin the build fails in some versions of Vagrant because the directories of the embedded 'libxml2' and 'libcurl' libraries are incorrect. To avoid that issue this patch changes the build scripts of the SDK so that they explicitly fix those directories. Change-Id: I8743b3b89161618e18e954d108f30aa649ba3edd Bug-Url: https://bugzilla.redhat.com/1411594 Related-To: https://github.com/mitchellh/vagrant/issues/8184 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit 338a7b246ebc7bfeb0cdcfc34d1870e2ea1ff877) --- M sdk/ext/ovirtsdk4c/extconf.rb 1 file changed, 10 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/70062 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8743b3b89161618e18e954d108f30aa649ba3edd Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: sdk_4.1 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> 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]: Enable installation in Vagrant with embedded Ruby
by Code Review 12 Jan '17

12 Jan '17

>From Juan Hernandez <juan.hernandez(a)redhat.com>: Juan Hernandez has submitted this change and it was merged. Change subject: Enable installation in Vagrant with embedded Ruby ...................................................................... Enable installation in Vagrant with embedded Ruby Currently when trying to install the SDK within as a Vagrant plugin the build fails in some versions of Vagrant because the directories of the embedded 'libxml2' and 'libcurl' libraries are incorrect. To avoid that issue this patch changes the build scripts of the SDK so that they explicitly fix those directories. Change-Id: I8743b3b89161618e18e954d108f30aa649ba3edd Bug-Url: https://bugzilla.redhat.com/1411594 Related-To: https://github.com/mitchellh/vagrant/issues/8184 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M sdk/ext/ovirtsdk4c/extconf.rb 1 file changed, 10 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/69992 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8743b3b89161618e18e954d108f30aa649ba3edd Gerrit-PatchSet: 2 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 otopi[master]: packaging: drop date and git hash from releases
by Code Review 12 Jan '17

12 Jan '17

>From Yedidyah Bar David <didi(a)redhat.com>: Yedidyah Bar David has submitted this change and it was merged. Change subject: packaging: drop date and git hash from releases ...................................................................... packaging: drop date and git hash from releases When building with an empty VERSION_SUFFIX, which should mean this is a release, build without date and git hash in the rpms file names. Change-Id: Ifd81e59b34bb48888ac4a92af0aa08a1d652860c Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M automation/build-artifacts.sh M automation/check-patch.sh A automation/config.sh.in M configure.ac 4 files changed, 15 insertions(+), 6 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Yedidyah Bar David: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/58420 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifd81e59b34bb48888ac4a92af0aa08a1d652860c Gerrit-PatchSet: 7 Gerrit-Project: otopi Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Rafael Martins <rmartins(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[ovirt-engine-4.1]: webadmin: Show popup upon import of images from external pro...
by Code Review 11 Jan '17

11 Jan '17

>From Tal Nisan <tnisan(a)redhat.com>: Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Show popup upon import of images from external provider ...................................................................... webadmin: Show popup upon import of images from external provider Show a popup notifying on the start of the import and the status in the footer similar to when importing VMs/templates from an export domain Change-Id: I5a9c6d5fe1ee6f3a2970f4065c6eb52c88a353ff Bug-Url: https://bugzilla.redhat.com/1302562 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/ImportRepoImageModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageIsoListModel.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIMessages.java M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/UIMessages.properties 4 files changed, 34 insertions(+), 1 deletion(-) Approvals: Tal Nisan: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/70046 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5a9c6d5fe1ee6f3a2970f4065c6eb52c88a353ff Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.1 Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Jenkins CI 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.1]: webadmin: Change use host label in new or manage storage domain
by Code Review 11 Jan '17

11 Jan '17

>From Tal Nisan <tnisan(a)redhat.com>: Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Change use host label in new or manage storage domain ...................................................................... webadmin: Change use host label in new or manage storage domain Change-Id: I95aa9e741276813faca4439dde7c01656349fd50 Bug-Url: https://bugzilla.redhat.com/1306110 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/ApplicationConstants.properties 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Tal Nisan: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/70047 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I95aa9e741276813faca4439dde7c01656349fd50 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.1 Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Unify connect/disconnect storage methods in StorageDom...
by Code Review 11 Jan '17

11 Jan '17

>From Tal Nisan <tnisan(a)redhat.com>: Tal Nisan has submitted this change and it was merged. Change subject: core: Unify connect/disconnect storage methods in StorageDomainCommandBase ...................................................................... core: Unify connect/disconnect storage methods in StorageDomainCommandBase The logic of disconnect and connect storage to hosts in up status is almost identical with the exception of the command type called in the storage helper class and the log printed, having both methods contain the same code makes it harder to maintain as every change is likely to be done in both methods at once. This patch extracts the logic to a single method to prevent the boilerplate code Change-Id: I0f034150ccbcddfff4ef5342789ec3871f6677c7 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/domain/StorageDomainCommandBase.java 1 file changed, 10 insertions(+), 26 deletions(-) Approvals: Tal Nisan: Verified Jenkins CI: Passed CI tests Liron Aravot: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/69873 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0f034150ccbcddfff4ef5342789ec3871f6677c7 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Ala Hino <ahino(a)redhat.com> Gerrit-Reviewer: Benny Zlotnik <bzlotnik(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Idan Shaby <ishaby(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Revert "install python-requests as a dependency"
by Code Review 11 Jan '17

11 Jan '17

>From Barak Korren <bkorren(a)redhat.com>: Barak Korren has submitted this change and it was merged. Change subject: Revert "install python-requests as a dependency" ...................................................................... Revert "install python-requests as a dependency" This reverts commit 2d68327fa8bf3ed64877db468a6628da5b8c7410. Patch not needed Change-Id: I1ac16648b2a7fdb4a3a601f0c850b625a2eb5384 --- M jobs/confs/projects/ovirt-engine-nodejs/ovirt-engine-nodejs.yaml 1 file changed, 0 insertions(+), 2 deletions(-) Approvals: Barak Korren: Verified; Looks good to me, approved; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/70043 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1ac16648b2a7fdb4a3a601f0c850b625a2eb5384 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <david(a)dcaro.es> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Gil Shinar <gshinar(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nadav Goldin <ngoldin(a)redhat.com> Gerrit-Reviewer: Pavel Zhukov <pavel(a)zhukoff.net> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Shlomo Ben David <sbendavi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0