use contain instead of include, fixes #367

manifests/client.pp

@@ -52,13 +52,13 @@
     $merged_options = deep_merge($options, delete($default_options, keys($options)))
   }
 
-  include ssh::client::install
-  include ssh::client::config
+  contain ssh::client::install
+  contain ssh::client::config
 
   # Provide option to *not* use storeconfigs/puppetdb, which means not managing
   #  hostkeys and knownhosts
   if ($storeconfigs_enabled) {
-    include ssh::knownhosts
+    contain ssh::knownhosts
 
     Class['ssh::client::install']
     -> Class['ssh::client::config']

manifests/client/config/user.pp

@@ -40,7 +40,7 @@
   String[1]                      $ssh_directory_default_mode = '0700',
   String[1]                      $ssh_config_default_mode    = '0600',
 ) {
-  include ssh::client
+  contain ssh::client
 
   # If a specific target file was specified,
   # it must have higher priority than any

manifests/server.pp

@@ -84,15 +84,15 @@
     $merged_options = deep_merge($default_options, $options)
   }
 
-  include ssh::server::install
-  include ssh::server::config
-  include ssh::server::service
+  contain ssh::server::install
+  contain ssh::server::config
+  contain ssh::server::service
 
   # Provide option to *not* use storeconfigs/puppetdb, which means not managing
   #  hostkeys and knownhosts
   if ($storeconfigs_enabled) {
-    include ssh::hostkeys
-    include ssh::knownhosts
+    contain ssh::hostkeys
+    contain ssh::knownhosts
 
     Class['ssh::server::install']
     -> Class['ssh::server::config']

manifests/server/config/setting.pp

@@ -15,7 +15,7 @@
   Variant[Boolean, Array, Hash, String] $value,
   Variant[String[1], Integer]           $order = '10'
 ) {
-  include ssh::server
+  contain ssh::server
 
   $real_value = $value ? {
     Boolean => $value ? {

manifests/server/host_key.pp

@@ -45,7 +45,7 @@
   Optional[String[1]]   $certificate_content = undef,
 ) {
   # Ensure the ssh::server class is included in the manifest
-  include ssh::server
+  contain ssh::server
 
   if $ensure == 'present' {
     if ! $public_key_source and ! $public_key_content {

manifests/server/instances.pp

@@ -35,7 +35,7 @@
   Stdlib::Absolutepath           $sshd_binary               = $ssh::server::sshd_binary,
   Optional[Stdlib::Absolutepath] $sshd_environments_file    = $ssh::server::sshd_environments_file,
 ) {
-  include ssh::server
+  contain ssh::server
 
   $sshd_instance_config             = assert_type(Hash, pick($options['sshd_config'], {}))
   $sshd_instance_matchblocks        = assert_type(Hash, pick($options['match_blocks'], {}))