community icon indicating copy to clipboard operation
community copied to clipboard

Allow configuration of volumes and environment variables in the controller charts

Open olemarkus opened this issue 3 years ago • 7 comments

Is your feature request related to a problem? In particular, I am interested in being able to add the necessary configuration options to support IRSA without using the identity webhook.

Describe the solution you'd like In simplest terms, it is a matter of allowing volumes, volumeMounts, and env to be set through helm charts. These are very common chart configuration options for other purposes as well.

olemarkus avatar Aug 26 '21 06:08 olemarkus

Sure. I could also see annotations, affinities, tolerations and node selectors being useful.

RedbackThomson avatar Aug 26 '21 20:08 RedbackThomson

@olemarkus what do you envision the Deployment for an ACK controller to need additional volume mounts for? I can definitely see the need for additional envVars, though.

jaypipes avatar Aug 30 '21 19:08 jaypipes

As mentioned, this is needed for IRSA. See https://kubernetes.io/docs/tasks/configure-pod-container/configure-service-account/#service-account-token-volume-projection

olemarkus avatar Aug 30 '21 20:08 olemarkus

Issues go stale after 90d of inactivity. Mark the issue as fresh with /remove-lifecycle stale. Stale issues rot after an additional 30d of inactivity and eventually close. If this issue is safe to close now please do so with /close. Provide feedback via https://github.com/aws-controllers-k8s/community. /lifecycle stale

ack-bot avatar Nov 28 '21 23:11 ack-bot

/remove-lifecycle stale

a-hilaly avatar Nov 29 '21 09:11 a-hilaly

Issues go stale after 90d of inactivity. Mark the issue as fresh with /remove-lifecycle stale. Stale issues rot after an additional 30d of inactivity and eventually close. If this issue is safe to close now please do so with /close. Provide feedback via https://github.com/aws-controllers-k8s/community. /lifecycle stale

ack-bot avatar Feb 27 '22 11:02 ack-bot

/lifecycle frozen

vijtrip2 avatar Feb 28 '22 16:02 vijtrip2