Skip to content

Commit

Permalink
Argo to use IRSA for Artifact Upload (#2324)
Browse files Browse the repository at this point in the history
* changing argo to use a service account instead of s3 creds as it is more secure

* changing the location of when to annotate the service accounts.

* moving annotation commands again for testing

* adding a sleep command to see if the sa gets annotated

* forgot to add namespace to the command

* removing argo-s3-creds from values.yaml

* reverting my changes

* testing out commenting the "artifactRespository" section entirely.

* commenting out only the access and secret access keys

* testing the "iamEnbpointResolution" value.

* testing the "useStaticCredentials" key in the values.yaml

* cleaning up files and adding the "gen3 awsrole create" command to allows SA's to assume role. Also, fixing the setup argo db to use environment variables instead of the argo s3 secret as it no longer exists.

* getting rid of create-role command as the "awsrole create" will automatically do that.

* changing the setup script to allow workflows to be run from any namespace!

* testing creating a service account in each namespace so workflows can be run from anywhere.

* Changes that allow for the following:
Create ONE sa in the argo namespace.
Create ONE role in the argo namespace.
Create multiple inline bucket policies for every namespace so any namespace can access separate buckets.

* Update values.yaml

* changing the kube-setup script to use the "default" sa in the role Trust policy and removing the argo sa as there is no need for it anymore.

* fixing typo

* revertting kube-setup-argo changes as PR 2360 was opened separately

* Update values.yaml

---------

Co-authored-by: Aidan Hilt <[email protected]>
  • Loading branch information
EliseCastle23 and AidanHilt authored Jun 25, 2024
1 parent e5315f4 commit fcc4535
Showing 1 changed file with 3 additions and 8 deletions.
11 changes: 3 additions & 8 deletions kube/services/argo/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -92,18 +92,13 @@ server:
# -- Influences the creation of the ConfigMap for the workflow-controller itself.
useDefaultArtifactRepo: true

# -- Use static credentials for S3 (eg. when not using AWS IRSA)
useStaticCredentials: false
artifactRepository:
# -- Archive the main container logs as an artifact
archiveLogs: true
# -- Store artifact in a S3-compliant object store
s3:
# Note the `key` attribute is not the actual secret, it's the PATH to
# the contents in the associated secret, as defined by the `name` attribute.
accessKeySecret:
name: argo-s3-creds
key: AccessKeyId
secretKeySecret:
name: argo-s3-creds
key: SecretAccessKey
bucket: GEN3_ARGO_BUCKET
endpoint: s3.amazonaws.com
useSDKCreds: true

0 comments on commit fcc4535

Please sign in to comment.