July 2013 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: build: allow specifying EXTRA_BUILD_FLAGS for development
by oschreib＠redhat.com 11 Jul '13

11 Jul '13

Ofer Schreiber has submitted this change and it was merged. Change subject: build: allow specifying EXTRA_BUILD_FLAGS for development ...................................................................... build: allow specifying EXTRA_BUILD_FLAGS for development support easier handling of flags for developers as EXTRA_BUILD_FLAGS is available for all targets (all, install, all-dev, install-dev, clean). Change-Id: I12d59fad2ca46a67e0c3ec0baa6189a4fb7035bf Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M Makefile M README.developer 2 files changed, 16 insertions(+), 8 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16395 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I12d59fad2ca46a67e0c3ec0baa6189a4fb7035bf Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: More appropriate edit message
by ofrenkel＠redhat.com 11 Jul '13

11 Jul '13

Omer Frenkel has submitted this change and it was merged. Change subject: engine: More appropriate edit message ...................................................................... engine: More appropriate edit message Log messages on edit changed from '<entity> parameters changed' to '<entity> configuration changed' Change-Id: Ia23f6217c43ecfd09531aef1956556877810f665 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=916634 Signed-off-by: Martin Betak <mbetak(a)redhat.com> --- M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties 1 file changed, 10 insertions(+), 10 deletions(-) Approvals: Omer Frenkel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16588 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia23f6217c43ecfd09531aef1956556877810f665 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>

1 0

Change in ovirt-engine[master]: host-deploy: supress tar timestamp warnings
by yzaslavs＠redhat.com 11 Jul '13

11 Jul '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: host-deploy: supress tar timestamp warnings ...................................................................... host-deploy: supress tar timestamp warnings Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=982733 Change-Id: I0b5313dd1fd5840afebf1ceeb4673ca9e3a0831e Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16640 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0b5313dd1fd5840afebf1ceeb4673ca9e3a0831e Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: doc: rename engine_dao database to engine_dao_tests
by yzaslavs＠redhat.com 11 Jul '13

11 Jul '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: doc: rename engine_dao database to engine_dao_tests ...................................................................... doc: rename engine_dao database to engine_dao_tests Change-Id: I8e71806a4fe336797c9ae92a08b746884337bd80 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M README.developer 1 file changed, 6 insertions(+), 4 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16308 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8e71806a4fe336797c9ae92a08b746884337bd80 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: packagine: setup: Keep md5 sums of entire conf files
by oschreib＠redhat.com 11 Jul '13

11 Jul '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packagine: setup: Keep md5 sums of entire conf files ...................................................................... packagine: setup: Keep md5 sums of entire conf files engine-setup keeps the md5 hash of configured files so that on cleanup we can know if it's safe to delete them or some manual change was made and we should keep them. This change makes the code keep the md5 hash of the entire file, instead of 1000 bytes before. Change-Id: Ie8dfc2f23d1a395dfb35d12fce97896df057816f Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-remove/files/simple.py M packaging/setup/plugins/ovirt-engine-setup/core/uninstall.py 2 files changed, 12 insertions(+), 2 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16681 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie8dfc2f23d1a395dfb35d12fce97896df057816f Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: setup: allow pki files removal
by oschreib＠redhat.com 11 Jul '13

11 Jul '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: setup: allow pki files removal ...................................................................... packaging: setup: allow pki files removal Allow pki files removal using engine-cleanup-2 after upgrading from legacy using engine-setup-2 Change-Id: Ib02123e8757fa7664c4df3e50b22019d0acb7597 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/legacy/__init__.py A packaging/setup/plugins/ovirt-engine-setup/legacy/ca.py 2 files changed, 79 insertions(+), 0 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16603 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib02123e8757fa7664c4df3e50b22019d0acb7597 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: setup: remove early statement from /api
by oschreib＠redhat.com 11 Jul '13

11 Jul '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: setup: remove early statement from /api ...................................................................... packaging: setup: remove early statement from /api The current implementation incorrectly users 'early' statement for /api links in apache configuration, which does not remove 'Expect' headers as expected. The new implementation removes the 'early' statement fixing the issue. Change-Id: I7f0618caec59c2da894f242e230b23ebc7847b04 Bug-Url: https://bugzilla.redhat.com/849442 Signed-off-by: Alex Lourie <alourie(a)redhat.com> --- M packaging/conf/ovirt-engine-proxy.conf.v2.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16563 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7f0618caec59c2da894f242e230b23ebc7847b04 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: added DB validation functions
by oschreib＠redhat.com 11 Jul '13

11 Jul '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: added DB validation functions ...................................................................... packaging: added DB validation functions Change-Id: I25979acbf54d980168be929638ff4aed800bf6d3 Signed-off-by: Alex Lourie <alourie(a)redhat.com> --- M packaging/fedora/setup/engine-upgrade.py M packaging/setup/ovirt_engine_setup/constants.py A packaging/setup/plugins/ovirt-engine-setup/upgrade/__init__.py A packaging/setup/plugins/ovirt-engine-setup/upgrade/async_tasks_map.py A packaging/setup/plugins/ovirt-engine-setup/upgrade/asynctasks.py A packaging/setup/plugins/ovirt-engine-setup/upgrade/dbvalidations.py 6 files changed, 740 insertions(+), 1 deletion(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/15970 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I25979acbf54d980168be929638ff4aed800bf6d3 Gerrit-PatchSet: 22 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: setup: Changed stage for temp pgpass
by oschreib＠redhat.com 11 Jul '13

11 Jul '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: setup: Changed stage for temp pgpass ...................................................................... packaging: setup: Changed stage for temp pgpass Currently temp pgpass file is created in misc stage. This causes problems for upgrade, when db validations must be run in validation stage. The new implementation moves creation of the temp pgpass file to validation stage, keeping its name and allowing other validation steps to do work when the file is available. Change-Id: Ib9fc158a98cfdf36a1b812a4022298dc6e011d05 Signed-off-by: Alex Lourie <alourie(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-common/db/pgpass.py M packaging/setup/plugins/ovirt-engine-remove/db/clear.py M packaging/setup/plugins/ovirt-engine-setup/config/database.py M packaging/setup/plugins/ovirt-engine-setup/db/schema.py 4 files changed, 1 insertion(+), 10 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16466 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib9fc158a98cfdf36a1b812a4022298dc6e011d05 Gerrit-PatchSet: 10 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: spec: missing dep on ovirt-host-deploy
by oschreib＠redhat.com 11 Jul '13

11 Jul '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: spec: missing dep on ovirt-host-deploy ...................................................................... packaging: spec: missing dep on ovirt-host-deploy Added missing dependency on ovirt-host-deploy avoiding Internal error: cannot import name hardware during upgrade if AIO is enabled. Change-Id: Ic9835b126611b399aaeb7fdd263bf1935a4235f5 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-engine.spec.in M packaging/setup/plugins/ovirt-engine-setup/all-in-one/cpu.py 2 files changed, 2 insertions(+), 3 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16614 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic9835b126611b399aaeb7fdd263bf1935a4235f5 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0