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

Bot Helm Charts should support sidecar tbot natively #46909

Open
knechtionscoding opened this issue Sep 25, 2024 · 2 comments
Open

Bot Helm Charts should support sidecar tbot natively #46909

knechtionscoding opened this issue Sep 25, 2024 · 2 comments
Labels
feature-request Used for new features in Teleport, improvements to current should be #enhancements helm machine-id

Comments

@knechtionscoding
Copy link

What would you like Teleport to do?

Currently when installing the slack plugin helm chart there's no opportunity to fill in a sidecar for tbot.

I would love it to be natively supported if we are on a version of k8s that has support for sidecars (i.e. 1.29+)

related issues:

What problem does this solve?

Makes slack, pagerduty, jira, etc plugins much more secure as it can utilize tbot easily and natively inside the cluster.

If a workaround exists, please include it.

@knechtionscoding knechtionscoding added the feature-request Used for new features in Teleport, improvements to current should be #enhancements label Sep 25, 2024
@strideynet
Copy link
Contributor

We've recently merged support for a tbot helm chart which is a seperate deployment you can run in the same namespace as one of the plugins - do you have a particular need for a sidecar instead? That would be interesting for us to explore, but, at the moment we haven't dived into it too much since the landscape of sidecar support in K8S isn't too great.

@knechtionscoding
Copy link
Author

I would prefer a sidecar, makes it easy to make sure that only approved things are using the tbot output, etc.

Sidecars have not been supported, but as of 1.29 they are! https://kubernetes.io/docs/concepts/workloads/pods/sidecar-containers/ so if the various helm charts were willing to allow us to define initContainers I am willing to do the leg work to define the relevant resources.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request Used for new features in Teleport, improvements to current should be #enhancements helm machine-id
Projects
None yet
Development

No branches or pull requests

2 participants