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

[v15] [buddy] fix(helm-chart): adjust to entrypoint within dockerfile #44698

Merged
merged 1 commit into from
Jul 30, 2024

Conversation

fheinecke
Copy link
Contributor

Backport #44607 to branch/v15

changelog: Fixed event-handler Helm charts using the wrong command when starting the event-handler container

Signed-off-by: Fred Heinecke <fred.heinecke@goteleport.com>
@fheinecke fheinecke added this pull request to the merge queue Jul 30, 2024
@public-teleport-github-review-bot public-teleport-github-review-bot bot removed the request for review from hugoShaka July 30, 2024 20:39
Merged via the queue into branch/v15 with commit 566d012 Jul 30, 2024
32 checks passed
@fheinecke fheinecke deleted the bot/backport-44607-branch/v15 branch July 30, 2024 20:51
@peikk0
Copy link

peikk0 commented Aug 9, 2024

@fheinecke this is broken, the image doesn't have /usr/local/bin/teleport-plugin:

Warning  Failed     5s (x2 over 6s)  kubelet            Error: failed to create containerd task: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: exec: "/usr/local/bin/teleport-plugin": stat /usr/local/bin/teleport-plugin: no such file or directory: unknown

@lavabyrd
Copy link

@fheinecke - is there a fix for this on the way? As @peikk0 mentioned, this doesn't work with 15.4.x currently

@fheinecke
Copy link
Contributor Author

Hey folks, I missed a backport of some plugins work for v15. I have a fix in the works for this and plan on merging it tomorrow after some tests run.

@fheinecke
Copy link
Contributor Author

A fix has been merged and will go out on the next v15 release (probably some time this week).

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.

6 participants