March 2017 - Engine-commits - oVirt List Archives

Change in ovirt-engine-sdk[master]: Add console ticket example
by Code Review 10 Mar '17

10 Mar '17

>From Ondra Machacek <omachace(a)redhat.com>: Ondra Machacek has submitted this change and it was merged. Change subject: Add console ticket example ...................................................................... Add console ticket example This patch adds an example that shows how to find a graphics console for a given access protocol (VNC or SPICE) and how to get a ticket. Change-Id: I5a5c302d922d963395ac8b46f834afb770adfb48 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- A sdk/examples/get_display_ticket.py 1 file changed, 83 insertions(+), 0 deletions(-) Approvals: Ondra Machacek: Looks good to me, approved Juan Hernandez: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/73786 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5a5c302d922d963395ac8b46f834afb770adfb48 Gerrit-PatchSet: 2 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: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk[sdk_4.1]: Add clone VM from snapshot example
by Code Review 10 Mar '17

10 Mar '17

>From Ondra Machacek <omachace(a)redhat.com>: Ondra Machacek has submitted this change and it was merged. Change subject: Add clone VM from snapshot example ...................................................................... Add clone VM from snapshot example This patch adds an example that shows how to create a new virtual machine cloning it from an snapshot. Change-Id: I1b9eb76715792b4ff2e9901177379976941ff14e Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit 1fdf53e9feb23943e9055eeaf298a40d174eabfa) --- A sdk/examples/clone_vm_from_snapshot.py 1 file changed, 87 insertions(+), 0 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/73865 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1b9eb76715792b4ff2e9901177379976941ff14e Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: sdk_4.1 Gerrit-Owner: Ondra Machacek <omachace(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-java[master]: Add console ticket example
by Code Review 10 Mar '17

10 Mar '17

>From Ondra Machacek <omachace(a)redhat.com>: Ondra Machacek has submitted this change and it was merged. Change subject: Add console ticket example ...................................................................... Add console ticket example This patch adds an example that shows how to find a graphics console for a given access protocol (VNC or SPICE) and how to get a ticket. Change-Id: I3f7b4db57d6968430e91cdf3e9038c53c6f2fd30 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- A sdk/src/test/java/org/ovirt/engine/sdk4/examples/GetDisplayTicket.java 1 file changed, 97 insertions(+), 0 deletions(-) Approvals: Ondra Machacek: Looks good to me, approved Juan Hernandez: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/73792 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3f7b4db57d6968430e91cdf3e9038c53c6f2fd30 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]: Add clone VM from snapshot example
by Code Review 10 Mar '17

10 Mar '17

>From Ondra Machacek <omachace(a)redhat.com>: Ondra Machacek has submitted this change and it was merged. Change subject: Add clone VM from snapshot example ...................................................................... Add clone VM from snapshot example This patch adds an example that shows how to create a new virtual machine cloning it from an snapshot. Change-Id: I1b9eb76715792b4ff2e9901177379976941ff14e Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- A sdk/examples/clone_vm_from_snapshot.py 1 file changed, 87 insertions(+), 0 deletions(-) Approvals: Ondra Machacek: Looks good to me, approved Juan Hernandez: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/73799 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1b9eb76715792b4ff2e9901177379976941ff14e 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[ovirt-engine-4.1]: restapi: Reduce severity of 'path not found'
by Code Review 10 Mar '17

10 Mar '17

>From Tal Nisan <tnisan(a)redhat.com>: Tal Nisan has submitted this change and it was merged. Change subject: restapi: Reduce severity of 'path not found' ...................................................................... restapi: Reduce severity of 'path not found' Currently when the API doesn't isn't able to calculate the URL path for a class, it sends an ERROR message to the log, something like this: ERROR Can't find relative path for class "o.o.e.a.r.VmDisksResource", will return null That isn't really an error, rather a hint for developers that something may be missing in the 'LinkHelper' class. But the ERROR serverity alarms users. This patch changes the severity to WARNING, which is closer to the real meaning. Change-Id: Ib65c904360787d50d22d21fe50747b6ed7453a01 Related-To: https://bugzilla.redhat.com/1410038 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit da7e8b20a4a04e128993a91b23da057804585e45) --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/util/LinkHelper.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Juan Hernandez: Verified Jenkins CI: Passed CI tests Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/73857 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib65c904360787d50d22d21fe50747b6ed7453a01 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.1 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-ha[master]: he: Added additional logging to the score calculation
by Code Review 10 Mar '17

10 Mar '17

