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: Correct command for initContainer config #28613

Merged
merged 1 commit into from
Oct 17, 2023
Merged

Conversation

sayboras
Copy link
Member

This is to fix the below issue:

  Warning  Failed     8m3s (x5 over 9m31s)    kubelet            Error: failed to start container "config": Error response from daemon: failed to create task for container: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: exec: "cilium": executable file not found in $PATH: unknown

Relates: #28085

@sayboras sayboras requested review from a team as code owners October 16, 2023 10:10
@sayboras sayboras requested a review from squeed October 16, 2023 10:10
@maintainer-s-little-helper
Copy link

Commit bdc9f81 does not match "(?m)^Signed-off-by:".

Please follow instructions provided in https://docs.cilium.io/en/stable/contributing/development/contributing_guide/#developer-s-certificate-of-origin

@maintainer-s-little-helper maintainer-s-little-helper bot added dont-merge/needs-sign-off The author needs to add signoff to their commits before merge. dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Oct 16, 2023
This is to fix the below issue:

```
  Warning  Failed     8m3s (x5 over 9m31s)    kubelet            Error: failed to start container "config": Error response from daemon: failed to create task for container: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: exec: "cilium": executable file not found in $PATH: unknown
````

Relates: #28085
Signed-off-by: Tam Mach <tam.mach@cilium.io>
@sayboras sayboras added the release-note/bug This PR fixes an issue in a previous release of Cilium. label Oct 16, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Oct 16, 2023
@maintainer-s-little-helper
Copy link

Commit bdc9f81 does not match "(?m)^Signed-off-by:".

Please follow instructions provided in https://docs.cilium.io/en/stable/contributing/development/contributing_guide/#developer-s-certificate-of-origin

@sayboras sayboras added area/helm Impacts helm charts and user deployment experience and removed dont-merge/needs-sign-off The author needs to add signoff to their commits before merge. labels Oct 16, 2023
@sayboras
Copy link
Member Author

/test

@sayboras sayboras merged commit 33573bd into main Oct 17, 2023
208 of 211 checks passed
@sayboras sayboras deleted the tam/fix-init-config-cmd branch October 17, 2023 12:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/helm Impacts helm charts and user deployment experience release-note/bug This PR fixes an issue in a previous release of Cilium.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants