Mirantis OpenStack

Mellanox plugin is broken in nova because of the removal of vif_driver configuration from nova in Juno

Bug #1386858 reported by Gil Meir
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mirantis OpenStack
Fix Released
High
Sergey Nikitin
Mirantis OpenStack 7.0
6.0.x
Fix Released
High
Sergey Nikitin
Mirantis OpenStack 6.0-updates
6.1.x
Fix Released
High
Sergey Nikitin
Mirantis OpenStack 6.1
7.0.x
Fix Released
High
Sergey Nikitin
Mirantis OpenStack 7.0

Bug Description

The removal of vif_driver blocks Mellanox from configuring eswitchd as the vif_driver in nova.conf.

Attached is a patch for nova/virt/libvirt/vif.py + nova/virt/libvirt/designer.py (compute nodes).
The patch adds methods which are in use only when using mechanism_driver=mlnx.

Tags: nova mellanox
Revision history for this message
Gil Meir (gilmeir-d) wrote :
Nastya Urlapova (aurlapova)
Changed in fuel:
milestone: none → 6.0
importance: Undecided → High
Evgeniya Shumakher (eshumakher)
tags: added: mellanox
Changed in fuel:
assignee: nobody → MOS Nova (mos-nova)
Andriy Kurilin (andreykurilin)
Changed in fuel:
assignee: MOS Nova (mos-nova) → Sergey Nikitin (snikitin)
Sergey Nikitin (snikitin)
Changed in fuel:
status: New → In Progress
Revision history for this message
OSCI Robot (oscirobot) wrote :

RPM package nova-2014.2-fuel6.0.mira7.git.109c8d5.a3fd766.rpm has been built for project openstack/nova.
Changeset project: openstack/nova
changeset: https://review.fuel-infra.org/534
branch: master
subject: Fixed a problem with Mellanox plugin
status: patchset-created
NOTE: Changeset is not merged, created temporary package repository.
RPM repository URL: http://osci-obs.vm.mirantis.net:82/centos-fuel-master-534/centos

Revision history for this message
OSCI Robot (oscirobot) wrote :

