diff options
author | Morten Tokle <mortent@verizonmedia.com> | 2021-06-01 14:24:40 +0200 |
---|---|---|
committer | Morten Tokle <mortent@verizonmedia.com> | 2021-06-01 14:24:40 +0200 |
commit | 298ff55faac1e51af3ffbb53e139569b82ed8eb4 (patch) | |
tree | fdf361b96e9cb8e29c1509020c228281d9c91c7e /controller-server | |
parent | ad0bd43f50719672848f3cb3859fad0d28a9820d (diff) |
Remove use of flags for tenant specific iam roles
Diffstat (limited to 'controller-server')
-rw-r--r-- | controller-server/src/main/java/com/yahoo/vespa/hosted/controller/TenantController.java | 15 |
1 files changed, 4 insertions, 11 deletions
diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/TenantController.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/TenantController.java index 4b102ef3077..1ff68ae641a 100644 --- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/TenantController.java +++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/TenantController.java @@ -42,14 +42,11 @@ public class TenantController { private final Controller controller; private final CuratorDb curator; private final AccessControl accessControl; - private final BooleanFlag provisionTenantRoles; - public TenantController(Controller controller, CuratorDb curator, AccessControl accessControl, FlagSource flagSource) { this.controller = Objects.requireNonNull(controller, "controller must be non-null"); this.curator = Objects.requireNonNull(curator, "curator must be non-null"); this.accessControl = accessControl; - this.provisionTenantRoles = Flags.PROVISION_TENANT_ROLES.bindTo(flagSource); // Update serialization format of all tenants @@ -116,15 +113,11 @@ public class TenantController { TenantId.validate(tenantSpec.tenant().value()); curator.writeTenant(accessControl.createTenant(tenantSpec, controller.clock().instant(), credentials, asList())); - // Provision tenant role if enabled - if (provisionTenantRoles.with(FetchVector.Dimension.TENANT_ID, tenantSpec.tenant().value()).value()) { - try { - controller.serviceRegistry().roleService().createTenantRole(tenantSpec.tenant()); - } catch (Exception e) { - throw new RuntimeException("Unable to create tenant role for tenant: " + tenantSpec.tenant()); - } + try { + controller.serviceRegistry().roleService().createTenantRole(tenantSpec.tenant()); + } catch (Exception e) { + throw new RuntimeException("Unable to create tenant role for tenant: " + tenantSpec.tenant()); } - } } |