diff options
author | bjormel <bjormel@verizonmedia.com> | 2022-05-26 15:34:40 +0200 |
---|---|---|
committer | bjormel <bjormel@verizonmedia.com> | 2022-05-26 15:34:40 +0200 |
commit | 7e7048b5c769408aeb06e5ab9bcd0c982526003a (patch) | |
tree | dc8d11cf322e01fac5b21b492dfe99bbd78c5a7c /config-model | |
parent | 5ad3ee45adbc357e41c9779f19a28c05c94b0239 (diff) |
Clear accessLogElements if set in hosted vespa
Diffstat (limited to 'config-model')
2 files changed, 3 insertions, 2 deletions
diff --git a/config-model/src/main/java/com/yahoo/vespa/model/container/xml/ContainerModelBuilder.java b/config-model/src/main/java/com/yahoo/vespa/model/container/xml/ContainerModelBuilder.java index abbd9644901..dd0d74aa1e8 100644 --- a/config-model/src/main/java/com/yahoo/vespa/model/container/xml/ContainerModelBuilder.java +++ b/config-model/src/main/java/com/yahoo/vespa/model/container/xml/ContainerModelBuilder.java @@ -433,8 +433,9 @@ public class ContainerModelBuilder extends ConfigModelBuilder<ContainerModel> { List<Element> accessLogElements = getAccessLogElements(spec); if (cluster.isHostedVespa() && !accessLogElements.isEmpty()) { + accessLogElements.clear(); log.logApplicationPackage( - Level.WARNING, "The element 'accesslog' is not overridable in hosted Vespa"); + Level.WARNING, "Applications are not allowed to override the 'accesslog' element"); } else { for (Element accessLog : accessLogElements) { AccessLogBuilder.buildIfNotDisabled(deployState, cluster, accessLog).ifPresent(cluster::addComponent); diff --git a/config-model/src/test/java/com/yahoo/vespa/model/container/xml/ContainerModelBuilderTest.java b/config-model/src/test/java/com/yahoo/vespa/model/container/xml/ContainerModelBuilderTest.java index 7c76aef0981..81649375c8a 100644 --- a/config-model/src/test/java/com/yahoo/vespa/model/container/xml/ContainerModelBuilderTest.java +++ b/config-model/src/test/java/com/yahoo/vespa/model/container/xml/ContainerModelBuilderTest.java @@ -1124,7 +1124,7 @@ public class ContainerModelBuilderTest extends ContainerModelBuilderTestBase { createModel(root, deployState, null, DomBuilderTest.parse(containerService)); assertFalse(logger.msgs.isEmpty()); assertEquals(Level.WARNING, logger.msgs.get(0).getFirst()); - assertEquals("The element 'accesslog' is not overridable in hosted Vespa", + assertEquals("Applications are not allowed to override the 'accesslog' element", logger.msgs.get(0).getSecond()); } |