DEB package nova_2014.2-fuel6.0~mira7+git.109c8d5.a3fd766.deb has been built for project openstack/nova.
Changeset project: openstack/nova
changeset: https://review.fuel-infra.org/534
branch: master
subject: Fixed a problem with Mellanox plugin
status: patchset-created
NOTE: Changeset is not merged, created temporary package repository.
DEB repository URL: http://osci-obs.vm.mirantis.net:82/ubuntu-fuel-master-534/ubuntu

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-main (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/133920

Revision history for this message
OSCI Robot (oscirobot) wrote :

RPM package nova has been built for project openstack/nova
Package version == 2014.2, package release == fuel6.0.mira7.git.c0c2396.a3fd766

Changeset: https://review.fuel-infra.org/749
project: openstack/nova
branch: master
author: Sergey Nikitin
committer: Sergey Nikitin
subject: Fixed a problem with Mellanox plugin
status: patchset-created

Files placed on repository:
openstack-nova-2014.2-fuel6.0.mira7.git.c0c2396.a3fd766.noarch.rpm
openstack-nova-api-2014.2-fuel6.0.mira7.git.c0c2396.a3fd766.noarch.rpm
openstack-nova-cells-2014.2-fuel6.0.mira7.git.c0c2396.a3fd766.noarch.rpm
openstack-nova-cert-2014.2-fuel6.0.mira7.git.c0c2396.a3fd766.noarch.rpm
openstack-nova-common-2014.2-fuel6.0.mira7.git.c0c2396.a3fd766.noarch.rpm
openstack-nova-compute-2014.2-fuel6.0.mira7.git.c0c2396.a3fd766.noarch.rpm
openstack-nova-conductor-2014.2-fuel6.0.mira7.git.c0c2396.a3fd766.noarch.rpm
openstack-nova-console-2014.2-fuel6.0.mira7.git.c0c2396.a3fd766.noarch.rpm
openstack-nova-doc-2014.2-fuel6.0.mira7.git.c0c2396.a3fd766.noarch.rpm
openstack-nova-network-2014.2-fuel6.0.mira7.git.c0c2396.a3fd766.noarch.rpm
openstack-nova-novncproxy-2014.2-fuel6.0.mira7.git.c0c2396.a3fd766.noarch.rpm
openstack-nova-objectstore-2014.2-fuel6.0.mira7.git.c0c2396.a3fd766.noarch.rpm
openstack-nova-scheduler-2014.2-fuel6.0.mira7.git.c0c2396.a3fd766.noarch.rpm
python-nova-2014.2-fuel6.0.mira7.git.c0c2396.a3fd766.noarch.rpm

NOTE: Changeset is not merged, created temporary package repository.
RPM repository URL: http://osci-obs.vm.mirantis.net:82/centos-fuel-master-749/centos

Revision history for this message
OSCI Robot (oscirobot) wrote :

RPM package nova has been built for project openstack/nova
Package version == 2014.2, package release == fuel6.0.mira11.git.f783393.93946b4

Changeset: https://review.fuel-infra.org/750
project: openstack/nova
branch: openstack-ci/fuel-6.0/2014.2
author: Sergey Nikitin
committer: Sergey Nikitin
subject: Fixed a problem with Mellanox plugin
status: patchset-created

Files placed on repository:
openstack-nova-2014.2-fuel6.0.mira11.git.f783393.93946b4.noarch.rpm
openstack-nova-api-2014.2-fuel6.0.mira11.git.f783393.93946b4.noarch.rpm
openstack-nova-cells-2014.2-fuel6.0.mira11.git.f783393.93946b4.noarch.rpm
openstack-nova-cert-2014.2-fuel6.0.mira11.git.f783393.93946b4.noarch.rpm
openstack-nova-common-2014.2-fuel6.0.mira11.git.f783393.93946b4.noarch.rpm
openstack-nova-compute-2014.2-fuel6.0.mira11.git.f783393.93946b4.noarch.rpm
openstack-nova-conductor-2014.2-fuel6.0.mira11.git.f783393.93946b4.noarch.rpm
openstack-nova-console-2014.2-fuel6.0.mira11.git.f783393.93946b4.noarch.rpm
openstack-nova-doc-2014.2-fuel6.0.mira11.git.f783393.93946b4.noarch.rpm
openstack-nova-network-2014.2-fuel6.0.mira11.git.f783393.93946b4.noarch.rpm
openstack-nova-novncproxy-2014.2-fuel6.0.mira11.git.f783393.93946b4.noarch.rpm
openstack-nova-objectstore-2014.2-fuel6.0.mira11.git.f783393.93946b4.noarch.rpm
openstack-nova-scheduler-2014.2-fuel6.0.mira11.git.f783393.93946b4.noarch.rpm
python-nova-2014.2-fuel6.0.mira11.git.f783393.93946b4.noarch.rpm

NOTE: Changeset is not merged, created temporary package repository.
RPM repository URL: http://osci-obs.vm.mirantis.net:82/centos-fuel-6.0-stable-750/centos

Revision history for this message
OSCI Robot (oscirobot) wrote :

DEB package nova has been built for project openstack/nova
Package version == 2014.2, package release == fuel6.0~mira7+git.c0c2396.a3fd766

Changeset: https://review.fuel-infra.org/749
project: openstack/nova
branch: master
author: Sergey Nikitin
committer: Sergey Nikitin
subject: Fixed a problem with Mellanox plugin
status: patchset-created

Files placed on repository:
nova-ajax-console-proxy_2014.2-fuel6.0~mira7+git.c0c2396.a3fd766_all.deb
nova-api-ec2_2014.2-fuel6.0~mira7+git.c0c2396.a3fd766_all.deb
nova-api-metadata_2014.2-fuel6.0~mira7+git.c0c2396.a3fd766_all.deb
nova-api-os-compute_2014.2-fuel6.0~mira7+git.c0c2396.a3fd766_all.deb
nova-api-os-volume_2014.2-fuel6.0~mira7+git.c0c2396.a3fd766_all.deb
nova-api_2014.2-fuel6.0~mira7+git.c0c2396.a3fd766_all.deb
nova-baremetal_2014.2-fuel6.0~mira7+git.c0c2396.a3fd766_all.deb
nova-cells_2014.2-fuel6.0~mira7+git.c0c2396.a3fd766_all.deb
nova-cert_2014.2-fuel6.0~mira7+git.c0c2396.a3fd766_all.deb
nova-common_2014.2-fuel6.0~mira7+git.c0c2396.a3fd766_all.deb
nova-compute-kvm_2014.2-fuel6.0~mira7+git.c0c2396.a3fd766_all.deb
nova-compute-libvirt_2014.2-fuel6.0~mira7+git.c0c2396.a3fd766_all.deb
nova-compute-lxc_2014.2-fuel6.0~mira7+git.c0c2396.a3fd766_all.deb
nova-compute-qemu_2014.2-fuel6.0~mira7+git.c0c2396.a3fd766_all.deb
nova-compute-vmware_2014.2-fuel6.0~mira7+git.c0c2396.a3fd766_all.deb
nova-compute-xen_2014.2-fuel6.0~mira7+git.c0c2396.a3fd766_all.deb
nova-compute_2014.2-fuel6.0~mira7+git.c0c2396.a3fd766_all.deb
nova-conductor_2014.2-fuel6.0~mira7+git.c0c2396.a3fd766_all.deb
nova-console_2014.2-fuel6.0~mira7+git.c0c2396.a3fd766_all.deb
nova-consoleauth_2014.2-fuel6.0~mira7+git.c0c2396.a3fd766_all.deb
nova-doc_2014.2-fuel6.0~mira7+git.c0c2396.a3fd766_all.deb
nova-network_2014.2-fuel6.0~mira7+git.c0c2396.a3fd766_all.deb
nova-novncproxy_2014.2-fuel6.0~mira7+git.c0c2396.a3fd766_all.deb
nova-objectstore_2014.2-fuel6.0~mira7+git.c0c2396.a3fd766_all.deb
nova-scheduler_2014.2-fuel6.0~mira7+git.c0c2396.a3fd766_all.deb
nova-spiceproxy_2014.2-fuel6.0~mira7+git.c0c2396.a3fd766_all.deb
nova-volume_2014.2-fuel6.0~mira7+git.c0c2396.a3fd766_all.deb
nova-xvpvncproxy_2014.2-fuel6.0~mira7+git.c0c2396.a3fd766_all.deb
python-nova_2014.2-fuel6.0~mira7+git.c0c2396.a3fd766_all.deb

NOTE: Changeset is not merged, created temporary package repository.
DEB repository URL: http://osci-obs.vm.mirantis.net:82/ubuntu-fuel-master-749/ubuntu

Revision history for this message
OSCI Robot (oscirobot) wrote :

DEB package nova has been built for project openstack/nova
Package version == 2014.2, package release == fuel6.0~mira11+git.f783393.93946b4

Changeset: https://review.fuel-infra.org/750
project: openstack/nova
branch: openstack-ci/fuel-6.0/2014.2
author: Sergey Nikitin
committer: Sergey Nikitin
subject: Fixed a problem with Mellanox plugin
status: patchset-created

Files placed on repository:
nova-ajax-console-proxy_2014.2-fuel6.0~mira11+git.f783393.93946b4_all.deb
nova-api-ec2_2014.2-fuel6.0~mira11+git.f783393.93946b4_all.deb
nova-api-metadata_2014.2-fuel6.0~mira11+git.f783393.93946b4_all.deb
nova-api-os-compute_2014.2-fuel6.0~mira11+git.f783393.93946b4_all.deb
nova-api-os-volume_2014.2-fuel6.0~mira11+git.f783393.93946b4_all.deb
nova-api_2014.2-fuel6.0~mira11+git.f783393.93946b4_all.deb
nova-baremetal_2014.2-fuel6.0~mira11+git.f783393.93946b4_all.deb
nova-cells_2014.2-fuel6.0~mira11+git.f783393.93946b4_all.deb
nova-cert_2014.2-fuel6.0~mira11+git.f783393.93946b4_all.deb
nova-common_2014.2-fuel6.0~mira11+git.f783393.93946b4_all.deb
nova-compute-kvm_2014.2-fuel6.0~mira11+git.f783393.93946b4_all.deb
nova-compute-libvirt_2014.2-fuel6.0~mira11+git.f783393.93946b4_all.deb
nova-compute-lxc_2014.2-fuel6.0~mira11+git.f783393.93946b4_all.deb
nova-compute-qemu_2014.2-fuel6.0~mira11+git.f783393.93946b4_all.deb
nova-compute-vmware_2014.2-fuel6.0~mira11+git.f783393.93946b4_all.deb
nova-compute-xen_2014.2-fuel6.0~mira11+git.f783393.93946b4_all.deb
nova-compute_2014.2-fuel6.0~mira11+git.f783393.93946b4_all.deb
nova-conductor_2014.2-fuel6.0~mira11+git.f783393.93946b4_all.deb
nova-console_2014.2-fuel6.0~mira11+git.f783393.93946b4_all.deb
nova-consoleauth_2014.2-fuel6.0~mira11+git.f783393.93946b4_all.deb
nova-doc_2014.2-fuel6.0~mira11+git.f783393.93946b4_all.deb
nova-network_2014.2-fuel6.0~mira11+git.f783393.93946b4_all.deb
nova-novncproxy_2014.2-fuel6.0~mira11+git.f783393.93946b4_all.deb
nova-objectstore_2014.2-fuel6.0~mira11+git.f783393.93946b4_all.deb
nova-scheduler_2014.2-fuel6.0~mira11+git.f783393.93946b4_all.deb
nova-spiceproxy_2014.2-fuel6.0~mira11+git.f783393.93946b4_all.deb
nova-volume_2014.2-fuel6.0~mira11+git.f783393.93946b4_all.deb
nova-xvpvncproxy_2014.2-fuel6.0~mira11+git.f783393.93946b4_all.deb
python-nova_2014.2-fuel6.0~mira11+git.f783393.93946b4_all.deb

NOTE: Changeset is not merged, created temporary package repository.
DEB repository URL: http://osci-obs.vm.mirantis.net:82/ubuntu-fuel-6.0-stable-750/ubuntu

Roman Podoliaka (rpodolyaka)
affects: fuel → mos
Changed in mos:
milestone: 6.0 → none
milestone: none → 6.0
Revision history for this message
OSCI Robot (oscirobot) wrote :

RPM package nova has been built for project openstack/nova
Package version == 2014.2, package release == fuel6.0.mira14.git.448ab1e.6bacacb

Changeset: https://review.fuel-infra.org/750
project: openstack/nova
branch: openstack-ci/fuel-6.0/2014.2
author: Sergey Nikitin
committer: Roman Podoliaka
subject: Fixed a problem with Mellanox plugin
status: patchset-created

Files placed on repository:
openstack-nova-2014.2-fuel6.0.mira14.git.448ab1e.6bacacb.noarch.rpm
openstack-nova-api-2014.2-fuel6.0.mira14.git.448ab1e.6bacacb.noarch.rpm
openstack-nova-cells-2014.2-fuel6.0.mira14.git.448ab1e.6bacacb.noarch.rpm
openstack-nova-cert-2014.2-fuel6.0.mira14.git.448ab1e.6bacacb.noarch.rpm
openstack-nova-common-2014.2-fuel6.0.mira14.git.448ab1e.6bacacb.noarch.rpm
openstack-nova-compute-2014.2-fuel6.0.mira14.git.448ab1e.6bacacb.noarch.rpm
openstack-nova-conductor-2014.2-fuel6.0.mira14.git.448ab1e.6bacacb.noarch.rpm
openstack-nova-console-2014.2-fuel6.0.mira14.git.448ab1e.6bacacb.noarch.rpm
openstack-nova-doc-2014.2-fuel6.0.mira14.git.448ab1e.6bacacb.noarch.rpm
openstack-nova-network-2014.2-fuel6.0.mira14.git.448ab1e.6bacacb.noarch.rpm
openstack-nova-novncproxy-2014.2-fuel6.0.mira14.git.448ab1e.6bacacb.noarch.rpm
openstack-nova-objectstore-2014.2-fuel6.0.mira14.git.448ab1e.6bacacb.noarch.rpm
openstack-nova-scheduler-2014.2-fuel6.0.mira14.git.448ab1e.6bacacb.noarch.rpm
python-nova-2014.2-fuel6.0.mira14.git.448ab1e.6bacacb.noarch.rpm

NOTE: Changeset is not merged, created temporary package repository.
RPM repository URL: http://osci-obs.vm.mirantis.net:82/centos-fuel-6.0-stable-750/centos

Revision history for this message
OSCI Robot (oscirobot) wrote :

DEB package nova has been built for project openstack/nova
Package version == 2014.2, package release == fuel6.0~mira14+git.448ab1e.6bacacb

Changeset: https://review.fuel-infra.org/750
project: openstack/nova
branch: openstack-ci/fuel-6.0/2014.2
author: Sergey Nikitin
committer: Roman Podoliaka
subject: Fixed a problem with Mellanox plugin
status: patchset-created

Files placed on repository:
nova-ajax-console-proxy_2014.2-fuel6.0~mira14+git.448ab1e.6bacacb_all.deb
nova-api-ec2_2014.2-fuel6.0~mira14+git.448ab1e.6bacacb_all.deb
nova-api-metadata_2014.2-fuel6.0~mira14+git.448ab1e.6bacacb_all.deb
nova-api-os-compute_2014.2-fuel6.0~mira14+git.448ab1e.6bacacb_all.deb
nova-api-os-volume_2014.2-fuel6.0~mira14+git.448ab1e.6bacacb_all.deb
nova-api_2014.2-fuel6.0~mira14+git.448ab1e.6bacacb_all.deb
nova-baremetal_2014.2-fuel6.0~mira14+git.448ab1e.6bacacb_all.deb
nova-cells_2014.2-fuel6.0~mira14+git.448ab1e.6bacacb_all.deb
nova-cert_2014.2-fuel6.0~mira14+git.448ab1e.6bacacb_all.deb
nova-common_2014.2-fuel6.0~mira14+git.448ab1e.6bacacb_all.deb
nova-compute-kvm_2014.2-fuel6.0~mira14+git.448ab1e.6bacacb_all.deb
nova-compute-libvirt_2014.2-fuel6.0~mira14+git.448ab1e.6bacacb_all.deb
nova-compute-lxc_2014.2-fuel6.0~mira14+git.448ab1e.6bacacb_all.deb
nova-compute-qemu_2014.2-fuel6.0~mira14+git.448ab1e.6bacacb_all.deb
nova-compute-vmware_2014.2-fuel6.0~mira14+git.448ab1e.6bacacb_all.deb
nova-compute-xen_2014.2-fuel6.0~mira14+git.448ab1e.6bacacb_all.deb
nova-compute_2014.2-fuel6.0~mira14+git.448ab1e.6bacacb_all.deb
nova-conductor_2014.2-fuel6.0~mira14+git.448ab1e.6bacacb_all.deb
nova-console_2014.2-fuel6.0~mira14+git.448ab1e.6bacacb_all.deb
nova-consoleauth_2014.2-fuel6.0~mira14+git.448ab1e.6bacacb_all.deb
nova-doc_2014.2-fuel6.0~mira14+git.448ab1e.6bacacb_all.deb
nova-network_2014.2-fuel6.0~mira14+git.448ab1e.6bacacb_all.deb
nova-novncproxy_2014.2-fuel6.0~mira14+git.448ab1e.6bacacb_all.deb
nova-objectstore_2014.2-fuel6.0~mira14+git.448ab1e.6bacacb_all.deb
nova-scheduler_2014.2-fuel6.0~mira14+git.448ab1e.6bacacb_all.deb
nova-spiceproxy_2014.2-fuel6.0~mira14+git.448ab1e.6bacacb_all.deb
nova-volume_2014.2-fuel6.0~mira14+git.448ab1e.6bacacb_all.deb
nova-xvpvncproxy_2014.2-fuel6.0~mira14+git.448ab1e.6bacacb_all.deb
python-nova_2014.2-fuel6.0~mira14+git.448ab1e.6bacacb_all.deb

NOTE: Changeset is not merged, created temporary package repository.
DEB repository URL: http://osci-obs.vm.mirantis.net:82/ubuntu-fuel-6.0-stable-750/ubuntu

Roman Podoliaka (rpodolyaka)
Changed in mos:
status: In Progress → Fix Committed
Revision history for this message
Alexander Gubanov (ogubanov) wrote :

I checked it on mos 6.0 - merged.

Changed in mos:
status: Fix Committed → Fix Released
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to openstack/nova (openstack-ci/fuel-7.0/2015.1.0)

Fix proposed to branch: openstack-ci/fuel-7.0/2015.1.0
Change author: Sergey Nikitin <email address hidden>
Review: https://review.fuel-infra.org/8239

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix merged to openstack/nova (openstack-ci/fuel-7.0/2015.1.0)

Reviewed: https://review.fuel-infra.org/8239
Submitter: mos-infra-ci <>
Branch: openstack-ci/fuel-7.0/2015.1.0

Commit: f022441a7d5b595fde704f4ec632349cab60cbaa
Author: Sergey Nikitin <email address hidden>
Date: Wed Jul 15 13:57:21 2015

Fixed a problem with Mellanox plugin

The removal of vif_driver blocks Mellanox from configuring eswitchd
as the vif_driver in nova.conf. The patch adds methods which are
in use only when using mechanism_driver=mlnx.

Closes-Bug: #1386858

Change-Id: I7b55926e3dee521d4ef35addc1123c6afbcd3e73

Revision history for this message
Sergey Nikitin (snikitin) wrote :

No need to verify on 7.0 because patch for this bug just add 3 methods which used nowhere in MOS. These methods were added by request of Mellanox guys. They said they need it.

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to openstack/nova (openstack-ci/fuel-8.0/liberty)

Fix proposed to branch: openstack-ci/fuel-8.0/liberty
Change author: Sergey Nikitin <email address hidden>
Review: https://review.fuel-infra.org/13293

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Change abandoned on openstack/nova (openstack-ci/fuel-8.0/liberty)

Change abandoned by Sergey Nikitin <email address hidden> on branch: openstack-ci/fuel-8.0/liberty
Review: https://review.fuel-infra.org/13293
Reason: This change is not needed in MOS 8.0

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.