diff options
author | Harald Musum <musum@yahooinc.com> | 2022-07-04 13:55:26 +0200 |
---|---|---|
committer | Harald Musum <musum@yahooinc.com> | 2022-07-04 13:55:26 +0200 |
commit | a61d52152eb749b0a128a55988a6fe4b16bd26d7 (patch) | |
tree | 52d80a48d51254198e283ebecb20ad660ba701ef /node-repository | |
parent | 062f10045c11c16b353fa8bd61f9e730028b49ea (diff) |
Update javadoc and reduce log level
Diffstat (limited to 'node-repository')
-rw-r--r-- | node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/ArchiveUris.java | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/ArchiveUris.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/ArchiveUris.java index 1cd8bb0fccd..e057fabc4fc 100644 --- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/ArchiveUris.java +++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/ArchiveUris.java @@ -7,11 +7,11 @@ import com.yahoo.vespa.curator.Lock; import com.yahoo.vespa.hosted.provision.Node; import com.yahoo.vespa.hosted.provision.node.Allocation; import com.yahoo.vespa.hosted.provision.persistence.CuratorDatabaseClient; - import java.time.Duration; import java.util.Map; import java.util.Optional; import java.util.TreeMap; +import java.util.logging.Level; import java.util.logging.Logger; import java.util.regex.Pattern; @@ -63,7 +63,7 @@ public class ArchiveUris { })); } - /** Set the docker image for nodes of given type */ + /** Set (or remove, if archiveURI is empty) archive URI to use for given tenant */ public void setArchiveUri(TenantName tenant, Optional<String> archiveUri) { try (Lock lock = db.lockArchiveUris()) { Map<TenantName, String> archiveUris = new TreeMap<>(db.readArchiveUris()); @@ -73,7 +73,8 @@ public class ArchiveUris { () -> archiveUris.remove(tenant)); db.writeArchiveUris(archiveUris); this.archiveUris.invalidate(); // Throw away current cache - log.info("Set archive URI for " + tenant + " to " + archiveUri.orElse(null)); + log.log(Level.FINE, () -> archiveUri.map(s -> "Set archive URI for " + tenant + " to " + s) + .orElseGet(() -> "Remove archive URI for " + tenant)); } } |