>From Martin Sivák <msivak(a)redhat.com>: Martin Sivák has submitted this change and it was merged. Change subject: he: Added additional logging to the score calculation ...................................................................... he: Added additional logging to the score calculation All penalizing decisions should be logged. Change-Id: Ib11e8f36dd564e2293ea6c5ccf650060ec0153cb Bug-Url: https://bugzilla.redhat.com/1406622 Signed-off-by: Denis Chaplygin <dchaplyg(a)redhat.com> --- M ovirt_hosted_engine_ha/agent/states.py M ovirt_hosted_engine_ha/lib/upgrade.py 2 files changed, 5 insertions(+), 0 deletions(-) Approvals: Denis Chaplygin: Verified Martin Sivák: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/72435 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib11e8f36dd564e2293ea6c5ccf650060ec0153cb Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Denis Chaplygin <dchaplyg(a)redhat.com> Gerrit-Reviewer: Denis Chaplygin <dchaplyg(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(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[master]: restapi: Reduce severity of 'path not found'
by Code Review 09 Mar '17

09 Mar '17

>From Oved Ourfali <oourfali(a)redhat.com>: Oved Ourfali has submitted this change and it was merged. Change subject: restapi: Reduce severity of 'path not found' ...................................................................... restapi: Reduce severity of 'path not found' Currently when the API doesn't isn't able to calculate the URL path for a class, it sends an ERROR message to the log, something like this: ERROR Can't find relative path for class "o.o.e.a.r.VmDisksResource", will return null That isn't really an error, rather a hint for developers that something may be missing in the 'LinkHelper' class. But the ERROR serverity alarms users. This patch changes the severity to WARNING, which is closer to the real meaning. Change-Id: Ib65c904360787d50d22d21fe50747b6ed7453a01 Related-To: https://bugzilla.redhat.com/1410038 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/util/LinkHelper.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Juan Hernandez: Verified Jenkins CI: Passed CI tests Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/73837 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib65c904360787d50d22d21fe50747b6ed7453a01 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine 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: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-ruby[sdk_4.1]: Add VM lease example
by Code Review 09 Mar '17

09 Mar '17

>From Juan Hernandez <juan.hernandez(a)redhat.com>: Juan Hernandez has submitted this change and it was merged. Change subject: Add VM lease example ...................................................................... Add VM lease example This patch adds an example that shows how to set the storage domain where the lease for a highly available virtual machine should be created. Change-Id: I43456c26a4ba10e38840fa8cf4118beedde00a2e Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit d0d0782735157d402db61ff03a055d106e66782d) --- A sdk/examples/set_vm_lease_storage_domain.rb 1 file changed, 62 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/73835 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I43456c26a4ba10e38840fa8cf4118beedde00a2e 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: 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 VM lease example
by Code Review 09 Mar '17

09 Mar '17

>From Juan Hernandez <juan.hernandez(a)redhat.com>: Juan Hernandez has submitted this change and it was merged. Change subject: Add VM lease example ...................................................................... Add VM lease example This patch adds an example that shows how to set the storage domain where the lease for a highly available virtual machine should be created. Change-Id: I43456c26a4ba10e38840fa8cf4118beedde00a2e Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- A sdk/examples/set_vm_lease_storage_domain.rb 1 file changed, 62 insertions(+), 0 deletions(-) Approvals: Ondra Machacek: Looks good to me, but someone else must approve Tal Nisan: 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/73747 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I43456c26a4ba10e38840fa8cf4118beedde00a2e 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: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in gerrit-admin[master]: Added support to set d/s bugs status to MODIFIED
by Code Review 09 Mar '17

09 Mar '17

>From Eyal Edri <eedri(a)redhat.com>: Eyal Edri has submitted this change and it was merged. Change subject: Added support to set d/s bugs status to MODIFIED ...................................................................... Added support to set d/s bugs status to MODIFIED The current set_modified hook changes the bug status to MODIFIED only u/s bugs. Added support to change the bug status to MODIFIED also for d/s bugs with 'Red Hat Enterprise Virtualization Manager' product. NOTE: - The product added to the config file (in 'PRODUCTS' variable) - The 'PRODUCTS' variable holds a list (separated with comma ',') with all the allowed products. In Addition: - Removed the draft variable (doesn't exist in change-merged triger) Change-Id: Ia83f4931dc3083cfdba3033c4112cc3e553892ec Jira-Ticket: https://ovirt-jira.atlassian.net/browse/OVIRT-1234 Signed-off-by: Shlomi Ben-David <sbendavi(a)redhat.com> --- M hooks/custom_hooks/set_modified 1 file changed, 51 insertions(+), 56 deletions(-) Approvals: Shlomo Ben David: Verified Eyal Edri: Looks good to me, approved Gil Shinar: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/73694 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia83f4931dc3083cfdba3033c4112cc3e553892ec Gerrit-PatchSet: 2 Gerrit-Project: gerrit-admin Gerrit-Branch: master Gerrit-Owner: Shlomo Ben David <sbendavi(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Gil Shinar <gshinar(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shlomo Ben David <sbendavi(a)redhat.com>

1 0