[network] plugin.ini symlink not updated if main plugin changes

Bug #1320011 reported by Mark Vanderwiel
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack + Chef
Fix Released
Medium
Mark Vanderwiel

Bug Description

If the main changes from the default of ML2 to OVS, the symlink is not updated. I believe the cause is related to the fact the the symlink action :create is triggered by the main plugin conf file. In the ML2 case, the ovs conf was also created and used by the ovs agent. When the main is then switched to ovs, the ovs conf already exits and is up to date so the symlink action :create is never notified.

I believe the solution here is to un-tie the symlink from the creation of the conf file and instead just create the link based upon the main plugin without any notifies or subscriptions. The symlink points to a conf file, but the conf files are not unique enough to cover all the cases, whereas the main plugin is always correct for this resource.

Tags: network
Changed in openstack-chef:
status: New → In Progress
milestone: none → icehouse-rc1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cookbook-openstack-network (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cookbook-openstack-network (master)

Reviewed: https://review.openstack.org/93962
Committed: https://git.openstack.org/cgit/stackforge/cookbook-openstack-network/commit/?id=27c2f7bdba938b324e83c032c3da15e0b055d347
Submitter: Jenkins
Branch: master

commit 27c2f7bdba938b324e83c032c3da15e0b055d347
Author: Mark Vanderwiel <email address hidden>
Date: Fri May 16 12:04:24 2014 -0500

    Fix symlink when main changed from ML2 to ovs

    When the main plugin is changed from ML2 to OpenVSwitch, the redhat
    required plugins.ini symlink does not get properly updated. Change the
    symlink to rely solely on the main plugin attribute and not the conf
    files as there is not a 1-1 relationship between them. See bug for more
    details.

    Change-Id: If32747e54e8a8f31d1d553b750e6ae663670b873
    Closes-Bug: #1320011

Changed in openstack-chef:
status: In Progress → Fix Released
Changed in openstack-chef:
milestone: icehouse-rc1 → icehouse-stable
Changed in openstack-chef:
importance: Undecided → Medium
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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