Skip to content

Commit

Permalink
Fix to remove conflicted symlink
Browse files Browse the repository at this point in the history
Since cisco plugin coexists with ovs, a symlink
pointing to ovs already exists causing a conflict
with cisco_plugins.ini. This fix should default
the symlink to ovs.

Change-Id: I98c7b2e08257a6d9c07916b9b114bcb21d913e4f
  • Loading branch information
Pradeep Kilambi committed Apr 2, 2014
1 parent fc4cb16 commit 55f952d
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 8 deletions.
10 changes: 4 additions & 6 deletions manifests/plugins/cisco.pp
Original file line number Diff line number Diff line change
Expand Up @@ -171,11 +171,9 @@

# In RH, this link is used to start Neutron process but in Debian, it's used only
# to manage database synchronization.
file {'/etc/neutron/plugin.ini':
ensure_resource('file', '/etc/neutron/plugin.ini', {
ensure => link,
target => '/etc/neutron/plugins/cisco/cisco_plugins.ini',
require => Package['neutron-plugin-cisco']
}


target => '/etc/neutron/plugins/openvswitch/ovs_neutron_plugin.ini',
require => Package['neutron-plugin-ovs']
})
}
4 changes: 2 additions & 2 deletions spec/classes/neutron_plugins_cisco_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -42,8 +42,8 @@ class { 'neutron': rabbit_password => 'passw0rd' }"
it 'should create plugin symbolic link' do
should contain_file('/etc/neutron/plugin.ini').with(
:ensure => 'link',
:target => '/etc/neutron/plugins/cisco/cisco_plugins.ini',
:require => 'Package[neutron-plugin-cisco]'
:target => '/etc/neutron/plugins/openvswitch/ovs_neutron_plugin.ini',
:require => 'Package[neutron-plugin-ovs]'
)
end

Expand Down

0 comments on commit 55f952d

Please sign in to comment.