diff options
| author | Jon Marius Venstad <jonmv@users.noreply.github.com> | 2023-10-11 17:52:46 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-10-11 17:52:46 +0200 |
| commit | 57deef966539acdd3c96b52701fb29cf9062f6e0 (patch) | |
| tree | dc007feceea38da3161a90314ff608d072b56b6e /controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/InternalStepRunner.java | |
| parent | b7a78eebb83c13bc6f4383218f251ed28f5f4b1e (diff) | |
| parent | dd7cda514d6b49d9c26aea498a3c010cf35d43d5 (diff) | |
Merge pull request #28869 from vespa-engine/jonmv/job-runner-thread-metrics
Jonmv/job runner thread metrics
Diffstat (limited to 'controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/InternalStepRunner.java')
| -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 662b4018a34..1080b379c4d 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 @@ -727,6 +727,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: |