October 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine-sdk-ruby[sdk_4.0]: Install built RPMs
by juan.hernandez＠redhat.com 24 Oct '16

24 Oct '16

Juan Hernandez has submitted this change and it was merged. Change subject: Install built RPMs ...................................................................... Install built RPMs In the past there have been situations where the built RPMs failed during installation, because they required packages that don't exist in the distribution. In order to detect that earlier in the future this patch changes the automation scripts so that the built RPMs are installed as part of the build process. Change-Id: I34352228f6b394ce9ad82d5a80a45056550b3d47 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit aa52a827fca3afbd6da116214b80fb591ad91bdc) --- D automation/build-artifacts.packages A automation/build-artifacts.packages.el7 A automation/build-artifacts.packages.fc23 A automation/build-artifacts.packages.fc24 R automation/build.packages.el7 C automation/build.packages.fc23 C automation/build.packages.fc24 M automation/build.py D automation/check-merged.packages A automation/check-merged.packages.el7 A automation/check-merged.packages.fc23 A automation/check-merged.packages.fc24 D automation/check-patch.packages A automation/check-patch.packages.el7 A automation/check-patch.packages.fc23 A automation/check-patch.packages.fc24 16 files changed, 29 insertions(+), 3 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/65689 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I34352228f6b394ce9ad82d5a80a45056550b3d47 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]: Require json >= 1.8 instead of >= 2.0
by juan.hernandez＠redhat.com 24 Oct '16

24 Oct '16

Juan Hernandez has submitted this change and it was merged. Change subject: Require json >= 1.8 instead of >= 2.0 ...................................................................... Require json >= 1.8 instead of >= 2.0 Currently the SDK requires version 2 or newer of the 'json' gem, but actually it doesn't need version 2, it can perfectly work with 1.8. In addition Fedora and CentOS provide version 1.8, so the RPM generated for the SDK won't work in those distributions. To solve this problem this patch changes the SDK so that it requires version 1.8. Change-Id: Ic1f5095c1f9181a91721484ab6838be78d4c0df4 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit 68e56414fe14e756ba842e27b3d8d38d96d736ac) --- M sdk/ovirt-engine-sdk.gemspec 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/65688 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic1f5095c1f9181a91721484ab6838be78d4c0df4 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-java[master]: Clarify that DiskAttachment.disk is only a link
by omachace＠redhat.com 24 Oct '16

24 Oct '16

Ondra Machacek has submitted this change and it was merged. Change subject: Clarify that DiskAttachment.disk is only a link ...................................................................... Clarify that DiskAttachment.disk is only a link This patch improves the example of how to list virtual machine disks in order to clarify that the DiskAttachment.disk attribute is only a link, and that in order to retrieve the actual disk data an additional request is required, for example using the 'follow_link' method. Change-Id: I32ad8628aa92bf45526dc5bb585f46befd85a49c Related-To: https://bugzilla.redhat.com/1387951 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M sdk/src/test/java/org/ovirt/engine/sdk4/examples/ListVmDisks.java 1 file changed, 9 insertions(+), 2 deletions(-) Approvals: Ondra Machacek: Looks good to me, approved Juan Hernandez: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/65676 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I32ad8628aa92bf45526dc5bb585f46befd85a49c Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-java 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: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk[master]: Clarify that DiskAttachment.disk is only a link
by omachace＠redhat.com 24 Oct '16

24 Oct '16

Ondra Machacek has submitted this change and it was merged. Change subject: Clarify that DiskAttachment.disk is only a link ...................................................................... Clarify that DiskAttachment.disk is only a link This patch improves the example of how to list virtual machine disks in order to clarify that the DiskAttachment.disk attribute is only a link, and that in order to retrieve the actual disk data an additional request is required, for example using the 'follow_link' method. Change-Id: Iae92b665622288afc13d176194223a4cc4366a7d Related-To: https://bugzilla.redhat.com/1387951 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M sdk/examples/list_vm_disks.py 1 file changed, 9 insertions(+), 2 deletions(-) Approvals: Ondra Machacek: Looks good to me, approved Juan Hernandez: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/65675 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iae92b665622288afc13d176194223a4cc4366a7d Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk 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: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-ruby[master]: Install built RPMs
by juan.hernandez＠redhat.com 24 Oct '16

24 Oct '16

Juan Hernandez has submitted this change and it was merged. Change subject: Install built RPMs ...................................................................... Install built RPMs In the past there have been situations where the built RPMs failed during installation, because they required packages that don't exist in the distribution. In order to detect that earlier in the future this patch changes the automation scripts so that the built RPMs are installed as part of the build process. Change-Id: I34352228f6b394ce9ad82d5a80a45056550b3d47 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- D automation/build-artifacts.packages A automation/build-artifacts.packages.el7 A automation/build-artifacts.packages.fc23 A automation/build-artifacts.packages.fc24 R automation/build.packages.el7 C automation/build.packages.fc23 C automation/build.packages.fc24 M automation/build.py D automation/check-merged.packages A automation/check-merged.packages.el7 A automation/check-merged.packages.fc23 A automation/check-merged.packages.fc24 D automation/check-patch.packages A automation/check-patch.packages.el7 A automation/check-patch.packages.fc23 A automation/check-patch.packages.fc24 16 files changed, 29 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/65680 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I34352228f6b394ce9ad82d5a80a45056550b3d47 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: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-ruby[master]: Require json >= 1.8 instead of >= 2.0
by juan.hernandez＠redhat.com 24 Oct '16

