Make config-refresh endpoint more tolerant of having zero cloud provi…

…ders configured.
spinnaker · Nov 14, 2015 · 093f025 · 093f025
1 parent d21ae06
commit 093f025
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/...om/netflix/spinnaker/clouddriver/security/NoopCredentialsInitializerSynchronizable.groovy b/...om/netflix/spinnaker/clouddriver/security/NoopCredentialsInitializerSynchronizable.groovy
@@ -17,5 +17,5 @@
 package com.netflix.spinnaker.clouddriver.security
 
 class NoopCredentialsInitializerSynchronizable implements CredentialsInitializerSynchronizable {
-  String credentialsSynchronizationBeanName = "noop"
+  String credentialsSynchronizationBeanName = null
 }
diff --git a/...main/groovy/com/netflix/spinnaker/clouddriver/listeners/CredentialsRefreshListener.groovy b/...main/groovy/com/netflix/spinnaker/clouddriver/listeners/CredentialsRefreshListener.groovy
@@ -35,7 +35,9 @@ public class CredentialsRefreshListener implements ApplicationListener<ConfigRef
   @Override
   void onApplicationEvent(ConfigRefreshedEvent event) {
     for (CredentialsInitializerSynchronizable credentialsInitializer : credentialsInitializers) {
-      appContext.getBean(credentialsInitializer.credentialsSynchronizationBeanName)
+      if (credentialsInitializer.credentialsSynchronizationBeanName) {
+        appContext.getBean(credentialsInitializer.credentialsSynchronizationBeanName)
+      }
     }
   }
 }