- Engine-commits - oVirt List Archives

Change in ovirt-engine-sdk-ruby[master]: Add the method to write integers
by juan.hernandez＠redhat.com 15 Jan '16

15 Jan '16

Juan Hernandez has submitted this change and it was merged. Change subject: Add the method to write integers ...................................................................... Add the method to write integers This patch adds the "XmlWriter.write_integer" method and a couple of simple unit tests, just to verify that it exists. Change-Id: I64a8e8d8721708a3655b3f5c3cef6b7836720378 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M sdk/lib/ovirt/sdk/v4/xml_writer.rb M sdk/spec/xml_writer_spec.rb 2 files changed, 35 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/51901 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I64a8e8d8721708a3655b3f5c3cef6b7836720378 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 jenkins[master]: Hiding spurious misleading message
by dcaro＠redhat.com 15 Jan '16

15 Jan '16

David Caro has submitted this change and it was merged. Change subject: Hiding spurious misleading message ...................................................................... Hiding spurious misleading message We don't really care what's the output of the touch, but usually adds an ugly error message that's not meaningful Change-Id: I1938c2ed31c8bc6ada47721531ffd5feafdf586e Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M jobs/confs/shell-scripts/mock_setup.sh 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Barak Korren: Looks good to me Eyal Edri: Looks good to me Jenkins CI: Passed CI tests David Caro: Verified; Ready for merge -- To view, visit https://gerrit.ovirt.org/51815 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1938c2ed31c8bc6ada47721531ffd5feafdf586e Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sagi Shnaidman <sshnaidm(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: imgbased: Only build el7 artifacts
by dcaro＠redhat.com 15 Jan '16

15 Jan '16

David Caro has submitted this change and it was merged. Change subject: imgbased: Only build el7 artifacts ...................................................................... imgbased: Only build el7 artifacts Currently imgbased is used on el7 only, thus only build those artifacts. Change-Id: I2df258170198513ab4304e14adb33bbb955f2ba8 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> --- M jobs/confs/projects/imgbased/imgbased.yaml 1 file changed, 0 insertions(+), 1 deletion(-) Approvals: Jenkins CI: Passed CI tests David Caro: Verified; Looks good to me; Ready for merge Ryan Barry: Looks good to me -- To view, visit https://gerrit.ovirt.org/51896 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2df258170198513ab4304e14adb33bbb955f2ba8 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ryan Barry <rbarry(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Remove unnecessary ternary operator
by amureini＠redhat.com 15 Jan '16

15 Jan '16

Allon Mureinik has submitted this change and it was merged. Change subject: core: Remove unnecessary ternary operator ...................................................................... core: Remove unnecessary ternary operator DestroyImageCheckCommand is part of the Live Merge flow and has no other callers besides RemoveSnapshotSingleDiskLiveCommand; as such we can see that the list used in getParameters().getImageList() was created in the caller's buildDestroyImageParameters and cannot be null. Change-Id: I785e71c277ee76e9fde358fd80e1cfd034279cbe Signed-off-by: Greg Padgett <gpadgett(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/DestroyImageCheckCommand.java 1 file changed, 1 insertion(+), 3 deletions(-) Approvals: Greg Padgett: Verified Allon Mureinik: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/51822 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I785e71c277ee76e9fde358fd80e1cfd034279cbe Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-ruby[master]: Add rubygems-devel to the list of packages used to build art...
by juan.hernandez＠redhat.com 15 Jan '16

15 Jan '16

Juan Hernandez has submitted this change and it was merged. Change subject: Add rubygems-devel to the list of packages used to build artifacts ...................................................................... Add rubygems-devel to the list of packages used to build artifacts This patch adds the "rubygems-devel" package to the list of packages required to run the "build-artifacts.sh" script, as it is a requirement to build the RPMs. Change-Id: I2d409529af69df10f73e8dbc0bf1cf36fe40a925 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M automation/build-artifacts.packages 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/51895 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2d409529af69df10f73e8dbc0bf1cf36fe40a925 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]: Add the RPM tools to the packages used to build artifacts
by juan.hernandez＠redhat.com 15 Jan '16

