- Engine-commits - oVirt List Archives

Change in ovirt-node-plugin-vdsm[master]: ovirt-register: Integration with ovirt-register tool
by dougsland＠redhat.com 06 Jul '15

06 Jul '15

Douglas Schilling Landgraf has submitted this change and it was merged. Change subject: ovirt-register: Integration with ovirt-register tool ...................................................................... ovirt-register: Integration with ovirt-register tool Replace vdsm-tool register call Change-Id: I875cddf7ac54d5a57922904a888bfdf8dfe3cee6 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> Reviewed-on: https://gerrit.ovirt.org/43180 Continuous-Integration: Jenkins CI --- M autoinstall/ovirt-node-plugin-vdsm-autoreg M ovirt-node-plugin-vdsm.spec.in M src/engine_page.py 3 files changed, 37 insertions(+), 33 deletions(-) Approvals: Douglas Schilling Landgraf: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/43180 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I875cddf7ac54d5a57922904a888bfdf8dfe3cee6 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: master Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5]: setup: Support force using a JRE using OVIRT_ENGINE_JAVA_HOM...
by tnisan＠redhat.com 06 Jul '15

06 Jul '15

Tal Nisan has submitted this change and it was merged. Change subject: setup: Support force using a JRE using OVIRT_ENGINE_JAVA_HOME_FORCE=1 ...................................................................... setup: Support force using a JRE using OVIRT_ENGINE_JAVA_HOME_FORCE=1 force using a custom JRE using VIRT_ENGINE_JAVA_HOME=/usr/java/jdk1.7.0_75/ \ OVIRT_ENGINE_JAVA_HOME_FORCE=1 \ bin/engine-setup --jboss-home Change-Id: I5f6c59fea689cff7ad4e8d33dad14e398a649ea6 Bug-Url: https://bugzilla.redhat.com/1238115 Signed-off-by: Roy Golan <rgolan(a)redhat.com> --- M packaging/bin/java-home 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests Roy Golan: Verified -- To view, visit https://gerrit.ovirt.org/43061 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5f6c59fea689cff7ad4e8d33dad14e398a649ea6 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Jenkins CI 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

1 0

Change in ovirt-engine[master]: setupNetworks: do not log policy reset stack trace
by masayag＠redhat.com 06 Jul '15

06 Jul '15

Moti Asayag has submitted this change and it was merged. Change subject: setupNetworks: do not log policy reset stack trace ...................................................................... setupNetworks: do not log policy reset stack trace Bug-Url: https://bugzilla.redhat.com/1235683 Change-Id: I24f77544d2be9a455dcc2393fb1a9a24aa0459c3 Signed-off-by: pkliczewski <piotr.kliczewski(a)gmail.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/FutureVDSCommand.java 1 file changed, 6 insertions(+), 0 deletions(-) Approvals: Piotr Kliczewski: Verified Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/43197 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I24f77544d2be9a455dcc2393fb1a9a24aa0459c3 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in jenkins[master]: ovirt-engine_create-rpms: add JPP devel el6 repo
by dcaroest＠redhat.com 06 Jul '15

06 Jul '15

David Caro has submitted this change and it was merged. Change subject: ovirt-engine_create-rpms: add JPP devel el6 repo ...................................................................... ovirt-engine_create-rpms: add JPP devel el6 repo Add JPP devel repo for el6 build. Solves: DEBUG util.py:378: Error: Package: plexus-cipher-1.7-3.jpp6.noarch (jpackage-6.0-generic) DEBUG util.py:378: Requires: sisu2 DEBUG util.py:378: Error: Package: maven-project-info-reports-plugin-2.1.2-1.jpp6.noarch (jpackage-6.0-generic) DEBUG util.py:378: Requires: maven-doxia-tools DEBUG util.py:378: Error: Package: maven3-3.0.3-8.jpp6.noarch (jpackage-6.0-generic) DEBUG util.py:378: Requires: sisu2 >= 2.1.1-2 DEBUG util.py:378: Error: Package: aether-1.13.1-2.jpp6.noarch (jpackage-6.0-generic) DEBUG util.py:378: Requires: ops4j-pax-swissbox DEBUG util.py:378: Error: Package: maven-remote-resources-plugin-1.2.1-2.jpp6.noarch (jpackage-6.0-generic) DEBUG util.py:378: Requires: maven-shared-artifact-resolver DEBUG util.py:378: Error: Package: maven-checkstyle-plugin-2.5-2.jpp6.noarch (jpackage-6.0-generic) DEBUG util.py:378: Requires: maven-doxia-tools Change-Id: I66d09c0c411cccbc045db6b635d912e09e7da2e3 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/yaml/jobs/ovirt-engine/ovirt-engine_create-rpms.yaml 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Ready for review Jenkins CI: Passed CI tests David Caro: Looks good to me; Ready for merge -- To view, visit https://gerrit.ovirt.org/43224 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I66d09c0c411cccbc045db6b635d912e09e7da2e3 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: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: "Extend size by" appears when creating new VM disk
by tnisan＠redhat.com 06 Jul '15

