- Engine-commits - oVirt List Archives

Change in ovirt-guest-agent[ovirt-3.5]: Open mode for logfiles should be 'a+' rather than 'a'
by vfeenstr＠redhat.com 15 Jul '14

15 Jul '14

Vinzenz Feenstra has submitted this change and it was merged. Change subject: Open mode for logfiles should be 'a+' rather than 'a' ...................................................................... Open mode for logfiles should be 'a+' rather than 'a' Change-Id: Ic4914ea6da41df191475896c0c9180d347c57501 Bug-Url: https://bugzilla.redhat.com/1115515 Signed-off-by: Vinzenz Feenstra <vfeenstr(a)redhat.com> --- M configurations/default-logger.conf M configurations/default-logger.ini M configurations/ovirt-guest-agent.conf M configurations/ovirt-guest-agent.ini 4 files changed, 4 insertions(+), 4 deletions(-) Approvals: Martin Betak: Looks good to me, but someone else must approve Vinzenz Feenstra: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/30106 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic4914ea6da41df191475896c0c9180d347c57501 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-guest-agent Gerrit-Branch: ovirt-3.5 Gerrit-Owner: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-guest-agent[ovirt-3.5]: Remove downstream names from upstream ini
by vfeenstr＠redhat.com 15 Jul '14

15 Jul '14

Vinzenz Feenstra has submitted this change and it was merged. Change subject: Remove downstream names from upstream ini ...................................................................... Remove downstream names from upstream ini Change-Id: Ic5ef41725a95bb19d2748d53a51bdbac432c9f84 Bug-Url: https://bugzilla.redhat.com/1117690 Signed-off-by: Vinzenz Feenstra <vfeenstr(a)redhat.com> --- M configurations/default-logger.ini M configurations/ovirt-guest-agent.ini 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Martin Betak: Looks good to me, but someone else must approve Vinzenz Feenstra: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/30105 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic5ef41725a95bb19d2748d53a51bdbac432c9f84 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-guest-agent Gerrit-Branch: ovirt-3.5 Gerrit-Owner: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5]: userportal, webadmin: Disabled type suggest boxes are not ful...
by tnisan＠redhat.com 15 Jul '14

15 Jul '14

Tal Nisan has submitted this change and it was merged. Change subject: userportal,webadmin: Disabled type suggest boxes are not fully filled ...................................................................... userportal,webadmin: Disabled type suggest boxes are not fully filled Changed the ListModelTypeAheadListBox.ui.xml in a way that the underlying <input> takes the whole available space and added to it's styles the same style what the GWT's ListBox is using. Change-Id: Ic26a695aa127f36023b4e0de01919613280ae960 Bug-Url: https://bugzilla.redhat.com/1066443 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/editor/ListModelTypeAheadListBox.ui.xml 1 file changed, 6 insertions(+), 4 deletions(-) Approvals: Tomas Jelinek: Verified Alexander Wels: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29852 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic26a695aa127f36023b4e0de01919613280ae960 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.5]: userportal, webadmin: Unable to scroll down template list usi...
by tnisan＠redhat.com 15 Jul '14

15 Jul '14

Tal Nisan has submitted this change and it was merged. Change subject: userportal,webadmin: Unable to scroll down template list using IE9 ...................................................................... userportal,webadmin: Unable to scroll down template list using IE9 Three issues: 1: when clicking the scrollbar next to the template selection, IE9 did not fire a click event properly (a known IE9 bug). Fixed by attaching the listeners to the parent element (the whole suggestion box). 2: when using up/down keys, the proper template has been selected, but the view was not scrolled into it. Fixed by scrolling into the view after pressing arrow keys and also when opening the suggestions. 3: tweaked the CSS a bit so the content of the suggestions are not shifted Tested on IE9 and FF. Change-Id: I93312c75f7605cb007ae395d900ef8ced35c6b14 Bug-Url: https://bugzilla.redhat.com/1100194 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/editor/BaseListModelSuggestBox.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/editor/ListModelTypeAheadListBox.java 2 files changed, 52 insertions(+), 4 deletions(-) Approvals: Tomas Jelinek: Verified Alexander Wels: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29851 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I93312c75f7605cb007ae395d900ef8ced35c6b14 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.5]: engine: Enable detaching network from host when a VM has an ...
by tnisan＠redhat.com 15 Jul '14

15 Jul '14

Tal Nisan has submitted this change and it was merged. Change subject: engine: Enable detaching network from host when a VM has an unplugged VNIC profile of that network ...................................................................... engine: Enable detaching network from host when a VM has an unplugged VNIC profile of that network Enable detaching network from host when a VM has an unplugged VNIC profile of that network. Change-Id: I10db794df55b25925423166d33549b92856e9dd0 Bug-Url: https://bugzilla.redhat.com/1115101 Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com> (cherry picked from commit 50a7ff177a8be2d9b8250e76882ad0208fc20169) --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/VmInterfaceManager.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/SetupNetworksHelper.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/network/VmInterfaceManagerTest.java 3 files changed, 32 insertions(+), 17 deletions(-) Approvals: Moti Asayag: Looks good to me, approved Yevgeny Zaspitsky: Verified -- To view, visit http://gerrit.ovirt.org/30096 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I10db794df55b25925423166d33549b92856e9dd0 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.5]: upgrade: update vds cpu name of existing hosts to new name.
by tnisan＠redhat.com 15 Jul '14

