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

[DOCFIX] makes plugin installation via initContainer more robust #7305

Merged
merged 4 commits into from
Nov 13, 2023

Conversation

jeanfabrice
Copy link
Contributor

@jeanfabrice jeanfabrice commented Nov 9, 2023

Removes plugin before installing it to prevent edge cases where initContainer starts crashing loopback when plugin installation failed partially.

remove plugin before installing it to prevent edge cases where initContainer starts crashing loopback after installation failed
@botelastic botelastic bot added the triage label Nov 9, 2023
@thbkrkr thbkrkr added the >docs Documentation label Nov 9, 2023
@botelastic botelastic bot removed the triage label Nov 9, 2023
@thbkrkr thbkrkr added the v2.11.0 label Nov 9, 2023
thbkrkr

This comment was marked as resolved.

@jeanfabrice
Copy link
Contributor Author

I added a second commit to fix the other files as well

Co-authored-by: Thibault Richard <thbkrkr@users.noreply.github.com>
…ndles-plugins.asciidoc

Co-authored-by: Jean-Fabrice Bobo <2949987+jeanfabrice@users.noreply.github.com>
@thbkrkr thbkrkr merged commit 588bc57 into main Nov 13, 2023
@jeanfabrice
Copy link
Contributor Author

Thank you Thibault!

@jeanfabrice jeanfabrice deleted the jeanfabrice-patch-1 branch November 13, 2023 17:52
robbavey pushed a commit to robbavey/cloud-on-k8s that referenced this pull request Dec 20, 2023
…lastic#7305)

Removes plugin before installing it to prevent edge cases where initContainer starts
crashing loopback when plugin installation failed partially.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>docs Documentation v2.11.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants