Skip to content

Commit

Permalink
fix: argo sa arn is wrong (#207)
Browse files Browse the repository at this point in the history
#### Motivation

The Argo service account ARN stored in cloudformation was the fluentbit
one (copiy-paste mistake).

#### Modification

Change it to the right Argo SA ARN

#### Checklist

_If not applicable, provide explanation of why._

- [ ] Tests updated
- [ ] Docs updated
- [ ] Issue linked in Title
  • Loading branch information
paulfouquet authored and amfage committed Oct 30, 2023
1 parent a84cadf commit 03a6eb8
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion config/eks/cluster.ts
Original file line number Diff line number Diff line change
Expand Up @@ -212,6 +212,6 @@ export class LinzEksCluster extends Stack {
namespace: 'argo',
});
argoRunnerSa.node.addDependency(argoNs);
new CfnOutput(this, 'ArgoRunnerServiceAccountRoleArn', { value: fluentBitSa.role.roleArn });
new CfnOutput(this, 'ArgoRunnerServiceAccountRoleArn', { value: argoRunnerSa.role.roleArn });
}
}

0 comments on commit 03a6eb8

Please sign in to comment.