July 2014 - Engine-commits - oVirt List Archives

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

Change in ovirt-hosted-engine-ha[ovirt-hosted-engine-ha-1.1]: try harder when initializing
by msivak＠redhat.com 15 Jul '14

15 Jul '14

Martin Sivák has submitted this change and it was merged. Change subject: try harder when initializing ...................................................................... try harder when initializing the storage initialization is async task, so sometimes we try to access some files before the file exists, in that case we should just wait a while and try again Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1119699 Change-Id: Iac39bedfa78479f71674c3f5e673ba3814f52279 Signed-off-by: Jiri Moskovcak <jmoskovc(a)redhat.com> --- M ovirt_hosted_engine_ha/agent/constants.py.in M ovirt_hosted_engine_ha/agent/hosted_engine.py M ovirt_hosted_engine_ha/lib/exceptions.py 3 files changed, 67 insertions(+), 35 deletions(-) Approvals: Martin Sivák: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29937 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iac39bedfa78479f71674c3f5e673ba3814f52279 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: ovirt-hosted-engine-ha-1.1 Gerrit-Owner: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-ha[ovirt-hosted-engine-ha-1.1]: try harder when connecting the storage
by msivak＠redhat.com 15 Jul '14

15 Jul '14

Martin Sivák has submitted this change and it was merged. Change subject: try harder when connecting the storage ...................................................................... try harder when connecting the storage the agent has to wait between attempts to connect the storage to give vdsm some time to initialize Change-Id: I4c512aad490366b0fbd890d156b37243993dc498 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1119702 Signed-off-by: Jiri Moskovcak <jmoskovc(a)redhat.com> --- M ovirt_hosted_engine_ha/agent/hosted_engine.py 1 file changed, 9 insertions(+), 5 deletions(-) Approvals: Martin Sivák: Looks good to me, approved Jiří Moskovčák: Verified -- To view, visit http://gerrit.ovirt.org/29924 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4c512aad490366b0fbd890d156b37243993dc498 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: ovirt-hosted-engine-ha-1.1 Gerrit-Owner: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-ha[ovirt-hosted-engine-ha-1.1]: try harder when starting vdsmd
by msivak＠redhat.com 15 Jul '14

15 Jul '14

Martin Sivák has submitted this change and it was merged. Change subject: try harder when starting vdsmd ...................................................................... try harder when starting vdsmd I've observed a situation when ha-agent is started too early and it fails to start vdsmd, but later when vdsmd is started ha-agent also starts fine. This patch makes agent to wait a while and retry if the first attempt to start vdsmd fails Change-Id: I806437b8c5eafd32fb37d0b3f2f59995faa28cdc Signed-off-by: Jiri Moskovcak <jmoskovc(a)redhat.com> --- M ovirt_hosted_engine_ha/agent/constants.py.in M ovirt_hosted_engine_ha/agent/hosted_engine.py 2 files changed, 17 insertions(+), 2 deletions(-) Approvals: Martin Sivák: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29923 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I806437b8c5eafd32fb37d0b3f2f59995faa28cdc Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: ovirt-hosted-engine-ha-1.1 Gerrit-Owner: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-ha[ovirt-hosted-engine-ha-1.1]: use max not min when computing the best score
by msivak＠redhat.com 15 Jul '14

15 Jul '14

Martin Sivák has submitted this change and it was merged. Change subject: use max not min when computing the best score ...................................................................... use max not min when computing the best score The best score is the highest score, so this was a 'thinko'. However this bug manifests itself only if there is more than 2 hosts in the setup. Change-Id: I0c265aa60611793b1cd4738ec057e9e7d02747a1 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1119763 Signed-off-by: Jiri Moskovcak <jmoskovc(a)redhat.com> --- M ovirt_hosted_engine_ha/agent/state_machine.py 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Martin Sivák: Looks good to me, approved Jiří Moskovčák: Verified; Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/29922 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0c265aa60611793b1cd4738ec057e9e7d02747a1 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: ovirt-hosted-engine-ha-1.1 Gerrit-Owner: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: restapi: Remove trailing slashes from RSDL prefix
by juan.hernandez＠redhat.com 15 Jul '14

15 Jul '14

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Remove trailing slashes from RSDL prefix ...................................................................... restapi: Remove trailing slashes from RSDL prefix The RSDL document is loaded lazily, as part of the first request to the RESTAPI. The prefix used to build the "href" attributes is extracted from the path of this first request, and depending on that request it may or may not contain a trailing slash. To make sure that the RSDL has always the same content we need to remove any trailing slashes from the prefix before using it. Change-Id: Ieac656924ca999ddc9abc502e3fc6c23537e4769 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/rsdl/RsdlManager.java 1 file changed, 5 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/30100 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ieac656924ca999ddc9abc502e3fc6c23537e4769 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0