15 Jan '16

Juan Hernandez has submitted this change and it was merged. Change subject: Add the RPM tools to the packages used to build artifacts ...................................................................... Add the RPM tools to the packages used to build artifacts This patch adds the packages "rpmdevtools" and "rpm-build" to the list of packages required to run the "build-artifacts.sh" script, as they are needed in order to build RPMs. Change-Id: I80172a9fd396cf36c722006a01b330f51cc32b9a Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M automation/build-artifacts.packages 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/51894 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I80172a9fd396cf36c722006a01b330f51cc32b9a 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]: Change automation to build RPM packages
by juan.hernandez＠redhat.com 15 Jan '16

15 Jan '16

Juan Hernandez has submitted this change and it was merged. Change subject: Change automation to build RPM packages ...................................................................... Change automation to build RPM packages This patch changes the automation scripts so that in addition to the .gem files they also build the .rpm files. Change-Id: I518c146876d5238faacbd46b2933de789defc418 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M automation/build-artifacts.sh 1 file changed, 14 insertions(+), 2 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/51893 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I518c146876d5238faacbd46b2933de789defc418 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]: Add RPM spec file
by juan.hernandez＠redhat.com 15 Jan '16

15 Jan '16

Juan Hernandez has submitted this change and it was merged. Change subject: Add RPM spec file ...................................................................... Add RPM spec file This patch adds an RPM .spec file that can be used to build a RPM package for the SDK. Change-Id: I61abdb7e04f053d1c435b4ba179e40c5dacbb7c2 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M .gitignore A packaging/README.md A packaging/build.sh A packaging/spec.el7.centos.in A packaging/spec.el7.in A packaging/spec.fc23.in 6 files changed, 210 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/51821 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I61abdb7e04f053d1c435b4ba179e40c5dacbb7c2 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 ovirt-node-plugin-vdsm[master]: Check for Hosted Engine in self.application.plugins
by fabiand＠redhat.com 15 Jan '16

15 Jan '16

Fabian Deutsch has submitted this change and it was merged. Change subject: Check for Hosted Engine in self.application.plugins ...................................................................... Check for Hosted Engine in self.application.plugins We cannot call hosted_engine_page.Plugin() directly (needs app as an argument), nor call we call it with app (already registered). Iterate through the plugins in the application and check from there. Change-Id: Ia375098593b827b7406ffedecc1763bc7da7a1b8 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1258397 Signed-off-by: Ryan Barry <rbarry(a)redhat.com> Reviewed-on: https://gerrit.ovirt.org/51825 Continuous-Integration: Jenkins CI Reviewed-by: Fabian Deutsch <fabiand(a)redhat.com> --- M src/engine_page.py 1 file changed, 2 insertions(+), 3 deletions(-) Approvals: Fabian Deutsch: Looks good to me, approved Jenkins CI: Passed CI tests Ryan Barry: Verified -- To view, visit https://gerrit.ovirt.org/51825 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia375098593b827b7406ffedecc1763bc7da7a1b8 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: master Gerrit-Owner: Ryan Barry <rbarry(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ryan Barry <rbarry(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[master]: Bump release to 002-0.6
by sbonazzo＠redhat.com 15 Jan '16

15 Jan '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: Bump release to 002-0.6 ...................................................................... Bump release to 002-0.6 Added required bits for host-node Change-Id: I2071af3732b39d72260a0b3ba7d743d4cf1967d3 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M ovirt-release-master/configure.ac M ovirt-release-master/ovirt-release-master.spec.in 2 files changed, 4 insertions(+), 1 deletion(-) Approvals: Fabian Deutsch: Looks good to me, but someone else must approve Douglas Schilling Landgraf: Verified Sandro Bonazzola: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/51872 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2071af3732b39d72260a0b3ba7d743d4cf1967d3 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0