diff options
author | Martin Polden <mpolden@mpolden.no> | 2018-06-07 11:42:12 +0200 |
---|---|---|
committer | Martin Polden <mpolden@mpolden.no> | 2018-06-07 11:42:12 +0200 |
commit | 433e9ead9656d2469aeabb11817b7438988dc1f8 (patch) | |
tree | 099cf1b31e32b84b45383e704a6b31bce34dbd3a | |
parent | 10ca90702284f1db54ef1e989e2c27d0349f7d4e (diff) |
Identify escalation user
-rw-r--r-- | controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/organization/Organization.java | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/organization/Organization.java b/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/organization/Organization.java index 00c0d87554a..776002f31cb 100644 --- a/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/organization/Organization.java +++ b/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/organization/Organization.java @@ -3,7 +3,6 @@ package com.yahoo.vespa.hosted.controller.api.integration.organization; import com.yahoo.vespa.hosted.controller.api.identifiers.PropertyId; -import java.io.UncheckedIOException; import java.net.URI; import java.time.Duration; import java.util.List; @@ -87,8 +86,9 @@ public interface Organization { * * @param issueId ID of the issue to escalate. * @param propertyId PropertyId of the tenant owning the application for which the issue was filed. + * @return User that was assigned issue as a result of the escalation, if any */ - default boolean escalate(IssueId issueId, PropertyId propertyId) { + default Optional<User> escalate(IssueId issueId, PropertyId propertyId) { List<? extends List<? extends User>> contacts = contactsFor(propertyId); Optional<User> assignee = assigneeOf(issueId); @@ -101,9 +101,9 @@ public interface Organization { for (int level = assigneeLevel + 1; level < contacts.size(); level++) for (User target : contacts.get(level)) if (reassign(issueId, target)) - return true; + return Optional.of(target); - return false; + return Optional.empty(); } /** |