- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: restapi: fix cluster's schedulingpolicy back-compat
by gchaplik＠redhat.com 15 Jul '14

15 Jul '14

Gilad Chaplik 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> --- 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/30042 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4a5fa4d95bc55bed36d800532b81eb4340e4083d Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master 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: 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]: core: Add space validation when creating snapshot
by tnisan＠redhat.com 15 Jul '14

15 Jul '14

Tal Nisan has submitted this change and it was merged. Change subject: core: Add space validation when creating snapshot ...................................................................... core: Add space validation when creating snapshot In CreateAllSnapshotsFromVmCommand, added to CDA missing storage space validations. For that, added allDomainsHaveSpaceForNewDisks to MultipleStorageDomainsValidator . Added relevant tests. Change-Id: I0a5b5df695d30a34fbdef31da2320b322e27466b Bug-Url: https://bugzilla.redhat.com/1053752 Signed-off-by: Vered Volansky <vvolansk(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CreateAllSnapshotsFromVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/MultipleStorageDomainsValidator.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/CreateAllSnapshotsFromVmCommandTest.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/validator/MultipleStorageDomainsValidatorTest.java 4 files changed, 118 insertions(+), 8 deletions(-) Approvals: Allon Mureinik: Looks good to me, approved Vered Volansky: Verified -- To view, visit http://gerrit.ovirt.org/30056 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0a5b5df695d30a34fbdef31da2320b322e27466b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Vered Volansky <vvolansk(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-log-collector[ovirt-log-collector-3.4]: sos: aligned sos2 plugins to sos 3.1
by sbonazzo＠redhat.com 15 Jul '14

15 Jul '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: sos: aligned sos2 plugins to sos 3.1 ...................................................................... sos: aligned sos2 plugins to sos 3.1 Aligned sos 2 plugins shipped within log-collector with the plugins included in sos 3.1 released in Fedora. The new ovirt plugin also include support for ovirt-scheduler-proxy. Aligned log-collector for using ovirt plugin. Require newest sos ovirt rpm in order to update sos previously shipped within ovirt repo. Bug-Url: https://bugzilla.redhat.com/1073944 Bug-Url: https://bugzilla.redhat.com/1037663 Change-Id: I74b1d24cff897d6cc393ee68edfe0d78030877c5 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-log-collector.spec.in M po/POTFILES.in M src/__main__.py M src/sos/plugins/Makefile.am D src/sos/plugins/engine.py A src/sos/plugins/ovirt.py D src/sos/plugins/ovirt_engine_dwh.py D src/sos/plugins/ovirt_engine_reports.py M src/sos/plugins/postgresql.py 9 files changed, 258 insertions(+), 346 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/30055 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I74b1d24cff897d6cc393ee68edfe0d78030877c5 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-log-collector Gerrit-Branch: ovirt-log-collector-3.4 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-hosted-engine-ha[ovirt-hosted-engine-ha-1.2]: 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=1093621 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: Sandro Bonazzola: Looks good to me, but someone else must approve 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/29921 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4c512aad490366b0fbd890d156b37243993dc498 Gerrit-PatchSet: 2 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]: 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=1093366 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: Sandro Bonazzola: Looks good to me, but someone else must approve Martin Sivák: Looks good to me, approved Jiří Moskovčák: Verified -- To view, visit http://gerrit.ovirt.org/29920 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0c265aa60611793b1cd4738ec057e9e7d02747a1 Gerrit-PatchSet: 2 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-engine[ovirt-engine-3.4.3]: core: minor cleanup in MigrateVDSCommand
by oourfali＠redhat.com 15 Jul '14

15 Jul '14

Oved Ourfali has submitted this change and it was merged. Change subject: core: minor cleanup in MigrateVDSCommand ...................................................................... core: minor cleanup in MigrateVDSCommand Change-Id: I55b4365d9a71a47b4a72f98bae649a363f43ef03 Bug-Url: https://bugzilla.redhat.com/1118187 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/MigrateVDSCommand.java 1 file changed, 40 insertions(+), 34 deletions(-) Approvals: Omer Frenkel: Looks good to me, approved Arik Hadas: Verified -- To view, visit http://gerrit.ovirt.org/29892 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I55b4365d9a71a47b4a72f98bae649a363f43ef03 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4.3 Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.4.3]: core: make migrate commands non-transactive
by oourfali＠redhat.com 15 Jul '14

15 Jul '14

Oved Ourfali has submitted this change and it was merged. Change subject: core: make migrate commands non-transactive ...................................................................... core: make migrate commands non-transactive Patch 94e48a02 removed the global transaction during RunVmCommand. Global transactions should also be removed from the migrate commands for the same reasons and because it prevents us from migrating multiple VMs from different hosts to the same host while one of the migrations is in its execute phase or start a VM on a host while another VM is migrating to that host and the migration is in its execute phase. Change-Id: I4894ebd5f062174f9780926a0ce6b499fcc0fc7b Bug-Url: https://bugzilla.redhat.com/1118187 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/InternalMigrateVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MigrateVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MigrateVmToServerCommand.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/MigrateVDSCommand.java 4 files changed, 38 insertions(+), 36 deletions(-) Approvals: Omer Frenkel: Looks good to me, approved Arik Hadas: Verified -- To view, visit http://gerrit.ovirt.org/29893 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4894ebd5f062174f9780926a0ce6b499fcc0fc7b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4.3 Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.4.3]: packaging: spec: updated deps
by oourfali＠redhat.com 15 Jul '14

15 Jul '14

Oved Ourfali has submitted this change and it was merged. Change subject: packaging: spec: updated deps ...................................................................... packaging: spec: updated deps Updated dependencies Change-Id: If059609fd1ce0e93f770e20f2428764a2bdb05e3 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> (cherry picked from commit 0a6c1a3d02e136edb82d1e26e4291d1f5317c985) --- M ovirt-engine.spec.in 1 file changed, 10 insertions(+), 7 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/30057 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If059609fd1ce0e93f770e20f2428764a2bdb05e3 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4.3 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.4.3]: packaging: spec: updated tools dependencies
by oourfali＠redhat.com 15 Jul '14

15 Jul '14

Oved Ourfali has submitted this change and it was merged. Change subject: packaging: spec: updated tools dependencies ...................................................................... packaging: spec: updated tools dependencies Removed log-collector requirement making it optional. Change-Id: I8036e9d70d5337f79df1602d65a4f7f943fb05c3 Bug-Url: https://bugzilla.redhat.com/1103876 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-engine.spec.in 1 file changed, 0 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/30058 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8036e9d70d5337f79df1602d65a4f7f943fb05c3 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4.3 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.4]: packaging: spec: updated tools dependencies
by oourfali＠redhat.com 15 Jul '14

15 Jul '14

Oved Ourfali has submitted this change and it was merged. Change subject: packaging: spec: updated tools dependencies ...................................................................... packaging: spec: updated tools dependencies Removed log-collector requirement making it optional. Change-Id: I8036e9d70d5337f79df1602d65a4f7f943fb05c3 Bug-Url: https://bugzilla.redhat.com/1103876 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-engine.spec.in 1 file changed, 0 insertions(+), 1 deletion(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/30053 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8036e9d70d5337f79df1602d65a4f7f943fb05c3 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0