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

[helm/teleport-cluster] Add audience into serviceAccountToken for proxy service #49756

Closed
verdel opened this issue Dec 4, 2024 · 0 comments · Fixed by #49796
Closed

[helm/teleport-cluster] Add audience into serviceAccountToken for proxy service #49756

verdel opened this issue Dec 4, 2024 · 0 comments · Fixed by #49796
Labels
bug feature-request Used for new features in Teleport, improvements to current should be #enhancements

Comments

@verdel
Copy link
Contributor

verdel commented Dec 4, 2024

What would you like Teleport to do?

We need to add the audience with the value {{ .Values.clusterName }} to the projected serviceAccountToken in the teleport-proxy deployment in the teleport-cluster helm chart.

What problem does this solve?

After backporting changes that allow the kubernetes in-cluster joining mechanism to use tokens with the clusterName specified in the audience, authorization for teleport-proxy in GKE (Google Cloud Kubernetes Engine) clusters breaks.

This happens because the TokenReview request now includes an audience that contains both https://kubernetes.default.svc and clusterName. However, in GKE, the default audience for ServiceAccount tokens is https://container.googleapis.com/v1/projects/PROJECT/locations/LOCATION/clusters/NAME. As a result, teleport-auth rejects the token from teleport-proxy.

It is necessary to explicitly specify the audience in the Deployment configuration to include clusterName, as it is already done in the chart for tbot.

@verdel verdel added the feature-request Used for new features in Teleport, improvements to current should be #enhancements label Dec 4, 2024
@hugoShaka hugoShaka added the bug label Dec 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug feature-request Used for new features in Teleport, improvements to current should be #enhancements
Projects
None yet
2 participants