April 2015 - Engine-commits - oVirt List Archives

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.2]: packaging: setup: more descriptive error message for invalid...
by sbonazzo＠redhat.com 02 Apr '15

02 Apr '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: more descriptive error message for invalid storage domain ...................................................................... packaging: setup: more descriptive error message for invalid storage domain Making the invalid storage domain error message a bit more descriptive Change-Id: I45012fee44304f9bf4fe20ff2b15c32106fcfdc3 Bug-Url: https://bugzilla.redhat.com/1208485 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M src/plugins/ovirt-hosted-engine-setup/storage/storage.py 1 file changed, 8 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/39491 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I45012fee44304f9bf4fe20ff2b15c32106fcfdc3 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.2 Gerrit-Owner: Simone Tiraboschi <stirabos(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-setup[ovirt-hosted-engine-setup-1.2]: build: post ovirt-hosted-engine-setup-1.2.3 branching
by sbonazzo＠redhat.com 02 Apr '15

02 Apr '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: post ovirt-hosted-engine-setup-1.2.3 branching ...................................................................... build: post ovirt-hosted-engine-setup-1.2.3 branching Change-Id: I893163376d941a43aac54f2a9da111e6f6657f1b Bug-Url: https://bugzilla.redhat.com/1186161 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M .gitignore M configure.ac M ovirt-hosted-engine-setup.spec.in 3 files changed, 7 insertions(+), 5 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/38002 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I893163376d941a43aac54f2a9da111e6f6657f1b Gerrit-PatchSet: 5 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.2 Gerrit-Owner: Sandro Bonazzola <sbonazzo(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 Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-log-collector[ovirt-log-collector-3.5]: archive: reorganizing the final tarball archive
by sbonazzo＠redhat.com 02 Apr '15

02 Apr '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: archive: reorganizing the final tarball archive ...................................................................... archive: reorganizing the final tarball archive Adding a top level directory to the tarball archive to avoid any other element in the tarball root Change-Id: If14717e8a9bc790e12c84c1fbaa968d786bed96f Bug-Url: https://bugzilla.redhat.com/1171179 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M src/__main__.py 1 file changed, 34 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified -- To view, visit https://gerrit.ovirt.org/37900 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If14717e8a9bc790e12c84c1fbaa968d786bed96f Gerrit-PatchSet: 3 Gerrit-Project: ovirt-log-collector Gerrit-Branch: ovirt-log-collector-3.5 Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(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.5]: sos: hosts: collect hosted-engine data
by sbonazzo＠redhat.com 02 Apr '15

02 Apr '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: sos: hosts: collect hosted-engine data ...................................................................... sos: hosts: collect hosted-engine data If sos >= 3.2 is found on the host enable also the ovirt-hosted-engine plugin which is available since sos-3.2_alpha. sos 3.2 is available on Fedora and EL 7.1 and will be available in EL 6.7. Change-Id: I8e456262f2242d6aa73449e57c81246eaf824ab3 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1208113 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> (cherry picked from commit 4592c6c18354270cb80d0d3c682b49d27dffa96f) --- M src/__main__.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/39493 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8e456262f2242d6aa73449e57c81246eaf824ab3 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-log-collector Gerrit-Branch: ovirt-log-collector-3.5 Gerrit-Owner: Sandro Bonazzola <sbonazzo(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-log-collector[ovirt-log-collector-3.5]: build: post ovirt-log-collector-3.5.2 branching
by sbonazzo＠redhat.com 02 Apr '15

02 Apr '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: post ovirt-log-collector-3.5.2 branching ...................................................................... build: post ovirt-log-collector-3.5.2 branching Change-Id: I54e0c1f93c667a2398d11338efb3d4c3888971d4 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M configure.ac M ovirt-log-collector.spec.in 2 files changed, 4 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/37999 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I54e0c1f93c667a2398d11338efb3d4c3888971d4 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-log-collector Gerrit-Branch: ovirt-log-collector-3.5 Gerrit-Owner: Sandro Bonazzola <sbonazzo(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 Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: add @PostConstruct to postConstruct methods
by masayag＠redhat.com 02 Apr '15

