Merge pull request #130 from ApplauseOSS/fix/include-warn

fix: don't fail on missing site config include file

Author: agaffney

deploy_config_generator/site_config.py

@@ -112,6 +112,9 @@ def load_file(self, path):
                 if not include_path.startswith('/'):
                     # Normalize include path based on location of parent file
                     include_path = os.path.join(os.path.dirname(path), include_path)
+                if not os.path.exists(include_path):
+                    self._display.vv('Skipping non-existant included file %s' % include_path)
+                    continue
                 self._display.v('Loading site config from included file %s' % include_path)
                 include_data = self.load_file(include_path)
                 data = dict_merge(data, include_data)

setup.py

@@ -53,7 +53,7 @@ def run(self):
 
 setup(
     name='deploy-config-generator',
-    version='2.30.1',
+    version='2.30.2',
     url='https://github.com/ApplauseOSS/deploy-config-generator',
     license='MIT',
     description='Utility to generate service deploy configurations',

tests/integration/site_config_includes/site_config.yml

@@ -12,3 +12,5 @@ plugins:
 include:
   - site_config.1.yml
   - site_config.{{ env }}.yml
+  # This should not cause a failure
+  - site_config.does.not.exist.yml