06 Jul '15

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: "Extend size by" appears when creating new VM disk ...................................................................... webadmin: "Extend size by" appears when creating new VM disk When creating or editing a VM, clicking on "Create" under "Instance Images" opens the "New Virtual Disk" window. Since it's a new disk, this patch removes the "Extend size by(GB)" field from this window. Change-Id: I7d58e70a0d7ab953372badcb758fc1c3ee8f4621 Bug-Url: https://bugzilla.redhat.com/1212477 Signed-off-by: Idan Shaby <ishaby(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/InstanceImageLineModel.java 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Tal Nisan: Looks good to me, approved; Passed CI tests Idan Shaby: Verified -- To view, visit https://gerrit.ovirt.org/43223 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7d58e70a0d7ab953372badcb758fc1c3ee8f4621 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Idan Shaby <ishaby(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(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

1 0

Change in ovirt-engine[master]: webadmin: Display missing fields in import SD Dialog
by tnisan＠redhat.com 06 Jul '15

06 Jul '15

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Display missing fields in import SD Dialog ...................................................................... webadmin: Display missing fields in import SD Dialog Name, description and comment were missing (invisible) from the Import SD dislog. Now they are displayed. Change-Id: I1ffdfdbb7ae5575ee3a5e75a28ca60ef970b2c82 Signed-off-by: Vered Volansky <vvolansk(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java 1 file changed, 0 insertions(+), 3 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Vered Volansky: Verified -- To view, visit https://gerrit.ovirt.org/43066 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1ffdfdbb7ae5575ee3a5e75a28ca60ef970b2c82 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vered Volansky <vvolansk(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: webadmin: move activate storage domain checkbox down
by tnisan＠redhat.com 06 Jul '15

06 Jul '15

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: move activate storage domain checkbox down ...................................................................... webadmin: move activate storage domain checkbox down In the import SD dialog, "Activate Domain in Data Center" checkbox was moved down to the end of the view, since A. It never really belonged up there (in the upper section). B. Once the missing fields (name, description, comment) are restored in a future patch, there's no room for this checkbox in the upper section. Change-Id: Iec242b302e3659c7b3f0da4f12eda3dfce5b1fc9 Signed-off-by: Vered Volansky <vvolansk(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/StoragePopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/StoragePopupView.ui.xml 2 files changed, 2 insertions(+), 4 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Vered Volansky: Verified -- To view, visit https://gerrit.ovirt.org/43065 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iec242b302e3659c7b3f0da4f12eda3dfce5b1fc9 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vered Volansky <vvolansk(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: webadmin: Change GlusterStorageView.ui.xml syntax
by tnisan＠redhat.com 06 Jul '15

06 Jul '15

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Change GlusterStorageView.ui.xml syntax ...................................................................... webadmin: Change GlusterStorageView.ui.xml syntax Replaced HTML with gwt elements. Aligned fieslds with those of Posix, so that they don't appear to move when switching between views. VFS Type is displayed as disabled as opposed to just regular text before. Change-Id: Id4042db2643de20f7cb262c35fc7c26a7bdea005 Signed-off-by: Vered Volansky <vvolansk(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/GlusterStorageView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/GlusterStorageView.ui.xml 2 files changed, 40 insertions(+), 80 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Vered Volansky: Verified -- To view, visit https://gerrit.ovirt.org/42892 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id4042db2643de20f7cb262c35fc7c26a7bdea005 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vered Volansky <vvolansk(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: webadmin: Align NFS and Advanced Parameters expanders.
by tnisan＠redhat.com 06 Jul '15

06 Jul '15

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Align NFS and Advanced Parameters expanders. ...................................................................... webadmin: Align NFS and Advanced Parameters expanders. Change-Id: I5fb7ced6756e07cf77965646e93472b0b4c9dcf0 Signed-off-by: Vered Volansky <vvolansk(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/NfsStorageView.ui.xml M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/StoragePopupView.ui.xml 2 files changed, 4 insertions(+), 3 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Vered Volansky: Verified -- To view, visit https://gerrit.ovirt.org/42862 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5fb7ced6756e07cf77965646e93472b0b4c9dcf0 Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vered Volansky <vvolansk(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: webadmin: Change PosixStorageView.ui.xml syntax
by tnisan＠redhat.com 06 Jul '15

06 Jul '15

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Change PosixStorageView.ui.xml syntax ...................................................................... webadmin: Change PosixStorageView.ui.xml syntax Replace HTML with gwt elements. Signed-off-by: Vered Volansky <vvolansk(a)redhat.com> Change-Id: I7f1d3166d20c5db5d1d730dc5b167f04abd8be09 --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/NfsStorageView.ui.xml M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/PosixStorageView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/PosixStorageView.ui.xml M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/StoragePopupView.ui.xml 4 files changed, 44 insertions(+), 94 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Vered Volansky: Verified -- To view, visit https://gerrit.ovirt.org/42861 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7f1d3166d20c5db5d1d730dc5b167f04abd8be09 Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vered Volansky <vvolansk(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0