- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: webadmin: CPU Type and CPU Architecture fields in Cluster di...
by tjelinek＠redhat.com 10 Nov '14

10 Nov '14

Tomas Jelinek has submitted this change and it was merged. Change subject: webadmin: CPU Type and CPU Architecture fields in Cluster dialog fix ...................................................................... webadmin: CPU Type and CPU Architecture fields in Cluster dialog fix * CPU Architecture listbox is placed right above the CPU Type because they are related. (ClusterPopupView.ui.xml) * CPU Architecture field doesn't disappear when CPU Type is selected. (Bug fix). (ClusterPopupView.java) * CPU Architecture filters content of CPU type listbox. (ClusterModel.java, FilteredListModel.java) Bug-Url: https://bugzilla.redhat.com/1159777 Change-Id: I4c0452426f58f09703382495a9527fdbe859d80e Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com> --- A frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/FilteredListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterModel.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterPopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterPopupView.ui.xml 4 files changed, 123 insertions(+), 17 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Jakub Niedermertl: Verified -- To view, visit http://gerrit.ovirt.org/34816 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4c0452426f58f09703382495a9527fdbe859d80e Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: Adding host to DB before calling provision
by yzaslavs＠redhat.com 10 Nov '14

10 Nov '14

Yair Zaslavsky has submitted this change and it was merged. Change subject: Adding host to DB before calling provision ...................................................................... Adding host to DB before calling provision When calling provision foreman expect the engine's host-id to allow calling the reinstallation after provisioning. Engine generates host-id while saving host info to db, therefore this patch switches between the DB. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1158588 Change-Id: I635d87d09eec0dd052d86329d2b6b7f0a1658836 Signed-off-by: Yaniv Bronhaim <ybronhei(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVdsCommand.java 1 file changed, 11 insertions(+), 11 deletions(-) Approvals: Yaniv Bronhaim: Verified Yair Zaslavsky: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34860 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I635d87d09eec0dd052d86329d2b6b7f0a1658836 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(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:[ver2] fail the build if osinfo-defaults.propertie...
by tnisan＠redhat.com 10 Nov '14

10 Nov '14

Tal Nisan has submitted this change and it was merged. Change subject: packaging:[ver2] fail the build if osinfo-defaults.properties fails to parse ...................................................................... packaging:[ver2] fail the build if osinfo-defaults.properties fails to parse check the properties file we ship with the exact code that checks it at runtime when the engine loads. this test is hooked into the packaging process and must fail it. Change-Id: Icaecda7d278fa898838f04547d5ab44e5ca6b3d9 Bug-Url: https://bugzilla.redhat.com/1056965 Signed-off-by: Roy Golan <rgolan(a)redhat.com> --- M build/validations/pom.xml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me, approved Roy Golan: Verified -- To view, visit http://gerrit.ovirt.org/34959 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icaecda7d278fa898838f04547d5ab44e5ca6b3d9 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(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]: webadmin, userportal: Remove explicit include for generated s...
by vszocs＠redhat.com 10 Nov '14

10 Nov '14

Vojtech Szocs has submitted this change and it was merged. Change subject: webadmin,userportal: Remove explicit include for generated sources ...................................................................... webadmin,userportal: Remove explicit include for generated sources This patch builds on top of http://gerrit.ovirt.org/#/c/34402/ and ensures that everything under target/generated-sources/gwt directory is added to project's resources. Change-Id: Iaa38ee88c655e6b93c2a53dda34d341c961e83ae Signed-off-by: Vojtech Szocs <vszocs(a)redhat.com> --- M frontend/webadmin/modules/frontend/pom.xml 1 file changed, 0 insertions(+), 3 deletions(-) Approvals: Alexander Wels: Verified; Looks good to me, approved Vojtech Szocs: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34708 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iaa38ee88c655e6b93c2a53dda34d341c961e83ae Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(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 jenkins[master]: build vdsm-jsonrpc-java 3.5 on el7
by dcaroest＠redhat.com 10 Nov '14

10 Nov '14

David Caro has submitted this change and it was merged. Change subject: build vdsm-jsonrpc-java 3.5 on el7 ...................................................................... build vdsm-jsonrpc-java 3.5 on el7 Change-Id: I81f2ce722a0229af2262ac0e4649b9351badb87e Bug-Url: https://bugzilla.redhat.com/1159293 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/yaml/jobs/vdsm-jsonrpc-java/vdsm-jsonrpc-java_create-rpms.yaml 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Eyal Edri: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34923 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I81f2ce722a0229af2262ac0e4649b9351badb87e Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(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: oVirt Jenkins CI Server

