Skip to content

Conversation

@awalford16
Copy link
Collaborator

@awalford16 awalford16 commented Oct 9, 2025

  • Enabled Minio chart to generate its own certs, minio then listens on port 443
  • Added API functionality to use STS auth if ACCESS_KEY and SECRET_KEY are not provided
  • Uses AssumeRoleWithWebIdentity endpoint: https://docs.min.io/enterprise/aistor-object-store/developers/security-token-service/assumerolewithwebidentity/ to grab access/secret key from minio STS
  • Uses the k8s certificate accessible through the mounted service account so the minio cert can be trusted
  • Mounts the k8s service account with audience sts.min.io which is used as the JWT for authenticating with minio
  • Creates a minio PolicyBinding which ties the STS authenticated account to readwrite permissions
  • Minio client requires SSL_CERT_FILE to be exported which is used by minio python client

  • Replaced the copy job to use curl which supports referencing a custom ca-cert
  • Added the trusted k8s CA cert to /tmp/.mc allowing the minio config setup to trust minio TLS
  • Changed argo-artifacts-tls to argo-artifacts-ingress-tls for minio ingress cert since Minio creates a secret called argo-artifacts-tls for the auto-generated certificates

  • Adds network policy rules to allow resolving sts endpoint and talking to the minio-operator namespace for STS auth
  • Added some improved error handling in the minio client

@craddm
Copy link
Contributor

craddm commented Nov 11, 2025

@awalford16 is this one ready for another review?

# The service account to use for the application
"serviceaccount": fridge_api_sa.metadata.name,
},
"policies": ["readwrite"],
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This policy is still quite open, since it allows bucket creation

Copy link
Contributor

@craddm craddm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@craddm craddm merged commit add3159 into main Nov 24, 2025
5 checks passed
@craddm craddm deleted the api/minio-sts-auth branch November 24, 2025 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants