diff options
author | Jon Marius Venstad <jvenstad@yahoo-inc.com> | 2017-09-15 15:41:32 +0200 |
---|---|---|
committer | Jon Marius Venstad <jvenstad@yahoo-inc.com> | 2017-09-15 15:41:32 +0200 |
commit | b2e303e8dde55fb2285a9be77afb7078505ca3a0 (patch) | |
tree | 512540c835d15a0adcc710e7c0783f19ccabd7bc /controller-server | |
parent | dcdb0188e4daf6b60c43b31b391fc32bc880f879 (diff) |
Add support for initial assignee for issues
Diffstat (limited to 'controller-server')
-rw-r--r-- | controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/DeploymentIssueReporter.java | 9 |
1 files changed, 6 insertions, 3 deletions
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 11d259962d2..b2f010eeb79 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 @@ -5,6 +5,7 @@ import com.yahoo.config.provision.ApplicationId; import com.yahoo.vespa.hosted.controller.Application; import com.yahoo.vespa.hosted.controller.Controller; import com.yahoo.vespa.hosted.controller.api.Tenant; +import com.yahoo.vespa.hosted.controller.api.application.v4.model.TenantType; import com.yahoo.vespa.hosted.controller.api.identifiers.PropertyId; import com.yahoo.vespa.hosted.controller.api.identifiers.TenantId; import com.yahoo.vespa.hosted.controller.api.integration.Contacts; @@ -40,7 +41,7 @@ public class DeploymentIssueReporter extends Maintainer { static final Duration maxFailureAge = Duration.ofDays(2); static final Duration maxInactivityAge = Duration.ofDays(4); static final String deploymentFailureLabel = "vespaDeploymentFailure"; - static final Classification vespaOps = new Classification("VESPA", "Services", deploymentFailureLabel); + static final Classification vespaOps = new Classification("VESPA", "Services", deploymentFailureLabel, null); static final UserContact terminalUser = new UserContact("frodelu", "Frode Lundgren", admin); private final Contacts contacts; @@ -85,7 +86,9 @@ public class DeploymentIssueReporter extends Maintainer { Classification applicationOwner = null; try { applicationTenant= ownerOf(application); - applicationOwner = jiraClassificationOf(applicationTenant); + applicationOwner = applicationTenant.tenantType() == TenantType.USER + ? vespaOps.withAssignee(applicationTenant.getId().id().replaceFirst("by-", "")) + : jiraClassificationOf(applicationTenant); fileFor(application, deploymentIssue.with(applicationOwner)); } catch (RuntimeException e) { // Catch errors due to inconsistent or missing data in Sherpa, OpsDB, JIRA, and send to ourselves. @@ -98,7 +101,7 @@ public class DeploymentIssueReporter extends Maintainer { applicationTenant.getPropertyId().get() + "&action=view] for your property was rejected by JIRA. Please check your spelling.")); else - fileFor(application, deploymentIssue.append(e.getMessage() + "\n\nAddressee:\n" + applicationOwner)); + fileFor(application, deploymentIssue.with(vespaOps).append(e.getMessage() + "\n\nAddressee:\n" + applicationOwner)); } } } |