1 0

Change in jenkins[master]: changed jpackage mirror
by dcaroest＠redhat.com 10 Nov '14

10 Nov '14

David Caro has submitted this change and it was merged. Change subject: changed jpackage mirror ...................................................................... changed jpackage mirror ftp://jpackage.hmdc.harvard.edu is not mirroring jpackage repo anymore moved to another mirror. Related-To: http://jenkins.ovirt.org/job/repos_3.5_check-closure_merged/DISTRIBUTION=ce… Change-Id: I3f2af8a3e9e97f7b4025b35de1c181ad9142d20f Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/packaging/repo_closure_check.sh 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified Ewoud Kohl van Wijngaarden: Looks good to me, but someone else must approve David Caro: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34849 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3f2af8a3e9e97f7b4025b35de1c181ad9142d20f Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Ewoud Kohl van Wijngaarden <ewoud(a)kohlvanwijngaarden.nl> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Paz Dangur <pdangur(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>

1 0

Change in ovirt-reports[master]: packaging: spec: add support for EL7 builds
by sradco＠redhat.com 10 Nov '14

10 Nov '14

Shirly Radco has submitted this change and it was merged. Change subject: packaging: spec: add support for EL7 builds ...................................................................... packaging: spec: add support for EL7 builds - add support for EL7 builds - avoid to require both java and java-1.7.0-openjdk since java is already provided by java-1.7.0-openjdk Change-Id: Ia3bf3df726a5092637e2e0b8e2dd95625f7f89da Bug-Url: https://bugzilla.redhat.com/1159293 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-engine-reports.spec.in 1 file changed, 6 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified Shirly Radco: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34913 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia3bf3df726a5092637e2e0b8e2dd95625f7f89da Gerrit-PatchSet: 1 Gerrit-Project: ovirt-reports Gerrit-Branch: master 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: Shirly Radco <sradco(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-reports[ovirt-engine-reports-3.5]: packaging: spec: add support for EL7 builds
by sradco＠redhat.com 10 Nov '14

10 Nov '14

Shirly Radco has submitted this change and it was merged. Change subject: packaging: spec: add support for EL7 builds ...................................................................... packaging: spec: add support for EL7 builds - add support for EL7 builds - avoid to require both java and java-1.7.0-openjdk since java is already provided by java-1.7.0-openjdk Change-Id: Ia3bf3df726a5092637e2e0b8e2dd95625f7f89da Bug-Url: https://bugzilla.redhat.com/1159293 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-engine-reports.spec.in 1 file changed, 6 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified Shirly Radco: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34914 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia3bf3df726a5092637e2e0b8e2dd95625f7f89da Gerrit-PatchSet: 1 Gerrit-Project: ovirt-reports Gerrit-Branch: ovirt-engine-reports-3.5 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: Shirly Radco <sradco(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-engine[ovirt-engine-3.5]: webadmin: glance domain - fix images tab
by tnisan＠redhat.com 09 Nov '14

09 Nov '14

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: glance domain - fix images tab ...................................................................... webadmin: glance domain - fix images tab Glance storage domain are unattached, hence, shouldn't be checked against Active status (change has been introduced in change-Id Idb29eb14a07a8084b62bda5ac41769ce964bac16 to prevent query on inactive domains). Change-Id: I5cdad6f443b19ad240b3c60c0f8f62798aca7bd8 Bug-Url: https://bugzilla.redhat.com/1161941 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageIsoListModel.java 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Tal Nisan: Looks good to me, approved Daniel Erez: Verified -- To view, visit http://gerrit.ovirt.org/34977 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5cdad6f443b19ad240b3c60c0f8f62798aca7bd8 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Idan Shaby <ishaby(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]: webadmin: glance domain - fix images tab
by derez＠redhat.com 09 Nov '14

09 Nov '14

Daniel Erez has submitted this change and it was merged. Change subject: webadmin: glance domain - fix images tab ...................................................................... webadmin: glance domain - fix images tab Glance storage domain are unattached, hence, shouldn't be checked against Active status (change has been introduced in change-Id Idb29eb14a07a8084b62bda5ac41769ce964bac16 to prevent query on inactive domains). Change-Id: I5cdad6f443b19ad240b3c60c0f8f62798aca7bd8 Bug-Url: https://bugzilla.redhat.com/1161941 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageIsoListModel.java 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Tal Nisan: Looks good to me, approved Daniel Erez: Verified -- To view, visit http://gerrit.ovirt.org/34976 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5cdad6f443b19ad240b3c60c0f8f62798aca7bd8 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Idan Shaby <ishaby(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