- Engine-commits - oVirt List Archives

Change in ovirt-hosted-engine-ha[ovirt-hosted-engine-ha-1.2]: added EngineStarting state
by msivak＠redhat.com 15 Jul '14

15 Jul '14

Martin Sivák has submitted this change and it was merged. Change subject: added EngineStarting state ...................................................................... added EngineStarting state when then VM with engine is starting it takes some time and goes thru states "waitforlaunch" and "powering up". During the start up the engine was marked as UnexpectedlyDown because it wasn't 'up' and the host got score 0 which might lead to killing the VM while it's starting and trying to start it on another host where the same situation repeats. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1093638 Change-Id: Ia7e5bb28eeb839b684be74ea768265d9dab01175 Signed-off-by: Jiri Moskovcak <jmoskovc(a)redhat.com> --- M ovirt_hosted_engine_ha/agent/constants.py.in M ovirt_hosted_engine_ha/agent/states.py M ovirt_hosted_engine_ha/broker/submonitors/engine_health.py 3 files changed, 43 insertions(+), 5 deletions(-) Approvals: Martin Sivák: Looks good to me, approved Jiří Moskovčák: Verified -- To view, visit http://gerrit.ovirt.org/30027 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia7e5bb28eeb839b684be74ea768265d9dab01175 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: ovirt-hosted-engine-ha-1.2 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.2]: refactoring: use constants for engine states instead of plai...
by msivak＠redhat.com 15 Jul '14

15 Jul '14

Martin Sivák has submitted this change and it was merged. Change subject: refactoring: use constants for engine states instead of plain strings ...................................................................... refactoring: use constants for engine states instead of plain strings Change-Id: I27ccb6533676edfc1b5bfbf009d5820853eebab6 Signed-off-by: Jiri Moskovcak <jmoskovc(a)redhat.com> --- M ovirt_hosted_engine_ha/broker/submonitors/engine_health.py M ovirt_hosted_engine_ha/lib/Makefile.am A ovirt_hosted_engine_ha/lib/engine.py 3 files changed, 42 insertions(+), 4 deletions(-) Approvals: Martin Sivák: Looks good to me, approved Jiří Moskovčák: Verified -- To view, visit http://gerrit.ovirt.org/30064 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I27ccb6533676edfc1b5bfbf009d5820853eebab6 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: ovirt-hosted-engine-ha-1.2 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: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: confusing error message retrieving from http session
by awels＠redhat.com 15 Jul '14

15 Jul '14

Alexander Wels has submitted this change and it was merged. Change subject: engine: confusing error message retrieving from http session ...................................................................... engine: confusing error message retrieving from http session - Fixed confusing error message when retrieving non existing value from http session. Instead of throwing error message in log, just return null. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1101551 Change-Id: I336430e35bbbad07ea9566c8f5293980056361de Signed-off-by: Alexander Wels <awels(a)redhat.com> --- M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/server/gwt/GenericApiGWTServiceImpl.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alexander Wels: Verified Vojtech Szocs: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29142 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I336430e35bbbad07ea9566c8f5293980056361de Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

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

15 Jul '14