24 Oct '16

Juan Hernandez has submitted this change and it was merged. Change subject: Require json >= 1.8 instead of >= 2.0 ...................................................................... Require json >= 1.8 instead of >= 2.0 Currently the SDK requires version 2 or newer of the 'json' gem, but actually it doesn't need version 2, it can perfectly work with 1.8. In addition Fedora and CentOS provide version 1.8, so the RPM generated for the SDK won't work in those distributions. To solve this problem this patch changes the SDK so that it requires version 1.8. Change-Id: Ic1f5095c1f9181a91721484ab6838be78d4c0df4 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M sdk/ovirt-engine-sdk.gemspec 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/65678 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic1f5095c1f9181a91721484ab6838be78d4c0df4 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: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-ruby[sdk_4.0]: Clarify that DiskAttachment.disk is only a link
by juan.hernandez＠redhat.com 24 Oct '16

24 Oct '16

Juan Hernandez has submitted this change and it was merged. Change subject: Clarify that DiskAttachment.disk is only a link ...................................................................... Clarify that DiskAttachment.disk is only a link This patch improves the example of how to list virtual machine disks in order to clarify that the DiskAttachment.disk attribute is only a link, and that in order to retrieve the actual disk data an additional request is required, for example using the 'follow_link' method. Change-Id: I3826d8d43bd571cf9326b104d716cf179b16664a Bug-Url: https://bugzilla.redhat.com/1387951 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit a4036d799aa347d1638aa537750a41e46d6de28f) --- M sdk/examples/list_vm_disks.rb 1 file changed, 9 insertions(+), 2 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/65679 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3826d8d43bd571cf9326b104d716cf179b16664a 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]: Clarify that DiskAttachment.disk is only a link
by juan.hernandez＠redhat.com 24 Oct '16

24 Oct '16

Juan Hernandez has submitted this change and it was merged. Change subject: Clarify that DiskAttachment.disk is only a link ...................................................................... Clarify that DiskAttachment.disk is only a link This patch improves the example of how to list virtual machine disks in order to clarify that the DiskAttachment.disk attribute is only a link, and that in order to retrieve the actual disk data an additional request is required, for example using the 'follow_link' method. Change-Id: I3826d8d43bd571cf9326b104d716cf179b16664a Bug-Url: https://bugzilla.redhat.com/1387951 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M sdk/examples/list_vm_disks.rb 1 file changed, 9 insertions(+), 2 deletions(-) Approvals: Ondra Machacek: Looks good to me, but someone else must approve Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/65674 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3826d8d43bd571cf9326b104d716cf179b16664a 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: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in mom[master]: Release mom-0.5.8
by msivak＠redhat.com 24 Oct '16

24 Oct '16

Martin Sivák has submitted this change and it was merged. Change subject: Release mom-0.5.8 ...................................................................... Release mom-0.5.8 Change-Id: I918f21a0c97d71e0c6c26e8d8e9266be1fbf8c6b Signed-off-by: Martin Sivak <msivak(a)redhat.com> Reviewed-on: https://gerrit.ovirt.org/65677 Continuous-Integration: Jenkins CI --- M configure.ac 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Martin Sivák: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/65677 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I918f21a0c97d71e0c6c26e8d8e9266be1fbf8c6b Gerrit-PatchSet: 2 Gerrit-Project: mom Gerrit-Branch: master Gerrit-Owner: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in mom[master]: Remove initscripts (in favor of systemd) from the spec file
by msivak＠redhat.com 24 Oct '16

24 Oct '16

Martin Sivák has submitted this change and it was merged. Change subject: Remove initscripts (in favor of systemd) from the spec file ...................................................................... Remove initscripts (in favor of systemd) from the spec file Change-Id: Id8f5ef67f0b27394eef5010d9f6b7e77967f1cac Signed-off-by: Martin Sivak <msivak(a)redhat.com> Reviewed-on: https://gerrit.ovirt.org/65660 Continuous-Integration: Jenkins CI Reviewed-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M mom.spec.in 1 file changed, 12 insertions(+), 16 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Martin Sivák: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/65660 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id8f5ef67f0b27394eef5010d9f6b7e77967f1cac Gerrit-PatchSet: 3 Gerrit-Project: mom Gerrit-Branch: master Gerrit-Owner: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Andrej Krejcir <akrejcir(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Jenny Tokar <jtokar(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Phillip Bailey <phbailey(a)redhat.com> Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yanir Quinn <yquinn(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0