Skip to content

Commit

Permalink
Merge branch 'master' into feat/config-karpenter-tde-903
Browse files Browse the repository at this point in the history
  • Loading branch information
paulfouquet committed Oct 24, 2023
2 parents 5c56550 + e73fe9f commit df7a413
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 9 deletions.
9 changes: 2 additions & 7 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -33,12 +33,7 @@ Then to setup the cluster, only the first time using the cluster you need to run
You will need a AWS CLI > 2.7.x

```bash

# For Imagery maintainers you will already have the correct role so no role arn is needed.
aws eks update-kubeconfig --name Workflow --region ap-southeast-2

# For AWS Admin users you will need to find the correct EKS role to use
aws eks update-kubeconfig --name Workflow --region ap-southeast-2 --role-arn arn:aws:iam::...
aws eks update-kubeconfig --name Workflows --region ap-southeast-2
```

to validate the cluster is connected,
Expand All @@ -54,7 +49,7 @@ ip-255-100-39-100.ap-southeast-2.compute.internal Ready <none> 7d v1.21
to make the cli access easier you can set the default namespace to `argo`

```bash
k config set-context --current --namespace=argo
k config set-context --current --namespace=argo
```

## Submitting a job
Expand Down
1 change: 1 addition & 0 deletions config/README.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
# Topo-Workflows Infrastructure

The infrastructure running the workflows is mainly based on a Kubernetes (EKS) cluster and Argo Workflows. It is currently run on AWS.
Generally all Kubernetes resources are defined with cdk8s and anything that needs AWS interactions such as service accounts are defined with CDK.

## EKS Cluster / AWS CDK

Expand Down
2 changes: 0 additions & 2 deletions config/eks/cluster.ts
Original file line number Diff line number Diff line change
Expand Up @@ -199,7 +199,6 @@ export class LinzEksCluster extends Stack {
namespace: 'fluent-bit',
});
fluentBitSa.node.addDependency(fluentBitNs); // Ensure the namespace created first

new CfnOutput(this, 'FluentBitServiceAccountRoleArn', { value: fluentBitSa.role.roleArn });

// Basic constructs for argo to be deployed into
Expand All @@ -213,7 +212,6 @@ export class LinzEksCluster extends Stack {
namespace: 'argo',
});
argoRunnerSa.node.addDependency(argoNs);

new CfnOutput(this, 'ArgoRunnerServiceAccountRoleArn', { value: fluentBitSa.role.roleArn });
}
}

0 comments on commit df7a413

Please sign in to comment.