dulcy: switch to postgresql 12

[chef.git] / cookbooks / openvpn / recipes / default.rb

diff --git a/cookbooks/openvpn/recipes/default.rb b/cookbooks/openvpn/recipes/default.rb
index cbd45ee65c319efcb5ca224fe48e2cff67fe1f8a..2446fe97090aadf5b5db4a24a0c9551e6079b713 100644 (file)
--- a/cookbooks/openvpn/recipes/default.rb
+++ b/cookbooks/openvpn/recipes/default.rb
@@ -1,14 +1,14 @@
 #
 #

-# Cookbook Name:: openvpn
+# Cookbook:: openvpn

 # Recipe:: default
 #
 # Recipe:: default
 #

-# Copyright 2012, OpenStreetMap Foundation
+# Copyright:: 2012, OpenStreetMap Foundation

 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
 # You may obtain a copy of the License at
 #
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
 # You may obtain a copy of the License at
 #

-#     http://www.apache.org/licenses/LICENSE-2.0
+#     https://www.apache.org/licenses/LICENSE-2.0

 #
 # Unless required by applicable law or agreed to in writing, software
 # distributed under the License is distributed on an "AS IS" BASIS,
 #
 # Unless required by applicable law or agreed to in writing, software
 # distributed under the License is distributed on an "AS IS" BASIS,


@@ -20,14 +20,16 @@
 package "openvpn"
 
 service "openvpn" do
 package "openvpn"
 
 service "openvpn" do

-  action [ :enable, :start ]
+  action [:enable, :start]

   supports :status => true, :restart => true, :reload => true
   ignore_failure true
 end
 
   supports :status => true, :restart => true, :reload => true
   ignore_failure true
 end
 

-node[:openvpn][:tunnels].each do |name,details|
-  if peer = search(:node, "fqdn:#{details[:peer][:host]}").first
-    if peer[:openvpn] and not details[:peer][:address]
+node[:openvpn][:tunnels].each do |name, details|
+  peer = search(:node, "fqdn:#{details[:peer][:host]}").first
+
+  if peer
+    if peer[:openvpn] && !details[:peer][:address]

       node.default[:openvpn][:tunnels][name][:peer][:address] = peer[:openvpn][:address]
     end
 
       node.default[:openvpn][:tunnels][name][:peer][:address] = peer[:openvpn][:address]
     end
 


@@ -46,14 +48,14 @@ node[:openvpn][:tunnels].each do |name,details|
       creates "/etc/openvpn/#{name}.key"
     end
 
       creates "/etc/openvpn/#{name}.key"
     end
 

-    if File.exists?("/etc/openvpn/#{name}.key")
-      node.set[:openvpn][:keys][name] = IO.read("/etc/openvpn/#{name}.key")
+    if File.exist?("/etc/openvpn/#{name}.key")
+      node.normal[:openvpn][:keys][name] = IO.read("/etc/openvpn/#{name}.key")

     end
     end

-  elsif peer and peer[:openvpn]
+  elsif peer && peer[:openvpn]

     file "/etc/openvpn/#{name}.key" do
       owner "root"
       group "root"
     file "/etc/openvpn/#{name}.key" do
       owner "root"
       group "root"

-      mode 0600
+      mode 0o600

       content peer[:openvpn][:keys][name]
     end
   end
       content peer[:openvpn][:keys][name]
     end
   end


@@ -63,13 +65,13 @@ node[:openvpn][:tunnels].each do |name,details|
       source "tunnel.conf.erb"
       owner "root"
       group "root"
       source "tunnel.conf.erb"
       owner "root"
       group "root"

-      mode 0644
+      mode 0o644

       variables :name => name,
       variables :name => name,

-      :address => node[:openvpn][:address],
-      :port => node[:openvpn][:tunnels][name][:port],
-      :mode => node[:openvpn][:tunnels][name][:mode],
-      :peer => node[:openvpn][:tunnels][name][:peer]
-      notifies :restart, resources(:service => "openvpn")
+                :address => node[:openvpn][:address],
+                :port => node[:openvpn][:tunnels][name][:port],
+                :mode => node[:openvpn][:tunnels][name][:mode],
+                :peer => node[:openvpn][:tunnels][name][:peer]
+      notifies :restart, "service[openvpn]"

     end
   else
     file "/etc/openvpn/#{name}.conf" do
     end
   else
     file "/etc/openvpn/#{name}.conf" do