diff options
author | Harald Musum <musum@yahooinc.com> | 2023-09-25 13:32:22 +0200 |
---|---|---|
committer | Harald Musum <musum@yahooinc.com> | 2023-09-25 13:32:22 +0200 |
commit | a4d231f899a77dec7f390c5c0e9b60cb3ea3e866 (patch) | |
tree | 258ac37b3f9c5b66ef3ea9d8bbec181a59f630d8 | |
parent | c80f7f61295309853b4d5ca2530fa6876aa72aff (diff) |
Check only for s3 urls in validator
-rw-r--r-- | config-model/src/main/java/com/yahoo/vespa/model/application/validation/UrlConfigValidator.java | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/config-model/src/main/java/com/yahoo/vespa/model/application/validation/UrlConfigValidator.java b/config-model/src/main/java/com/yahoo/vespa/model/application/validation/UrlConfigValidator.java index 8332ba0387a..d9dd3729bd3 100644 --- a/config-model/src/main/java/com/yahoo/vespa/model/application/validation/UrlConfigValidator.java +++ b/config-model/src/main/java/com/yahoo/vespa/model/application/validation/UrlConfigValidator.java @@ -31,7 +31,7 @@ public class UrlConfigValidator extends Validator { } private static void validateS3UlsInConfig(DeployState state, ApplicationContainerCluster cluster, boolean isExclusive) { - if (hasUrlInConfig(cluster)) { + if (hasS3UrlInConfig(cluster)) { // TODO: Would be even better if we could add which config/field the url is set for in the error message String message = "Found s3:// urls in config for container cluster " + cluster.getName(); if ( ! state.zone().system().isPublic()) @@ -42,8 +42,9 @@ public class UrlConfigValidator extends Validator { } } - private static boolean hasUrlInConfig(ApplicationContainerCluster cluster) { - return cluster.userConfiguredUrls().all().size() > 0; + private static boolean hasS3UrlInConfig(ApplicationContainerCluster cluster) { + return cluster.userConfiguredUrls().all().stream() + .anyMatch(url -> url.startsWith("s3://")); } } |