- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: engine: Common changes for Bonding modes 0, 5 and 6 should b...
by alkaplan＠redhat.com 05 Nov '15

05 Nov '15

Alona Kaplan has submitted this change and it was merged. Change subject: engine: Common changes for Bonding modes 0, 5 and 6 should be avoided for VM networks ...................................................................... engine: Common changes for Bonding modes 0, 5 and 6 should be avoided for VM networks Extracting hardcoded strings and values for bond modes. To be used in patches 46560 and 46251 Change-Id: Id5353a23ace44eef8ce6fd373b239a0d485d831d Bug-Url: https://bugzilla.redhat.com/1094842 Signed-off-by: Marcin Mirecki <mmirecki(a)redhat.com> --- A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/network/BondMode.java M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/core/Common.gwt.xml M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/dataprovider/AsyncDataProvider.java 3 files changed, 79 insertions(+), 14 deletions(-) Approvals: Marcin Mirecki: Verified Martin Mucha: Looks good to me, but someone else must approve Alona Kaplan: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/46797 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id5353a23ace44eef8ce6fd373b239a0d485d831d Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine-sdk[master]: sdk: Add spec file for Fedora 23
by juan.hernandez＠redhat.com 05 Nov '15

05 Nov '15

Juan Hernandez has submitted this change and it was merged. Change subject: sdk: Add spec file for Fedora 23 ...................................................................... sdk: Add spec file for Fedora 23 This patch adds an spec file for Fedora 23. Change-Id: I6e2eb7848adecd46bb0bd849129132cbe115d455 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- A spec.fc23.in 1 file changed, 599 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me Juan Hernandez: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/48112 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6e2eb7848adecd46bb0bd849129132cbe115d455 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: restapi: Document scheduling_policy.policy removal
by juan.hernandez＠redhat.com 05 Nov '15

05 Nov '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Document scheduling_policy.policy removal ...................................................................... restapi: Document scheduling_policy.policy removal Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1176820 Change-Id: I159b412bdea4b58701ffa1c4fc921503f0e776b5 Signed-off-by: Roman Mohr <rmohr(a)redhat.com> --- M backend/manager/modules/restapi/README.md 1 file changed, 19 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Looks good to me, approved Roman Mohr: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/47917 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I159b412bdea4b58701ffa1c4fc921503f0e776b5 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5-gluster]: gluster: disbale rebalance for non distribute volumes
by sabose＠redhat.com 05 Nov '15

05 Nov '15

Sahina Bose has submitted this change and it was merged. Change subject: gluster: disbale rebalance for non distribute volumes ...................................................................... gluster: disbale rebalance for non distribute volumes Disable gluster volume rebalance option for non distribute volume in UI. Change-Id: I9455294afe5f42c21922356b788079e9325c6714 Bug-Url: https://bugzilla.redhat.com/1261516 Signed-off-by: Ramesh Nachimuthu <rnachimu(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/volumes/VolumeListModel.java 1 file changed, 5 insertions(+), 2 deletions(-) Approvals: Sahina Bose: Looks good to me, approved; Passed CI tests Ramesh N: Verified -- To view, visit https://gerrit.ovirt.org/48101 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9455294afe5f42c21922356b788079e9325c6714 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5-gluster Gerrit-Owner: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine-sdk-java[master]: sdk: Add spec file for Fedora 23
by juan.hernandez＠redhat.com 05 Nov '15

05 Nov '15

Juan Hernandez has submitted this change and it was merged. Change subject: sdk: Add spec file for Fedora 23 ...................................................................... sdk: Add spec file for Fedora 23 This patch adds an spec file for Fedora 23. Change-Id: I5aafe1e22806cc5f8d1a9d3e398ca40f465eb2d7 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- A sdk/spec.fc23.in 1 file changed, 377 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/48114 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5aafe1e22806cc5f8d1a9d3e398ca40f465eb2d7 Gerrit-PatchSet: 2 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> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: packaging: Require WildFly 8 overlay package
by sbonazzo＠redhat.com 05 Nov '15

