summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMartin Polden <mpolden@mpolden.no>2018-06-07 11:42:12 +0200
committerMartin Polden <mpolden@mpolden.no>2018-06-07 11:42:12 +0200
commit433e9ead9656d2469aeabb11817b7438988dc1f8 (patch)
tree099cf1b31e32b84b45383e704a6b31bce34dbd3a
parent10ca90702284f1db54ef1e989e2c27d0349f7d4e (diff)
Identify escalation user
-rw-r--r--controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/organization/Organization.java8
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();
}
/**