July 2015 - Engine-commits - oVirt List Archives

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

Change in ovirt-engine[master]: webadmin: remove override from Custom Connection Parameters
by tnisan＠redhat.com 06 Jul '15

06 Jul '15

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: remove override from Custom Connection Parameters ...................................................................... webadmin: remove override from Custom Connection Parameters This checkbox is unnecessary. Change-Id: Ifb54d97a9eb06df07c443bde9ddc1b76948ced80 Signed-off-by: Vered Volansky <vvolansk(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/NfsStorageModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/NfsStorageView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/NfsStorageView.ui.xml 4 files changed, 25 insertions(+), 67 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Vered Volansky: Verified -- To view, visit https://gerrit.ovirt.org/42735 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifb54d97a9eb06df07c443bde9ddc1b76948ced80 Gerrit-PatchSet: 8 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 NfsStorageView.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 NfsStorageView.ui.xml syntax ...................................................................... webadmin: Change NfsStorageView.ui.xml syntax Replace HTML with gwt elements. Changed expander visibility to align with other fields in the view. Change-Id: I78a2c4b65534dd6058b2fddfcef1b9369e76f93a Signed-off-by: Vered Volansky <vvolansk(a)redhat.com> --- A frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/editor/generic/ShortEntityModelTextBoxEditor.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/NfsStorageView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/NfsStorageView.ui.xml 3 files changed, 134 insertions(+), 194 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Vered Volansky: Verified -- To view, visit https://gerrit.ovirt.org/42734 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I78a2c4b65534dd6058b2fddfcef1b9369e76f93a Gerrit-PatchSet: 8 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: Remove '*' and bold style from the NFS override de...
by tnisan＠redhat.com 06 Jul '15

06 Jul '15

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Remove '*' and bold style from the NFS override defaults alert ...................................................................... webadmin: Remove '*' and bold style from the NFS override defaults alert Change-Id: I0c27c97114d2e25e6a5f559405121a462da3f24e Signed-off-by: Vered Volansky <vvolansk(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/CommonApplicationConstants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/NfsStorageView.ui.xml 2 files changed, 1 insertion(+), 2 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Vered Volansky: Verified -- To view, visit https://gerrit.ovirt.org/42733 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0c27c97114d2e25e6a5f559405121a462da3f24e 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