Skip to content

Machine ID: Render kubernetes template without exec plugin when using non-directory destination#43401

Merged
strideynet merged 6 commits intomasterfrom
strideynet/output-kubernetes-template-without-exec-plugin
Jun 25, 2024
Merged

Machine ID: Render kubernetes template without exec plugin when using non-directory destination#43401
strideynet merged 6 commits intomasterfrom
strideynet/output-kubernetes-template-without-exec-plugin

Conversation

@strideynet
Copy link
Copy Markdown
Contributor

A few customers have run into "must be a directory in exec plugin mode" - whilst there's a configurable option for this, it makes more sense for tbot to just default to enabling this setting to reduce friction. This should reduce support load.

changelog: Machine ID defaults to disabling the use of the Kubernetes exec plugin when writing a Kubeconfig to a directory destination. This removes the need to manually configure disable_exec_plugin.

@strideynet
Copy link
Copy Markdown
Contributor Author

CC @webvictim

Comment thread lib/tbot/config/template_kubernetes.go Outdated
Comment thread lib/tbot/config/template_kubernetes.go Outdated
Comment thread lib/tbot/config/template_kubernetes.go Outdated
@public-teleport-github-review-bot public-teleport-github-review-bot Bot removed the request for review from timothyb89 June 25, 2024 08:41
Co-authored-by: Edoardo Spadolini <edoardo.spadolini@goteleport.com>
@strideynet strideynet enabled auto-merge June 25, 2024 13:18
@strideynet strideynet added this pull request to the merge queue Jun 25, 2024
Merged via the queue into master with commit 014a847 Jun 25, 2024
@strideynet strideynet deleted the strideynet/output-kubernetes-template-without-exec-plugin branch June 25, 2024 14:05
@public-teleport-github-review-bot
Copy link
Copy Markdown

@strideynet See the table below for backport results.

Branch Result
branch/v14 Create PR
branch/v15 Create PR
branch/v16 Create PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants