diff options
author | Jon Bratseth <bratseth@verizonmedia.com> | 2019-05-27 10:49:05 +0200 |
---|---|---|
committer | Jon Bratseth <bratseth@verizonmedia.com> | 2019-05-27 10:49:05 +0200 |
commit | 8d442c3f2872561817b5ae523902a03286e57c77 (patch) | |
tree | 472bee4cff5b6f6d92b72789e25281e97aa93bf8 /config-model-api/src/main/java/com/yahoo/config/application | |
parent | 1b8db1a592d2523f5253e51c79e6650a79ba8f30 (diff) |
Log validation override deactivation to deploy log
Diffstat (limited to 'config-model-api/src/main/java/com/yahoo/config/application')
-rw-r--r-- | config-model-api/src/main/java/com/yahoo/config/application/api/ValidationOverrides.java | 15 |
1 files changed, 13 insertions, 2 deletions
diff --git a/config-model-api/src/main/java/com/yahoo/config/application/api/ValidationOverrides.java b/config-model-api/src/main/java/com/yahoo/config/application/api/ValidationOverrides.java index e076ffd0f10..7514a66e282 100644 --- a/config-model-api/src/main/java/com/yahoo/config/application/api/ValidationOverrides.java +++ b/config-model-api/src/main/java/com/yahoo/config/application/api/ValidationOverrides.java @@ -16,6 +16,7 @@ import java.time.format.DateTimeFormatter; import java.util.ArrayList; import java.util.List; import java.util.Optional; +import java.util.logging.Level; import java.util.logging.Logger; /** @@ -172,16 +173,26 @@ public class ValidationOverrides { } - private static class AllowAllValidationOverrides extends ValidationOverrides { + public static class AllowAllValidationOverrides extends ValidationOverrides { + private final DeployLogger logger; + + /** Create an instance of this which doesn't log */ public AllowAllValidationOverrides() { + this(null); + } + + /** Creates an instance of this which logs what is allows to the given deploy logger */ + public AllowAllValidationOverrides(DeployLogger logger) { super(List.of()); + this.logger = logger; } /** Returns whether the given (assumed invalid) change is allowed by this at the moment */ @Override public boolean allows(ValidationId validationId, Instant now) { - log.warning("Possibly destructive change '" + validationId + "' allowed"); + if (logger != null) + logger.log(Level.WARNING, "Possibly destructive change '" + validationId + "' allowed"); return true; } |