Merge pull request voxpupuli#832 from sorreltree/remove_storeconfigs_…

…warning_on_puppet_apply Remove storeconfigs warning on puppet apply
slm0n87 · Jul 20, 2016 · be668cf · be668cf
2 parents 35cc199 + 6aee974
commit be668cf
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 2 deletions.
diff --git a/manifests/resource/upstream.pp b/manifests/resource/upstream.pp
@@ -97,8 +97,10 @@
       content => template('nginx/conf.d/upstream_members.erb'),
     }
   } else {
-    # Collect exported members:
-    ::Nginx::Resource::Upstream::Member <<| upstream == $name |>>
+    class { 'nginx::resource::upstream::collect':
+      # Collect exported members
+      upstream_name => $name,
+    }
   }
 
   concat::fragment { "${name}_upstream_footer":

diff --git a/manifests/resource/upstream/collect.pp b/manifests/resource/upstream/collect.pp
@@ -0,0 +1,12 @@
+# Class to collect exported upstream members, if needed.
+#
+# This is separated into a separate class in order to remove errors if
+# storeconfigs are unavailable.
+
+class nginx::resource::upstream::collect (
+  $upstream_name,
+)
+{
+  # Collect exported members:
+  ::Nginx::Resource::Upstream::Member <<| upstream == $upstream_name |>>
+}