actions / actions-runner-controller

Kubernetes controller for GitHub Actions self-hosted runners
Apache License 2.0
4.41k stars 1.04k forks source link

AutoscalingRunnerSet not accepting tolerations for the listener #3424

Closed cyber-kamil closed 2 months ago

cyber-kamil commented 2 months ago

Checks

Controller Version

0.9.0

Deployment Method

Helm

Checks

To Reproduce

Setting values for the helm chart of ghcr.io/actions/actions-runner-controller-charts/gha-runner-scale-set 

listenerTemplate:
  spec:
    containers:
    - name: listener
      tolerations:
      - key: "node-taints.xxxxx"
        value: "app"
        operator: "Equal"
        effect: "NoSchedule"

Describe the bug

When trying to create the runner scale set and trying to set pod tolerations in the listenerTemplate as specified in the documentation the values are not picked up by the listener pods having looked over the definition of AutoscalingRunnerSet it's clear that they aren't picked up. Having looked over the templates it's clear that any tolerations are missing as well as node affinity or anything allowing you to select where the pods are supposed to be created:

https://github.com/actions/actions-runner-controller/blob/master/charts/gha-runner-scale-set/templates/autoscalingrunnerset.yaml

Describe the expected behavior

When setting those values pods should have those tolerations created on them.

Additional Context

Having a setup where every node has some sort of a taint it's nearly impossible to create the AutoscailingRunnerSet as the listeners can't be created.

Controller Logs

2024-04-09T23:27:12Z    INFO    AutoscalingListener Created listener pod    {"autoscalinglistener": {"name":"arc-runner-set-754b578d-listener","namespace":"arc-systems"}, "namespace": "arc-systems", "name": "arc-runner-set-754b578d-listener"}

Runner Pod Logs

Creates them correctly however the pods have an error of: Failed to schedule pod, incompatible with nodepool "xxxx",
github-actions[bot] commented 2 months ago

Hello! Thank you for filing an issue.

The maintainers will triage your issue shortly.

In the meantime, please take a look at the troubleshooting guide for bug reports.

If this is a feature request, please review our contribution guidelines.

cyber-kamil commented 2 months ago

https://github.com/actions/actions-runner-controller/issues/3265 this seems to fix the issue