January 2014 - Engine-commits - oVirt List Archives

Change in mom[master]: build: Fix %install on < F20
by alitke＠redhat.com 10 Jan '14

10 Jan '14

Adam Litke has submitted this change and it was merged. Change subject: build: Fix %install on < F20 ...................................................................... build: Fix %install on < F20 The setup.py script always installs documentation into an unversioned doc dir so adjust the spec file to look in the right place when manually installing files. Change-Id: Ia6aaea13996675b12f3ccc8c8e5eeda6a278ca57 Signed-off-by: Adam Litke <alitke(a)redhat.com> Reviewed-on: http://gerrit.ovirt.org/23146 --- M mom.spec.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Adam Litke: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/23146 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia6aaea13996675b12f3ccc8c8e5eeda6a278ca57 Gerrit-PatchSet: 2 Gerrit-Project: mom Gerrit-Branch: master Gerrit-Owner: Adam Litke <alitke(a)redhat.com> Gerrit-Reviewer: Adam Litke <alitke(a)redhat.com>

1 0

Change in mom[master]: Sync spec file with official fedora spec file
by alitke＠redhat.com 10 Jan '14

10 Jan '14

Adam Litke has submitted this change and it was merged. Change subject: Sync spec file with official fedora spec file ...................................................................... Sync spec file with official fedora spec file This patch cleans up the in-tree spec file to make it as similar as possible to the spec file that builds official fedora packages. Change-Id: I1362ff2bc24eb79eed4728c00e3b7c0ddb34496c Signed-off-by: Adam Litke <alitke(a)redhat.com> Reviewed-on: http://gerrit.ovirt.org/23145 Reviewed-by: Martin Sivák <msivak(a)redhat.com> --- M mom.spec.in 1 file changed, 44 insertions(+), 10 deletions(-) Approvals: Adam Litke: Verified; Looks good to me, approved Martin Sivák: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/23145 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1362ff2bc24eb79eed4728c00e3b7c0ddb34496c Gerrit-PatchSet: 2 Gerrit-Project: mom Gerrit-Branch: master Gerrit-Owner: Adam Litke <alitke(a)redhat.com> Gerrit-Reviewer: Adam Litke <alitke(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com>

1 0

Change in ovirt-engine[master]: restapi: Remove Unnecessary Code
by juan.hernandez＠redhat.com 10 Jan '14

10 Jan '14

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Remove Unnecessary Code ...................................................................... restapi: Remove Unnecessary Code Change-Id: I6d84b63a6f954015240865ffbb3d5fb95bab0bb1 Signed-off-by: Ori Liel <oliel(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resteasy/json/JsonProvider.java 1 file changed, 0 insertions(+), 13 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21134 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6d84b63a6f954015240865ffbb3d5fb95bab0bb1 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mishka8520(a)yahoo.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in mom[master]: rpm: Depend on procps not procps-ng
by alitke＠redhat.com 10 Jan '14

10 Jan '14

Adam Litke has submitted this change and it was merged. Change subject: rpm: Depend on procps not procps-ng ...................................................................... rpm: Depend on procps not procps-ng MOM only uses a very basic ps invocation to find the pid of a running qemu process. Therefore the basic procps package is enough. This allows MOM to build on CentOS which has procps but not procps-ng Change-Id: I3b967cdbe63558f1c374a1e2ae0ba175533c98b5 Signed-off-by: Adam Litke <alitke(a)redhat.com> Reviewed-on: http://gerrit.ovirt.org/23137 Reviewed-by: Sandro Bonazzola <sbonazzo(a)redhat.com> Reviewed-by: Doron Fediuck <dfediuck(a)redhat.com> --- M mom.spec.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Adam Litke: Verified; Looks good to me, approved Doron Fediuck: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/23137 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3b967cdbe63558f1c374a1e2ae0ba175533c98b5 Gerrit-PatchSet: 2 Gerrit-Project: mom Gerrit-Branch: master Gerrit-Owner: Adam Litke <alitke(a)redhat.com> Gerrit-Reviewer: Adam Litke <alitke(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine-sdk-java[master]: sdk: Fix RPM dependencies for EL6
by juan.hernandez＠redhat.com 10 Jan '14

10 Jan '14

Juan Hernandez has submitted this change and it was merged. Change subject: sdk: Fix RPM dependencies for EL6 ...................................................................... sdk: Fix RPM dependencies for EL6 The names of some packages are different in EL6 and in Fedora. The current .spec uses the Fedora names. This patch changes the .spec so that it will work in both Fedora and EL6. Change-Id: Ia18cfd0e8be01286727f2ad16782972c4a50a3c5 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M ovirt-engine-sdk-java/ovirt-engine-sdk-java.spec.in 1 file changed, 15 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/23132 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia18cfd0e8be01286727f2ad16782972c4a50a3c5 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.3]: core : Plaintext user passwords in async_tasks database
by iheim＠redhat.com 10 Jan '14

