Argo: Support initContainers

Created on 11 Jan 2019 · 2Comments · Source: argoproj/argo

IFEATURE REQUEST?:

I'm evaluating argo and noticed that the Workflow Spec doesn't allow for initContainers (it looks like you use an initContainer in the pods you generate, but don't allow additional ones to be provided).
Would you consider a PR to add this? We use initContainers for our secret injection, and it would be a lot easier if we could specify them directly.

We could add one via an mutating admission webhook, but I've avoided that so far.

Source

tcolgate

👍6

Most helpful comment

I also want this feature, so I implemented it.
https://github.com/argoproj/argo/pull/1183

dtaniwaki on 18 Jan 2019

👍4

All 2 comments

I also want this feature, so I implemented it.
https://github.com/argoproj/argo/pull/1183

dtaniwaki on 18 Jan 2019

👍4

Sweet. Hope it gets merged!

Cheers,
Wes

On Fri, Jan 18, 2019, 6:41 AM Daisuke Taniwaki <[email protected]
wrote:

I also want this feature, so I implemented it.

1183 https://github.com/argoproj/argo/pull/1183

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/argoproj/argo/issues/1169#issuecomment-455568300, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA3JQKqfKgK_vIlI7CdhnfWwDLVUluIgks5vEd0hgaJpZM4Z7jFq
.