diff options
| author | jonmv <venstad@gmail.com> | 2023-10-11 13:37:26 +0200 |
|---|---|---|
| committer | jonmv <venstad@gmail.com> | 2023-10-11 13:37:26 +0200 |
| commit | 8f47db52179cc7d5a0d64b10c57f8d2df3a8e957 (patch) | |
| tree | 484d17abbf550fc85d6dbb6ade3b1a5a9fa881ba /controller-server/src/main/java/com | |
| parent | 0f76f2837215f3202dab2ea1c84c53bbf5bd0e7f (diff) | |
Log particular warning about declared test suite, but no tests found
Diffstat (limited to 'controller-server/src/main/java/com')
| -rw-r--r-- | controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/InternalStepRunner.java | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/InternalStepRunner.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/InternalStepRunner.java index 1b4fda28095..576bcf95364 100644 --- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/InternalStepRunner.java +++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/InternalStepRunner.java @@ -726,6 +726,8 @@ public class InternalStepRunner implements StepRunner { DeploymentSpec spec = controller.applications().requireApplication(TenantAndApplicationId.from(id.application())).deploymentSpec(); boolean requireTests = spec.steps().stream().anyMatch(step -> step.concerns(id.type().environment())); + logger.log(WARNING, "No tests were actually run, but this test suite is explicitly declared in 'deployment.xml'. " + + "Either add tests, ensure they're correctly configured, or remove the test declaration."); return Optional.of(requireTests ? testFailure : noTests); } case SUCCESS: |