Skip to content

Commit 0305baf

Browse files
author
Martin Linkhorst
committed
make deployment-service's ML experimentation role name configurable
1 parent b28edaa commit 0305baf

File tree

3 files changed

+3
-2
lines changed

3 files changed

+3
-2
lines changed

cluster/cluster.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1677,7 +1677,7 @@ Resources:
16771677
DeploymentControllerMLExperimentDeploymentRole:
16781678
Type: AWS::IAM::Role
16791679
Properties:
1680-
RoleName: "{{.Cluster.LocalID}}-deployment-service-ml-experiment-deployment"
1680+
RoleName: "{{ .Cluster.ConfigItems.deployment_service_ml_experiments_role_name }}"
16811681
AssumeRolePolicyDocument:
16821682
Version: "2012-10-17"
16831683
Statement:

cluster/config-defaults.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1040,6 +1040,7 @@ deployment_service_api_role_arn: ""
10401040
deployment_service_tokeninfo_url: ""
10411041
deployment_service_lightstep_token: ""
10421042
deployment_service_ml_experiments_enabled: "true"
1043+
deployment_service_ml_experiments_role_name: "{{ .Cluster.LocalID }}-deployment-service-ml-experiment-deployment"
10431044
deployment_service_cf_auto_expand_enabled: "false"
10441045
deployment_service_cf_update_source_branch_changes: "true"
10451046
deployment_service_executor_cdp_permissions: "false"

cluster/manifests/deployment-service/01-config.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ data:
4747
status-service-url-local: "http://deployment-status-service.ingress.cluster.local."
4848
deployment-role-arn: "arn:aws:iam::{{accountID .Cluster.InfrastructureAccount}}:role/{{.Cluster.LocalID}}-deployment-service-deployment"
4949
{{- if eq .Cluster.ConfigItems.deployment_service_ml_experiments_enabled "true"}}
50-
ml-experiment-deployment-role-arn: "arn:aws:iam::{{accountID .Cluster.InfrastructureAccount}}:role/{{.Cluster.LocalID}}-deployment-service-ml-experiment-deployment"
50+
ml-experiment-deployment-role-arn: "arn:aws:iam::{{accountID .Cluster.InfrastructureAccount}}:role/{{ .Cluster.ConfigItems.deployment_service_ml_experiments_role_name }}"
5151
{{- end }}
5252
cloudformation-enable-auto-expand: "{{.Cluster.ConfigItems.deployment_service_cf_auto_expand_enabled}}"
5353
cloudformation-update-source-branch-changes: "{{.Cluster.ConfigItems.deployment_service_cf_update_source_branch_changes}}"

0 commit comments

Comments
 (0)