Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

When starting gha-runner-scale-set in "dind" mode, I would like to be able to change the Dockerd settings in the Pod using the values.yaml. #3403

Closed
t-matsu200 opened this issue Apr 3, 2024 · 4 comments
Labels
community Community contribution enhancement New feature or request needs triage Requires review from the maintainers

Comments

@t-matsu200
Copy link

What would you like added?

When gha-runner-scale-set is started in "dind" mode, I would like to be able to change the proxy settings and insecure-registry settings for Dockerd on the Pod from values.yaml.

Why is this needed?

When starting gha-runner-scale-set in "dind" mode behind a proxy server, it was not possible to retrieve container images from the internet when running GitHub Actions workflows.
Additionally, it was not possible to retrieve images from container registries with self-certificates applied.
If I modify the settings of "gha-runner-scale-set.dind-container" appropriately, it will behave as expected.
I would like to be able to change proxy settings and insecure-registry settings without having to perform these workarounds.

Actually, I modified the following parts and started gha-runner-scale-set.

◆Added "HTTP_PROXY", "HTTPS_PROXY" and "NO_PROXY"
https://github.com/actions/actions-runner-controller/blob/master/charts/gha-runner-scale-set/templates/_helpers.tpl#L104-L106

◆Mount ConfigMap to "/etc/docker/daemon.json"
https://github.com/actions/actions-runner-controller/blob/master/charts/gha-runner-scale-set/templates/_helpers.tpl#L109-L115

Additional context

Add any other context or screenshots about the feature request here.

@t-matsu200 t-matsu200 added community Community contribution enhancement New feature or request needs triage Requires review from the maintainers labels Apr 3, 2024
Copy link
Contributor

github-actions bot commented Apr 3, 2024

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.

@Hazmi35
Copy link

Hazmi35 commented Apr 12, 2024

I need to modify dockerd args on dind container, turns out it is not templated and is hardcoded directly at

{{- define "gha-runner-scale-set.dind-container" -}}
image: docker:dind
args:
- dockerd
- --host=unix:///var/run/docker.sock
- --group=$(DOCKER_GROUP_GID)
env:
- name: DOCKER_GROUP_GID
value: "123"
securityContext:
privileged: true
volumeMounts:
- name: work
mountPath: /home/runner/_work
- name: dind-sock
mountPath: /var/run
- name: dind-externals
mountPath: /home/runner/externals
{{- end }}

@Hazmi35
Copy link

Hazmi35 commented Apr 12, 2024

I need to modify dockerd args on dind container, turns out it is not templated and is hardcoded directly at

{{- define "gha-runner-scale-set.dind-container" -}}
image: docker:dind
args:
- dockerd
- --host=unix:///var/run/docker.sock
- --group=$(DOCKER_GROUP_GID)
env:
- name: DOCKER_GROUP_GID
value: "123"
securityContext:
privileged: true
volumeMounts:
- name: work
mountPath: /home/runner/_work
- name: dind-sock
mountPath: /var/run
- name: dind-externals
mountPath: /home/runner/externals
{{- end }}

cc @nikola-jokic.
I apologize for the mention, but as you are one of the maintainers who frequently commits here.

This issue is labeled community but the author clearly states this is for gha-runner-scale-set. It is kinda confusing where to report bug and or feature request regarding helm charts for gha-runner-scale-set.

I am not even sure if this is a feature request or a bug because is this intended or something that need to be addressed? I see that there's also PR related to this issue: #3394

@nikola-jokic
Copy link
Member

Hey everyone,

@Hazmi35 you are right, this one is for the gha-runner-scale-set.
Feature requests related to the gha-runner-scale-set should be submitted to the GitHub Community Support Forum:
image

That being said, we documented that:

To customize the spec, comment out or remove containerMode, and append the configuration you want in the template section.

It should be easy since we documented the exact setup in values.yaml file that would be used for that container mode. I will close this issue here. ☺️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community Community contribution enhancement New feature or request needs triage Requires review from the maintainers
Projects
None yet
Development

No branches or pull requests

3 participants