Actions-runner-controller: Pod annotations support

Created on 19 Jan 2021  路  4Comments  路  Source: summerwind/actions-runner-controller

Hi!, this is more a question than a "bug report".

I see that there is support for "EKS IAM role for service accounts" which is pretty nice but how do we enable IAM authentication using other solutions such as https://github.com/jtblin/kube2iam? This one asks for setting pod annotations to match IAM role permissions. It is possible to set up Pod annotations for the Runner's resources?

enhancement

Most helpful comment

@jlsan92 @sp-ludovic-ivain Well I was pretty confused- I believe you can just use RunnerDeployment.Spec.Template.Metadata.Annotations for setting custom runner pod annotations, as I've commented in another issue https://github.com/summerwind/actions-runner-controller/issues/299#issuecomment-775566247

Would you mind giving it a shot?

All 4 comments

@jlsan92 Hey! Apparently it's mostly impossible today. You're using RunnerDeployment, not Runner, right?

Runner annotations are already inherited to runner pods, but there seems no way to set Runner annotations via RunnerDeployment.

https://github.com/summerwind/actions-runner-controller/blob/a63860029a85ac35c2839c19d3bcf3b5294377e5/controllers/runnerdeployment_controller.go#L272

Got the same problem, would love to have support for annotations in RunnerDeployment 馃檹

@jlsan92 @sp-ludovic-ivain Well I was pretty confused- I believe you can just use RunnerDeployment.Spec.Template.Metadata.Annotations for setting custom runner pod annotations, as I've commented in another issue https://github.com/summerwind/actions-runner-controller/issues/299#issuecomment-775566247

Would you mind giving it a shot?

Hi @mumoshu, yes, I was doing it wrong, putting the annotations into the RunnerDeployment's metadata...

Setting it as you said into RunnerDeployment.Spec.Template.Metadata.Annotations is working fine

Thanks!

Was this page helpful?
0 / 5 - 0 ratings

Related issues

BrendanGalloway picture BrendanGalloway  路  3Comments

raj-saxena picture raj-saxena  路  5Comments

missedone picture missedone  路  10Comments

stackdumper picture stackdumper  路  6Comments

Puneeth-n picture Puneeth-n  路  3Comments