Oved Ourfali 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_06_0010_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/29784 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I89d41ac4ed0c28ec330c2db82f91bc3fcc9e2073 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: mooli tayer <mtayer(a)redhat.com> Gerrit-Reviewer: Eli Mesika <elimesika(a)gmail.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: mooli tayer <mtayer(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: restapi: Faster JAXB element creation
by juan.hernandez＠redhat.com 15 Jul '14

15 Jul '14

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Faster JAXB element creation ...................................................................... restapi: Faster JAXB element creation Currently when we need to create a JAXB element we always iterate the object factory looking for the appropiate method. To improve the performance of this lookup this patch introduces an index that is created during startup so that the iteration is done only once. Change-Id: Ie6f2cc91701ff45a443d3b8b37e8225d7b26c4fa Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/xml/JAXBProvider.java 1 file changed, 19 insertions(+), 13 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved anmolbabu: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/30090 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie6f2cc91701ff45a443d3b8b37e8225d7b26c4fa 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: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: anmolbabu <anbabu(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]: restapi: Fix parameters of add DataCenter
by tnisan＠redhat.com 15 Jul '14

15 Jul '14

Tal Nisan has submitted this change and it was merged. Change subject: restapi: Fix parameters of add DataCenter ...................................................................... restapi: Fix parameters of add DataCenter The "local" property of the data center is mandatory, and the version numbers are optional. Change-Id: I5db741f4a6d9d479feaf5a51a745e0d0190f841b Bug-Url: https://bugzilla.redhat.com/1104085 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit 013d573fd71139418da30ea53113cdde470ee0eb) --- M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml 1 file changed, 7 insertions(+), 4 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/30092 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5db741f4a6d9d479feaf5a51a745e0d0190f841b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: restapi: Fix parameters of add DataCenter
by juan.hernandez＠redhat.com 15 Jul '14

15 Jul '14

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Fix parameters of add DataCenter ...................................................................... restapi: Fix parameters of add DataCenter The "local" property of the data center is mandatory, and the version numbers are optional. Change-Id: I5db741f4a6d9d479feaf5a51a745e0d0190f841b Bug-Url: https://bugzilla.redhat.com/1104085 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml 1 file changed, 7 insertions(+), 4 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29938 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5db741f4a6d9d479feaf5a51a745e0d0190f841b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5]: restapi: Fix prefix in RSDL
by tnisan＠redhat.com 15 Jul '14

15 Jul '14

Tal Nisan has submitted this change and it was merged. Change subject: restapi: Fix prefix in RSDL ...................................................................... restapi: Fix prefix in RSDL Currently during the build process we generate a RSDL document that contains "href" attributes including the prefix "/ovirt-engine/api". But the application is deployed twice, once for "/api" and another for "/ovirt-engine/api", thus the RSDL document presented by "/api" isn't valid. To avoid this issue this patch changes the build process so that it will generate a RSDL document that doesn't contain any prefix. The correct prefix will be added during runtime by each deployment of the application. Change-Id: If5fadb92b6f3a6b06e44729c967289c5917f9a25 Bug-Url: https://bugzilla.redhat.com/1102772 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit 5a047b30553124a83946ca4447fb801e1bf8ad35) --- M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/rsdl/RsdlBuilder.java M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/rsdl/RsdlManager.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendApiResource.java 3 files changed, 66 insertions(+), 43 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/30091 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If5fadb92b6f3a6b06e44729c967289c5917f9a25 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

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

15 Jul '14

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Fix prefix in RSDL ...................................................................... restapi: Fix prefix in RSDL Currently during the build process we generate a RSDL document that contains "href" attributes including the prefix "/ovirt-engine/api". But the application is deployed twice, once for "/api" and another for "/ovirt-engine/api", thus the RSDL document presented by "/api" isn't valid. To avoid this issue this patch changes the build process so that it will generate a RSDL document that doesn't contain any prefix. The correct prefix will be added during runtime by each deployment of the application. Change-Id: If5fadb92b6f3a6b06e44729c967289c5917f9a25 Bug-Url: https://bugzilla.redhat.com/1102772 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/RsdlBuilder.java M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/rsdl/RsdlManager.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendApiResource.java 3 files changed, 66 insertions(+), 43 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/30018 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If5fadb92b6f3a6b06e44729c967289c5917f9a25 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: Ori Liel <oliel(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]: restapi: fix cluster's schedulingpolicy back-compat
by tnisan＠redhat.com 15 Jul '14

15 Jul '14

Tal Nisan has submitted this change and it was merged. Change subject: restapi: fix cluster's schedulingpolicy back-compat ...................................................................... restapi: fix cluster's schedulingpolicy back-compat Ignores old (still supported) behavior (thresholds). Change-Id: I4a5fa4d95bc55bed36d800532b81eb4340e4083d Bug-Url: https://bugzilla.redhat.com/1118384 Signed-off-by: Gilad Chaplik <gchaplik(a)redhat.com> (cherry picked from commit fb8cef7c16f47efca18da76297a80234042f220d) --- M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/ClusterMapper.java 1 file changed, 9 insertions(+), 3 deletions(-) Approvals: Juan Hernandez: Looks good to me, approved Gilad Chaplik: Verified -- To view, visit http://gerrit.ovirt.org/30085 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4a5fa4d95bc55bed36d800532b81eb4340e4083d Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0