02 Apr '15

Moti Asayag has submitted this change and it was merged. Change subject: core: add @PostConstruct to postConstruct methods ...................................................................... core: add @PostConstruct to postConstruct methods on newer JBoss (tested with EAP 6.3) the postConstruct methods are not called without the @PostConstruct this of course cause all the commands that are using this not to work. Change-Id: Ibf9cdd60457e60fa4c5257b0c0bc7e659bb7c704 Signed-off-by: Omer Frenkel <ofrenkel(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CloneVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommonVmPoolWithVmsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportVmCommandBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportVmFromConfigurationCommand.java 6 files changed, 12 insertions(+), 1 deletion(-) Approvals: Omer Frenkel: Verified Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/39477 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibf9cdd60457e60fa4c5257b0c0bc7e659bb7c704 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: frontend: Common.gwt.xml clean up
by tjelinek＠redhat.com 02 Apr '15

02 Apr '15

Tomas Jelinek has submitted this change and it was merged. Change subject: frontend: Common.gwt.xml clean up ...................................................................... frontend: Common.gwt.xml clean up 1. Replaces tabs with space and fix indentation 2. Removes no longer existent classes Change-Id: I195ffb1e7d111e0e799219c5da9d2de62e1eafcd Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/core/Common.gwt.xml 1 file changed, 289 insertions(+), 300 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Martin Peřina: Verified -- To view, visit https://gerrit.ovirt.org/39440 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I195ffb1e7d111e0e799219c5da9d2de62e1eafcd Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(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[ovirt-engine-3.5]: packaging: setup: raising on invalid application mode values
by piotr.kliczewski＠gmail.com 02 Apr '15

02 Apr '15

Piotr Kliczewski has submitted this change and it was merged. Change subject: packaging: setup: raising on invalid application mode values ...................................................................... packaging: setup: raising on invalid application mode values It was treating every invalid application mode values as virtOnly. Checking it for a more consistent behavior. The default value for ApplicationMode at DB creation is AllModes = 255; making it more explicit. Change-Id: Ic47e171ddc76a9881e4a915f62dbae762a72f9f8 Bug-Url: https://bugzilla.redhat.com/1208440 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/config/appmode.py 1 file changed, 26 insertions(+), 20 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified oVirt Jenkins CI Server: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/39392 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic47e171ddc76a9881e4a915f62dbae762a72f9f8 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(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

Change in ovirt-engine[master]: engine: Cleanup untyped warning in ProviderProxyFactory
by masayag＠redhat.com 02 Apr '15

02 Apr '15

Moti Asayag has submitted this change and it was merged. Change subject: engine: Cleanup untyped warning in ProviderProxyFactory ...................................................................... engine: Cleanup untyped warning in ProviderProxyFactory Change-Id: I3bfacc5c1ffdd5a0297af42cd2951ca3054431c7 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/provider/ProviderProxyFactory.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Mike Kolesnik: Looks good to me, approved Moti Asayag: Verified -- To view, visit https://gerrit.ovirt.org/39473 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3bfacc5c1ffdd5a0297af42cd2951ca3054431c7 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: Add query for reporting host errata stats
by masayag＠redhat.com 02 Apr '15

02 Apr '15

Moti Asayag has submitted this change and it was merged. Change subject: engine: Add query for reporting host errata stats ...................................................................... engine: Add query for reporting host errata stats The query reports a summary for the host errata categorized by the errata type. Change-Id: Idaf39c5f1db9796babc97c22aad9754564f2d729 Bug-Url: https://bugzilla.redhat.com/1157377 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetErrataCountsForHostQuery.java A backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/GetErrataCountsForHostQueryTest.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/ErrataCounts.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/VdcQueryType.java M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/core/Common.gwt.xml 5 files changed, 333 insertions(+), 0 deletions(-) Approvals: Moti Asayag: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/39329 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idaf39c5f1db9796babc97c22aad9754564f2d729 Gerrit-PatchSet: 10 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0