Skip to content

Commit

Permalink
Do not require openvpn::client for client specific configs
Browse files Browse the repository at this point in the history
There is no need to require openvpn::client tpe as client configs
don't rely on  any of that type resources. It also not possible to use
client_specific_config if extca_enabled is set to true for a server due
to openvpn::client failure.

Fixes voxpupuli#229

Signed-off-by: Anton Baranov <[email protected]>
  • Loading branch information
spacedog committed Mar 16, 2017
1 parent 7a6895a commit a498b56
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 6 deletions.
1 change: 0 additions & 1 deletion manifests/client_specific_config.pp
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,6 @@
) {

Openvpn::Server[$server] ->
Openvpn::Client[$name] ->
Openvpn::Client_specific_config[$name]

file { "${::openvpn::params::etc_directory}/openvpn/${server}/client-configs/${name}":
Expand Down
5 changes: 0 additions & 5 deletions spec/defines/openvpn_client_specific_config_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -11,18 +11,13 @@
:operatingsystemrelease => '12.04',
} }
let(:pre_condition) do
[
'openvpn::server { "test_server":
country => "CO",
province => "ST",
city => "Some City",
organization => "example.org",
email => "[email protected]"
}',
'openvpn::client { "test_client":
server => "test_server"
}'
].join
end

it { should contain_file('/etc/openvpn/test_server/client-configs/test_client') }
Expand Down

0 comments on commit a498b56

Please sign in to comment.