diff options
author | jonmv <venstad@gmail.com> | 2023-10-19 16:13:08 +0200 |
---|---|---|
committer | jonmv <venstad@gmail.com> | 2023-10-19 16:13:08 +0200 |
commit | 293231c92fb4e843ccac41c8dfc22c0086e37973 (patch) | |
tree | 3dc1d3ec746ff030e19465c50cf38b19c6f0aafd /controller-api/src/main | |
parent | 4916aab2d6da2fee98bb576e33192c753197ff62 (diff) |
Improve logged message
Diffstat (limited to 'controller-api/src/main')
-rw-r--r-- | controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/application/v4/model/DeploymentData.java | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/application/v4/model/DeploymentData.java b/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/application/v4/model/DeploymentData.java index 34682204163..fd4a34118c5 100644 --- a/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/application/v4/model/DeploymentData.java +++ b/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/application/v4/model/DeploymentData.java @@ -61,14 +61,14 @@ public class DeploymentData { this.zone = requireNonNull(zone); this.applicationPackage = requireNonNull(applicationPackage); this.platform = requireNonNull(platform); - this.endpoints = wrap(requireNonNull(endpoints), Duration.ofSeconds(30), "deployment endpoints"); + this.endpoints = wrap(requireNonNull(endpoints), Duration.ofSeconds(30), "deployment endpoints for " + instance + " in " + zone); this.dockerImageRepo = requireNonNull(dockerImageRepo); this.athenzDomain = athenzDomain; - this.quota = wrap(requireNonNull(quota), Duration.ofSeconds(10), "quota"); + this.quota = wrap(requireNonNull(quota), Duration.ofSeconds(10), "quota for " + instance); this.tenantSecretStores = List.copyOf(requireNonNull(tenantSecretStores)); this.operatorCertificates = List.copyOf(requireNonNull(operatorCertificates)); - this.cloudAccount = wrap(requireNonNull(cloudAccount), Duration.ofSeconds(5), "cloud account"); - this.dataPlaneTokens = wrap(dataPlaneTokens, Duration.ofSeconds(5), "data plane tokens"); + this.cloudAccount = wrap(requireNonNull(cloudAccount), Duration.ofSeconds(5), "cloud account for " + instance + " in " + zone); + this.dataPlaneTokens = wrap(dataPlaneTokens, Duration.ofSeconds(5), "data plane tokens for " + instance + " in " + zone); this.dryRun = dryRun; } @@ -155,7 +155,7 @@ public class DeploymentData { long durationNanos = System.nanoTime() - startNanos; Level level = durationNanos > timeout.toNanos() ? Level.WARNING : Level.FINE; String thrownMessage = thrown == null ? "" : " with exception " + thrown; - log.log(level, () -> "Getting " + description + " took " + Duration.ofNanos(durationNanos) + thrownMessage); + log.log(level, () -> String.format("Getting %s took %.6f seconds%s", description, durationNanos / 1e9, thrownMessage)); } } |