Merge pull request #3951 from vespa-engine/bjorncs/athenz-identity-provider-cleanup

Bjorncs/athenz identity provider cleanup

1 files changed, 2 insertions, 0 deletions

diff --git a/athenz-identity-provider-service/src/main/java/com/yahoo/vespa/hosted/athenz/instanceproviderservice/AthenzInstanceProviderService.java b/athenz-identity-provider-service/src/main/java/com/yahoo/vespa/hosted/athenz/instanceproviderservice/AthenzInstanceProviderService.java
index 668444e2769..e66130332ac 100644
--- a/athenz-identity-provider-service/src/main/java/com/yahoo/vespa/hosted/athenz/instanceproviderservice/AthenzInstanceProviderService.java
+++ b/athenz-identity-provider-service/src/main/java/com/yahoo/vespa/hosted/athenz/instanceproviderservice/AthenzInstanceProviderService.java
@@ -66,6 +66,7 @@ public class AthenzInstanceProviderService extends AbstractComponent {
                     config, keyProvider, sslContextFactory, nodeRepository, zone);
             AthenzCertificateUpdater reloader =
                     new AthenzCertificateUpdater(certificateClient, sslContextFactory, keyProvider, config);
+            // TODO Configurable update frequency
             scheduler.scheduleAtFixedRate(reloader, 0, 1, TimeUnit.DAYS);
             try {
                 jetty.start();
@@ -117,6 +118,7 @@ public class AthenzInstanceProviderService extends AbstractComponent {
 
     private static class AthenzCertificateUpdater implements Runnable {
 
+        // TODO Make expiry a configuration parameter
         private static final TemporalAmount EXPIRY_TIME = Duration.ofDays(30);
         private static final Logger log = Logger.getLogger(AthenzCertificateUpdater.class.getName());