neutron

core_plugin is defaulted to a non existed plugin

Bug #1093989 reported by yong sheng gong
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Undecided
Gary Kotton
neutron 2013.1 "grizzly"

Bug Description

    cfg.StrOpt('core_plugin',
               default='quantum.plugins.sample.SamplePlugin.FakePlugin'),

I propose let it point to:
quantum.plugins.openvswitch.ovs_quantum_plugin.OVSQuantumPluginV2

Revision history for this message
dan wendlandt (danwent) wrote :

To me, setting a default plugin is something that should be done by a distro. This already happens in practice, with Canonical setting it to OVS, and devstack defaulting to q-plugin as OVS. I think some versions of fedora default to OVS, while perhaps older versions of Red Hat default to linux-bridge, since OVS is not available.

I agree that having a non-existant plugin there makes no sense. I think having the field empty probably by default makes the most sense, but if we're going to pick one, the OVS plugin probably makes sense.

Revision history for this message
Gary Kotton (garyk) wrote :

I am in favor of having it empty. It would be best if the distro or application runnig quantum update the core_plugin (it may require database creation, setting etc, extra packages...)
I have taken the liberty to post a patch for this.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to quantum (master)

Fix proposed to branch: master
Review: https://review.openstack.org/18778

Changed in quantum:
assignee: nobody → Gary Kotton (garyk)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to quantum (master)

Reviewed: https://review.openstack.org/18778
Committed: http://github.com/openstack/quantum/commit/b5ea60252f3421beb1bf1d82ca95757d383058f5
Submitter: Jenkins
Branch: master

commit b5ea60252f3421beb1bf1d82ca95757d383058f5
Author: Gary Kotton <email address hidden>
Date: Mon Dec 31 16:03:21 2012 +0000

    Set default core_plugin to None

    Fixes bug 1093989

    Change-Id: Ib0b77994b813e6b2ec0621949371c66d66472896

Changed in quantum:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in quantum:
milestone: none → grizzly-2
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in quantum:
milestone: grizzly-2 → 2013.1
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.