15 Jul '14

Tal Nisan has submitted this change and it was merged. Change subject: upgrade: update vds cpu name of existing hosts to new name. ...................................................................... upgrade: update vds cpu name of existing hosts to new name. Commit 25b963 changed a certain cpu_name in the vds_option 'ServerCPUList' to make it more consistent with other cpu names: 'Intel Haswell' -> 'Intel Haswell Family' This commit add an update of existing clusters to the new name. Change-Id: I89d41ac4ed0c28ec330c2db82f91bc3fcc9e2073 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1114994 Signed-off-by: Mooli Tayer <mtayer(a)redhat.com> --- A packaging/dbscripts/upgrade/03_05_0760_update_haswell_vds_to_new_name.sql 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: mooli tayer: Verified Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/30014 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I89d41ac4ed0c28ec330c2db82f91bc3fcc9e2073 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: mooli tayer <mtayer(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: mooli tayer <mtayer(a)redhat.com>

1 0

Change in ovirt-engine[master]: restapi: Expose Vm#runInPaused property
by juan.hernandez＠redhat.com 15 Jul '14

15 Jul '14

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Expose Vm#runInPaused property ...................................................................... restapi: Expose Vm#runInPaused property Added new element to VM and Template schema: <start_paused> true|false </start_paused> Change-Id: I142adaa9929e3fc8d67311ff7610a9f4557666ff Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1116550 Signed-off-by: Martin Betak <mbetak(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/TemplateMapper.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java M backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/TemplateMapperTest.java M backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/VmMapperTest.java 6 files changed, 24 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Looks good to me, approved Martin Betak: Verified -- To view, visit http://gerrit.ovirt.org/30104 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I142adaa9929e3fc8d67311ff7610a9f4557666ff Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-guest-agent[master]: Handle exceptions on reading package lists
by vfeenstr＠redhat.com 15 Jul '14

15 Jul '14

Vinzenz Feenstra has submitted this change and it was merged. Change subject: Handle exceptions on reading package lists ...................................................................... Handle exceptions on reading package lists In some cases it might be possible that the package database is not accessible to non-privileged users. This patch handles and logs this issue and prevents the exception from stopping the guest agent thread with an unhandled exception. Change-Id: Id44210235a3bd8fadf91cb919896c1abe8b4c250 Bug-Url: https://bugzilla.redhat.com/1117632 Signed-off-by: Vinzenz Feenstra <vfeenstr(a)redhat.com> --- M ovirt-guest-agent/GuestAgentLinux2.py 1 file changed, 7 insertions(+), 4 deletions(-) Approvals: Martin Betak: Looks good to me, but someone else must approve Vinzenz Feenstra: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/30062 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id44210235a3bd8fadf91cb919896c1abe8b4c250 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-guest-agent Gerrit-Branch: master Gerrit-Owner: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <mskrivan(a)redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-guest-agent[master]: Open mode for logfiles should be 'a+' rather than 'a'
by vfeenstr＠redhat.com 15 Jul '14

15 Jul '14

Vinzenz Feenstra has submitted this change and it was merged. Change subject: Open mode for logfiles should be 'a+' rather than 'a' ...................................................................... Open mode for logfiles should be 'a+' rather than 'a' Change-Id: Ic4914ea6da41df191475896c0c9180d347c57501 Bug-Url: https://bugzilla.redhat.com/1115515 Signed-off-by: Vinzenz Feenstra <vfeenstr(a)redhat.com> --- M configurations/default-logger.conf M configurations/default-logger.ini M configurations/ovirt-guest-agent.conf M configurations/ovirt-guest-agent.ini 4 files changed, 4 insertions(+), 4 deletions(-) Approvals: Martin Betak: Looks good to me, but someone else must approve Vinzenz Feenstra: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/30089 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic4914ea6da41df191475896c0c9180d347c57501 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-guest-agent Gerrit-Branch: master Gerrit-Owner: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-guest-agent[master]: Remove downstream names from upstream ini
by vfeenstr＠redhat.com 15 Jul '14

15 Jul '14

Vinzenz Feenstra has submitted this change and it was merged. Change subject: Remove downstream names from upstream ini ...................................................................... Remove downstream names from upstream ini Change-Id: Ic5ef41725a95bb19d2748d53a51bdbac432c9f84 Bug-Url: https://bugzilla.redhat.com/1117690 Signed-off-by: Vinzenz Feenstra <vfeenstr(a)redhat.com> --- M configurations/default-logger.ini M configurations/ovirt-guest-agent.ini 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Martin Betak: Looks good to me, but someone else must approve Vinzenz Feenstra: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/30088 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic5ef41725a95bb19d2748d53a51bdbac432c9f84 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-guest-agent Gerrit-Branch: master Gerrit-Owner: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0