05 Nov '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: Require WildFly 8 overlay package ...................................................................... packaging: Require WildFly 8 overlay package Require ovirt-engine-wildfly-overlay which is compatible with WildFly 8. Change-Id: I3bedcc49497407018d52f8d76660154a79681750 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M ovirt-engine.spec.in 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me, approved; Passed CI tests Martin Peřina: Verified -- To view, visit https://gerrit.ovirt.org/48070 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3bedcc49497407018d52f8d76660154a79681750 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in jenkins[master]: Fixed fc23 chroots
by dcaro＠redhat.com 05 Nov '15

05 Nov '15

David Caro has submitted this change and it was merged. Change subject: Fixed fc23 chroots ...................................................................... Fixed fc23 chroots This still is not perfect, as we should be using dnf instead of yum, but allows us to work on any slave and does not seem to have any major impact Change-Id: I8fc0759068afd6e57272ff46cde384e99c6685d4 Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M mock_configs/fedora-23-x86_64.cfg M mock_configs/fedora-23-x86_64_proxied.cfg 2 files changed, 4 insertions(+), 2 deletions(-) Approvals: David Caro: Verified; Looks good to me; Passed CI tests; Ready for merge -- To view, visit https://gerrit.ovirt.org/48125 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8fc0759068afd6e57272ff46cde384e99c6685d4 Gerrit-PatchSet: 3 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Max Kovgan <mvk(a)redhat.com> Gerrit-Reviewer: Paz Dangur <pdangur(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in jenkins[master]: Fixed wrong echo
by dcaro＠redhat.com 05 Nov '15

05 Nov '15

David Caro has submitted this change and it was merged. Change subject: Fixed wrong echo ...................................................................... Fixed wrong echo When doing the echo the quotes made the resulting command not correct, now it's a bit better Change-Id: Iaf439572b6ef8d78ab22dc120efb75bdb427cb93 Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M mock_configs/mock_runner.sh 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me; Ready for merge David Caro: Verified; Passed CI tests; Ready for review -- To view, visit https://gerrit.ovirt.org/48124 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iaf439572b6ef8d78ab22dc120efb75bdb427cb93 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Max Kovgan <mvk(a)redhat.com> Gerrit-Reviewer: Paz Dangur <pdangur(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: new key dialog - tool-tip for UUID text-box
by derez＠redhat.com 05 Nov '15

05 Nov '15

Daniel Erez has submitted this change and it was merged. Change subject: webadmin: new key dialog - tool-tip for UUID text-box ...................................................................... webadmin: new key dialog - tool-tip for UUID text-box Cinder provider -> Authentication Keys -> New Dialog: added a tool-tip with explanation for UUID text-box. Change-Id: I288095208041393638101221f375cd298a4e1d3c Bug-Url: https://bugzilla.redhat.com/1273885 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/label/EnableableFormLabel.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/provider/ProviderSecretPopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/provider/ProviderSecretPopupView.ui.xml 4 files changed, 26 insertions(+), 3 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Daniel Erez: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/48107 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I288095208041393638101221f375cd298a4e1d3c Gerrit-PatchSet: 2 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: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: packaging: Require slf4j-jdk14 on Fedora
by sbonazzo＠redhat.com 05 Nov '15

05 Nov '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: Require slf4j-jdk14 on Fedora ...................................................................... packaging: Require slf4j-jdk14 on Fedora On EL6/7 slf4j package contains all part of slf4j library, but on Fedora slf4j package was split into several packagesi. Besides api part we need also jdk14 part (for engine command line tools) which is contained in slf4j-jdk14 package. This issue is not visibible on 3.6, because of this dependency: ovirt-engine -> spring-ldap -> springframework-ws -> springframework-webmvc -> tiles -> slf4j-jdk14 Engine dependency on spring-ldap was removed on master with removal of legacy kerbldap provider (engine-manage-domains). Change-Id: I0448fd16e9ee6b131070ba8d2960c1357ee05744 Bug-Url: https://bugzilla.redhat.com/1264670 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M ovirt-engine.spec.in 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Martin Peřina: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/48018 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0448fd16e9ee6b131070ba8d2960c1357ee05744 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0