aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorHarald Musum <musum@yahooinc.com>2023-09-25 13:32:22 +0200
committerHarald Musum <musum@yahooinc.com>2023-09-25 13:32:22 +0200
commita4d231f899a77dec7f390c5c0e9b60cb3ea3e866 (patch)
tree258ac37b3f9c5b66ef3ea9d8bbec181a59f630d8
parentc80f7f61295309853b4d5ca2530fa6876aa72aff (diff)
Check only for s3 urls in validator
-rw-r--r--config-model/src/main/java/com/yahoo/vespa/model/application/validation/UrlConfigValidator.java7
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://"));
}
}