May 2015 - Engine-commits - oVirt List Archives

Change in jenkins[master]: add python-six dep to vdsm build jobs
by dcaroest＠redhat.com 07 May '15

07 May '15

David Caro has submitted this change and it was merged. Change subject: add python-six dep to vdsm build jobs ...................................................................... add python-six dep to vdsm build jobs Fixing breakage introduced by vdsm commit 872ca9dff0a36021b7c6dd2603bb42f5606e600e adding python-six build dependency to VDSM. Change-Id: I0c9800da0361ee59e9d0cafba9c6f29f44cb5f08 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/yaml/jobs/vdsm/vdsm_create-rpms.yaml M jobs/confs/yaml/jobs/vdsm/vdsm_create_libgfapi-rpms.yaml 2 files changed, 2 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Ready for merge Jenkins CI: Looks good to me David Caro: Looks good to me -- To view, visit https://gerrit.ovirt.org/40654 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0c9800da0361ee59e9d0cafba9c6f29f44cb5f08 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: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: resapi: adding PCI-passthrough vnic type
by alkaplan＠redhat.com 07 May '15

07 May '15

Alona Kaplan has submitted this change and it was merged. Change subject: resapi: adding PCI-passthrough vnic type ...................................................................... resapi: adding PCI-passthrough vnic type PCI-passthrough vnic type means the vnic should be connected directly to a VF. Change-Id: Ia83b405017cc6b5f3a908cb342133c99262cdb5f Signed-off-by: Alona Kaplan <alkaplan(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/model/NicInterface.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/NicMapper.java 2 files changed, 5 insertions(+), 1 deletion(-) Approvals: Juan Hernandez: Looks good to me, approved Alona Kaplan: Verified -- To view, visit https://gerrit.ovirt.org/39532 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia83b405017cc6b5f3a908cb342133c99262cdb5f Gerrit-PatchSet: 24 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: engine: protection in case iommu group is null
by alkaplan＠redhat.com 07 May '15

07 May '15

Alona Kaplan has submitted this change and it was merged. Change subject: engine: protection in case iommu group is null ...................................................................... engine: protection in case iommu group is null Change-Id: I0493ba0f3fbab7776169d867e8b7100f8784db74 Signed-off-by: Alona Kaplan <alkaplan(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/VfSchedulerImpl.java 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Alona Kaplan: Verified Jenkins CI: Looks good to me, but someone else must approve Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/40645 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0493ba0f3fbab7776169d867e8b7100f8784db74 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: webadmin: blocking unlinking of passthrough vnic
by alkaplan＠redhat.com 07 May '15

07 May '15

Alona Kaplan has submitted this change and it was merged. Change subject: webadmin: blocking unlinking of passthrough vnic ...................................................................... webadmin: blocking unlinking of passthrough vnic Change-Id: Icad94250b042dfe9a668b7e415dc7c7847f18705 Signed-off-by: Alona Kaplan <alkaplan(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmInterfaceModel.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java 2 files changed, 21 insertions(+), 2 deletions(-) Approvals: Alona Kaplan: Verified; Looks good to me, approved Jenkins CI: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/38890 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icad94250b042dfe9a668b7e415dc7c7847f18705 Gerrit-PatchSet: 24 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: webadmin: sr-iov validation of profile in vnic dialog
by alkaplan＠redhat.com 07 May '15

07 May '15

Alona Kaplan has submitted this change and it was merged. Change subject: webadmin: sr-iov validation of profile in vnic dialog ...................................................................... webadmin: sr-iov validation of profile in vnic dialog 1. Passthrough profile can be attached 1.1 just if it is compatable with the cluster version. 1.2 the vnic type is pci-passthrough. 2. non-passthrough profile cannor be used if the vnic type is pci-passthrough. Change-Id: Ib2d0d834d0bab6e2490279fef8f1140edcdfdcb1 Signed-off-by: Alona Kaplan <alkaplan(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmInterfaceModel.java A frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/validation/VnicProfileValidation.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIMessages.java 3 files changed, 63 insertions(+), 1 deletion(-) Approvals: Alona Kaplan: Verified Gilad Chaplik: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/36101 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib2d0d834d0bab6e2490279fef8f1140edcdfdcb1 Gerrit-PatchSet: 33 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: webadmin: add passthrough property to vnic profile
by alkaplan＠redhat.com 07 May '15

07 May '15

Alona Kaplan has submitted this change and it was merged. Change subject: webadmin: add passthrough property to vnic profile ...................................................................... webadmin: add passthrough property to vnic profile Add 'passthrough' property to vnic profile dialog. Change-Id: I8d5d34b3700301d3557706905fd8b07b48d37018 Signed-off-by: Alona Kaplan <alkaplan(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/profiles/EditVnicProfileModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/profiles/NewVnicProfileModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/profiles/VnicProfileModel.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.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/profile/VnicProfilePopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/profile/VnicProfilePopupView.ui.xml 7 files changed, 76 insertions(+), 16 deletions(-) Approvals: Alona Kaplan: Verified; Looks good to me, approved Objections: Jenkins CI: Fails -- To view, visit https://gerrit.ovirt.org/35946 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8d5d34b3700301d3557706905fd8b07b48d37018 Gerrit-PatchSet: 36 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in jenkins[master]: uses yum-deprecated where available
by dcaroest＠redhat.com 07 May '15

07 May '15

David Caro has submitted this change and it was merged. Change subject: uses yum-deprecated where available ...................................................................... uses yum-deprecated where available F22 moved from yum to dnf, leaving yum as yum-deprecated. Change-Id: I64ad89d1c8202f3da00763cc93fca6178864899f Related-To: http://bugzilla.redhat.com/1211978 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/shell-scripts/mock_install.sh 1 file changed, 6 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Ready for review Jenkins CI: Looks good to me, but someone else must approve David Caro: Looks good to me, approved; Ready for merge -- To view, visit https://gerrit.ovirt.org/40643 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I64ad89d1c8202f3da00763cc93fca6178864899f Gerrit-PatchSet: 3 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: Jenkins CI Gerrit-Reviewer: Sagi Shnaidman <sshnaidm(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine-sdk-java[master]: sdk: Use a dot to separate release number and date
by juan.hernandez＠redhat.com 07 May '15

07 May '15

Juan Hernandez has submitted this change and it was merged. Change subject: sdk: Use a dot to separate release number and date ...................................................................... sdk: Use a dot to separate release number and date Currently the top level Makefile of the project generates the RPM release tag adding the date and short git hash, but it does so without adding a separator. This means that if the release number is 0.8 and the date and git hash are 20150506.git012345 the complete release tag will be 0.820150506.git012345. This makes an incorrect release tag. To solve that issue this patch modifies the top level Makefile so that it adds a dot to separate the release number and the date, thus generating a release tag like 0.8.20150506.git012345. Change-Id: Ia6c4a6ad8a186d6308b1cf74fbe2e309b0388488 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M Makefile 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Juan Hernandez: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/40603 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia6c4a6ad8a186d6308b1cf74fbe2e309b0388488 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(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]: restapi: Fix typo in import image RSDL metadata
by juan.hernandez＠redhat.com 07 May '15

07 May '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Fix typo in import image RSDL metadata ...................................................................... restapi: Fix typo in import image RSDL metadata The action that imports an image is documented using "{images:id}", in plural, but the actual URL is "image", in singular. Due to this typo the metadata for this action isn't added to the generated RSDL document. Change-Id: I84969f455a3d21d3a822675d704fd419551ac4fa Bug-Url: https://bugzilla.redhat.com/1218935 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Juan Hernandez: Verified Daniel Erez: Looks good to me, approved Allon Mureinik: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/40588 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I84969f455a3d21d3a822675d704fd419551ac4fa Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: packaging: Fix OpenJDK dependency
by sbonazzo＠redhat.com 07 May '15

07 May '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: Fix OpenJDK dependency ...................................................................... packaging: Fix OpenJDK dependency Fixes dependency on OpenJDK 1.7 by requiring version >= 1.7.0 and < 1.8.0. This should fix issues with new oVirt installation on distributions with default OpenJDK 1.8 Change-Id: I94332ec92646dcb4bee967eb4b80bcfe1bcab6c2 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 Martin Peřina: Verified Jenkins CI: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/40615 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I94332ec92646dcb4bee967eb4b80bcfe1bcab6c2 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