Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Annotate ServiceAccount in Helm chart and add commented-out examples in values.yaml #1912

Merged
merged 2 commits into from
Oct 25, 2024

Conversation

jayg2309
Copy link
Contributor

With reference to issue #958

  1. it would be nice if you could annotate the serviceaccount via the helm chart rather than creating a seperate service account for workload identity
    Screenshot 2024-10-25 215611

  2. also, it would be nice if the values.yaml in git had some commented-out examples for configuration: secret: etc
    Screenshot 2024-10-25 215625
    Screenshot 2024-10-25 215639

@anna-geller Please have a look

@@ -59,6 +59,29 @@ helm repo add kestra https://helm.kestra.io/
helm install my-kestra kestra/kestra
```

**Workload Identity Setup(Optional)**

If you are using Google Cloud Workload Identity, you can annotate your Kubernetes service account in the Helm chart configuration. This will allow Kestra to automatically use the associated GCP service account for authentication.
Copy link
Member

Choose a reason for hiding this comment

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

Did you test that this works? if so, can you add a couple of screenshots?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I haven't tested it since GKE is not free.

Copy link
Member

Choose a reason for hiding this comment

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

no worries

Co-authored-by: Anna Geller <anna.m.geller@gmail.com>
@anna-geller anna-geller merged commit 4dd976d into kestra-io:main Oct 25, 2024
@jayg2309 jayg2309 deleted the jayg2309-branch branch October 25, 2024 20:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants