Rename ApplicationRoles to TenantRoles

4 files changed, 10 insertions, 17 deletions

diff --git a/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/application/v4/model/DeploymentData.java b/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/application/v4/model/DeploymentData.java
index 78f8197062c..702c6bd0f41 100644
--- a/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/application/v4/model/DeploymentData.java
+++ b/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/application/v4/model/DeploymentData.java
@@ -5,7 +5,7 @@ import com.yahoo.config.provision.ApplicationId;
 import com.yahoo.config.provision.DockerImage;
 import com.yahoo.config.provision.zone.ZoneId;
 import com.yahoo.vespa.athenz.api.AthenzDomain;
-import com.yahoo.vespa.hosted.controller.api.integration.aws.ApplicationRoles;
+import com.yahoo.vespa.hosted.controller.api.integration.aws.TenantRoles;
 import com.yahoo.vespa.hosted.controller.api.integration.billing.Quota;
 import com.yahoo.vespa.hosted.controller.api.integration.certificates.EndpointCertificateMetadata;
 import com.yahoo.vespa.hosted.controller.api.integration.configserver.ContainerEndpoint;
@@ -30,7 +30,7 @@ public class DeploymentData {
     private final Optional<EndpointCertificateMetadata> endpointCertificateMetadata;
     private final Optional<DockerImage> dockerImageRepo;
     private final Optional<AthenzDomain> athenzDomain;
-    private final Optional<ApplicationRoles> applicationRoles;
+    private final Optional<TenantRoles> tenantRoles;
     private final Quota quota;
 
     public DeploymentData(ApplicationId instance, ZoneId zone, byte[] applicationPackage, Version platform,
@@ -38,7 +38,7 @@ public class DeploymentData {
                           Optional<EndpointCertificateMetadata> endpointCertificateMetadata,
                           Optional<DockerImage> dockerImageRepo,
                           Optional<AthenzDomain> athenzDomain,
-                          Optional<ApplicationRoles> applicationRoles,
+                          Optional<TenantRoles> tenantRoles,
                           Quota quota) {
         this.instance = requireNonNull(instance);
         this.zone = requireNonNull(zone);
@@ -48,7 +48,7 @@ public class DeploymentData {
         this.endpointCertificateMetadata = requireNonNull(endpointCertificateMetadata);
         this.dockerImageRepo = requireNonNull(dockerImageRepo);
         this.athenzDomain = athenzDomain;
-        this.applicationRoles = applicationRoles;
+        this.tenantRoles = tenantRoles;
         this.quota = quota;
     }
 
@@ -84,8 +84,8 @@ public class DeploymentData {
         return athenzDomain;
     }
 
-    public Optional<ApplicationRoles> applicationRoles() {
-        return applicationRoles;
+    public Optional<TenantRoles> tenantRoles() {
+        return tenantRoles;
     }
 
     public Quota quota() {
diff --git a/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/aws/NoopRoleService.java b/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/aws/NoopRoleService.java
index 81fec1582d0..dceb56d14c1 100644
--- a/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/aws/NoopRoleService.java
+++ b/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/aws/NoopRoleService.java
@@ -12,16 +12,11 @@ import java.util.Optional;
 public class NoopRoleService implements RoleService {
 
     @Override
-    public Optional<ApplicationRoles> createApplicationRoles(ApplicationId applicationId) {
+    public Optional<TenantRoles> createTenantRole(TenantName tenant) {
         return Optional.empty();
     }
 
     @Override
-    public String createTenantRole(TenantName tenant) {
-        return "";
-    }
-
-    @Override
     public String createTenantPolicy(TenantName tenant, String policyName, String awsId, String role) {
         return "";
     }
diff --git a/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/aws/RoleService.java b/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/aws/RoleService.java
index 93c86c406b4..3c04546f479 100644
--- a/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/aws/RoleService.java
+++ b/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/aws/RoleService.java
@@ -11,9 +11,7 @@ import java.util.Optional;
  */
 public interface RoleService {
 
-    Optional<ApplicationRoles> createApplicationRoles(ApplicationId applicationId);
-
-    String createTenantRole(TenantName tenant);
+    Optional<TenantRoles> createTenantRole(TenantName tenant);
 
     String createTenantPolicy(TenantName tenant, String policyName, String awsId, String role);
 
diff --git a/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/aws/ApplicationRoles.java b/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/aws/TenantRoles.java
index de3e84ac0c3..4c5e3257d0e 100644
--- a/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/aws/ApplicationRoles.java
+++ b/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/aws/TenantRoles.java
@@ -4,11 +4,11 @@ package com.yahoo.vespa.hosted.controller.api.integration.aws;
 /**
  * @author mortent
  */
-public class ApplicationRoles {
+public class TenantRoles {
     private final String hostRole;
     private final String containerRole;
 
-    public ApplicationRoles(String hostRole, String containerRole) {
+    public TenantRoles(String hostRole, String containerRole) {
         this.hostRole = hostRole;
         this.containerRole = containerRole;
     }