10 Jan '14

Itamar Heim has submitted this change and it was merged. Change subject: core : Plaintext user passwords in async_tasks database ...................................................................... core : Plaintext user passwords in async_tasks database The VdcUser password is saved in async_tasks table in plain text This is a 3.3 issue, the problem has been fixed in 3.4 by directory refactoring Change-Id: I337818f6c2a376553409587a0982c1f9e8705bec Bug-Url: https://bugzilla.redhat.com/1017267 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/serialization/json/JsonObjectDeserializer.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/serialization/json/JsonObjectSerializer.java A backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/serialization/json/JsonVdcUserMixIn.java 3 files changed, 63 insertions(+), 0 deletions(-) Approvals: Ravi Nori: Verified Yair Zaslavsky: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/23116 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I337818f6c2a376553409587a0982c1f9e8705bec Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: setup: rpm-distro: allow use by other packages
by sbonazzo＠redhat.com 10 Jan '14

10 Jan '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: rpm-distro: allow use by other packages ...................................................................... packaging: setup: rpm-distro: allow use by other packages add a list into environment of groups/packages that should be updated. per entry group has higher priority than packages. add a list into environment of setup packages that must be updated manually. Change-Id: I3e4ad1c7f08d0a41742bc44c2ae469c6074cb768 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/constants.py A packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-common/distro-rpm/__init__.py A packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-common/distro-rpm/packages.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/distro-rpm/__init__.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/distro-rpm/packages.py 5 files changed, 513 insertions(+), 404 deletions(-) Approvals: Alon Bar-Lev: Verified Sandro Bonazzola: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/23122 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3e4ad1c7f08d0a41742bc44c2ae469c6074cb768 Gerrit-PatchSet: 2 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: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: setup: reusable postresql provisioning
by sbonazzo＠redhat.com 10 Jan '14

10 Jan '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: reusable postresql provisioning ...................................................................... packaging: setup: reusable postresql provisioning took the simple approach... each installer will use provisioning as-is, it may cause postgres multiple restarts and such, but it keeps loosely coupled components. Change-Id: Ieff824fa5115954925f8fd7cabf59d535f39e96c Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/constants.py A packaging/setup/ovirt_engine_setup/postgres.py A packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-common/provisioning/__init__.py A packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-common/provisioning/postgres.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/provisioning/postgres.py 5 files changed, 688 insertions(+), 507 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/23121 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ieff824fa5115954925f8fd7cabf59d535f39e96c Gerrit-PatchSet: 2 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: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: setup: add name for hostile services detection phase
by sbonazzo＠redhat.com 10 Jan '14

10 Jan '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: add name for hostile services detection phase ...................................................................... packaging: setup: add name for hostile services detection phase Change-Id: Ide6a22e008529c7d9d784912950f4107878c42fa Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/constants.py M packaging/setup/plugins/ovirt-engine-common/base/system/hostile_services.py 2 files changed, 2 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/23120 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ide6a22e008529c7d9d784912950f4107878c42fa Gerrit-PatchSet: 2 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: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: setup: add product option stage in customization
by sbonazzo＠redhat.com 10 Jan '14

10 Jan '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: add product option stage in customization ...................................................................... packaging: setup: add product option stage in customization Change-Id: I6ae2d79218745cf5d6cb58d1aa14a3fc95734dfd Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/constants.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/dialog/titles.py 2 files changed, 25 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/23119 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6ae2d79218745cf5d6cb58d1aa14a3fc95734dfd Gerrit-PatchSet: 2 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: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0