actions-runner-controller icon indicating copy to clipboard operation
actions-runner-controller copied to clipboard

helm charts for actions runner

Open saravanan30erd opened this issue 2 years ago • 1 comments

Fixes: #942

helm charts for actions runner, currently its having only RunnerDeployment and Autoscaler resources.

Looks like deployment order is important here, facing the below issue if Autoscaler deployed first and then autoscaling not working as expected.

2022-04-21T12:13:08Z    DEBUG    controllers.webhookbasedautoscaler    RunnerDeployment not found with scale target ref name test-actions-runner for hra test-actions-runner-autoscaler

Helm doesn't support ordering for custom resources. So using List to overcome this issue, didn't use helm chart hooks for ordering since its not tracked after creation.

saravanan30erd avatar Apr 22 '22 11:04 saravanan30erd

I was about to start working on this and then saw that you helpfully already have! 😄 One suggestion would be to add the ability to specify the Runner Group. I added this and created saravanan30erd/actions-runner-controller#1 to get that into your branch.

yardbirdsax avatar May 03 '22 14:05 yardbirdsax

@saravanan30erd @yardbirdsax Hey! First of all, thank you so much for your contribution and patience.

Sorry for the radio silence here. We've been discussing how to maintain this. The main problem is that maintaining the manifests, the kustomize config, and the chart for ARC itself is already too much to maintain for a team of only two, and we weren't sure if we could commit to maintain another right now. I do believe this chart is super valuable to the community if maintained well, though. That said, could you let me merge this under a directory like contrib/examples first, and start collecting feedbacks from early adopters/advanced users, and then graduate to the official chart under charts once we gain more confidence?

mumoshu avatar Sep 27 '22 00:09 mumoshu

That said, could you let me merge this under a directory like contrib/examples first, and start collecting feedbacks from early adopters/advanced users, and then graduate to the official chart under charts once we gain more confidence?

@mumoshu Sure.

saravanan30erd avatar Sep 27 '22 09:09 saravanan30erd