Add flag to skip loading props that don't exist when creating a new e…

…nv (#127)

gradle.properties

@@ -16,4 +16,4 @@
 
 group=com.nike
 artifactId=cerberus-lifecycle-cli
-version=4.6.3
+version=4.6.4

src/main/java/com/nike/cerberus/operation/cms/CreateCmsConfigOperation.java

@@ -44,7 +44,7 @@ public CreateCmsConfigOperation(final ConfigStore configStore) {
     @Override
     public void run(final CreateCmsConfigCommand command) {
         logger.info("Retrieving configuration data from the configuration bucket.");
-        final Properties cmsConfigProperties = configStore.getCmsSystemProperties();
+        final Properties cmsConfigProperties = configStore.getCmsSystemProperties(false);
 
         cmsConfigProperties.put(CMS_ADMIN_GROUP_KEY, command.getAdminGroup());
 

src/main/java/com/nike/cerberus/operation/cms/UpdateCmsConfigOperation.java

@@ -49,7 +49,7 @@ public void run(final UpdateCmsConfigCommand command) {
 
         logger.debug("Retrieving configuration data from the configuration bucket.");
 
-        final Properties newProperties = configStore.getCmsSystemProperties();
+        final Properties newProperties = configStore.getCmsSystemProperties(true);
         final Properties existingCustomProperties = configStore.getExistingCmsUserProperties();
         if (!command.getOverwrite()) {
             // keep existing custom properties

src/main/java/com/nike/cerberus/store/ConfigStore.java

@@ -324,19 +324,23 @@ private Properties generateBaseCmsSystemProperties() {
     /**
      * System properties not set with -P param
      */
-    public Properties getCmsSystemProperties() {
+    public Properties getCmsSystemProperties(boolean shouldLoadExistingProperties) {
 
         Properties properties = new Properties();
-        Properties existingProperties = getAllExistingCmsEnvProperties();
 
-        // overwrite any of the automatically generated properties that may have changed
-        existingProperties.putAll(generateBaseCmsSystemProperties());
+        Properties existingProperties;
+        if (shouldLoadExistingProperties) {
+            existingProperties = getAllExistingCmsEnvProperties();
 
-        existingProperties.forEach((key, value) -> {
-            if (SYSTEM_CONFIGURED_CMS_PROPERTIES.contains(key)) {
-                properties.put(key, value);
-            }
-        });
+            existingProperties.forEach((key, value) -> {
+                if (SYSTEM_CONFIGURED_CMS_PROPERTIES.contains(key)) {
+                    properties.put(key, value);
+                }
+            });
+        }
+
+        // overwrite any of the automatically generated properties that may have changed
+        properties.putAll(generateBaseCmsSystemProperties());
 
         if (!properties.containsKey(HASH_SALT)) {
             properties.put(HASH_SALT, saltGenerator.generateSalt());