diff options
author | Jon Marius Venstad <jvenstad@yahoo-inc.com> | 2018-09-24 10:09:39 +0200 |
---|---|---|
committer | Jon Marius Venstad <jvenstad@yahoo-inc.com> | 2018-09-24 10:09:39 +0200 |
commit | fe0d25dac741bedb70f0fda173243b31787d851e (patch) | |
tree | 4e31713f1c06fd752b1b7b0598666abc9f418a1c /controller-server | |
parent | e249cea8da57c677d5aa9c41ec3ac5558ee64f98 (diff) |
Ignore apps without project Ids
Diffstat (limited to 'controller-server')
2 files changed, 12 insertions, 3 deletions
diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/ApplicationOwnershipConfirmer.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/ApplicationOwnershipConfirmer.java index 93578e97cc4..349f4390fd1 100644 --- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/ApplicationOwnershipConfirmer.java +++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/ApplicationOwnershipConfirmer.java @@ -50,6 +50,7 @@ public class ApplicationOwnershipConfirmer extends Maintainer { private void confirmApplicationOwnerships() { ApplicationList.from(controller().applications().asList()) .notPullRequest() + .withProjectId() .hasProductionDeployment() .asList() .stream() diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/DeploymentIssueReporter.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/DeploymentIssueReporter.java index a1351ab570c..70bc4133772 100644 --- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/DeploymentIssueReporter.java +++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/DeploymentIssueReporter.java @@ -48,15 +48,23 @@ public class DeploymentIssueReporter extends Maintainer { @Override protected void maintain() { try { - maintainDeploymentIssues(controller().applications().asList()); - maintainPlatformIssue(controller().applications().asList()); - escalateInactiveDeploymentIssues(controller().applications().asList()); + maintainDeploymentIssues(applications()); + maintainPlatformIssue(applications()); + escalateInactiveDeploymentIssues(applications()); } catch (UncheckedIOException e) { log.log(Level.INFO, () -> "IO exception handling issues, will retry in " + maintenanceInterval() + ": '" + Exceptions.toMessageString(e)); } } + /** Returns the applications to maintain issue status for. */ + private List<Application> applications() { + return ApplicationList.from(controller().applications().asList()) + .withProjectId() + .notPullRequest() + .asList(); + } + /** * File issues for applications which have failed deployment for longer than maxFailureAge * and store the issue id for the filed issues. Also, clear the issueIds of applications |