apisix-helm-chart icon indicating copy to clipboard operation
apisix-helm-chart copied to clipboard

add extraInitContainers

Open ohayak opened this issue 2 years ago • 2 comments

I added an optional parameter to configure extra initContainers

the purpose of this option is to give user an option to do more customization, in my case it was to patch config files in order to support Stand-alone mode. Stand-alone mode won't be easily deployable before v3.0.0 but for people using v2 it's tricky to make it work without extra customization.

ohayak avatar Oct 18 '22 07:10 ohayak

Can you add more detailed background to the description? Why do we need this? What problem can it solve? thanks

tao12345666333 avatar Oct 18 '22 07:10 tao12345666333

Can you add more detailed background to the description? Why do we need this? What problem can it solve? thanks

description updated

ohayak avatar Oct 18 '22 09:10 ohayak

I added an optional parameter to configure extra initContainers

the purpose of this option is to give user an option to do more customization, in my case it was to patch config files in order to support Stand-alone mode. Stand-alone mode won't be easily deployable before v3.0.0 but for people using v2 it's tricky to make it work without extra customization.

Interesting, I think the way is worthy to be written in the docs. Would you also update the docs to introduce the way you patch the config to support the standalone mode.

tokers avatar Oct 19 '22 00:10 tokers

in my case it was to patch config files in order to support Stand-alone mode.

Could you update the documentation to add a specific description about it?

There is currently a similar document describing stand-alone mode https://github.com/apache/apisix-helm-chart/pull/366

tao12345666333 avatar Oct 19 '22 06:10 tao12345666333

This is also helpful when changing ownership of mounted volumes (e.g. hostpath) when the file-logger plugin is used.

g-javi avatar Nov 03 '22 12:11 g-javi

@tao12345666333 @tokers any update on this?

g-javi avatar Nov 03 '22 23:11 g-javi