Skip to content

feat(chart): add custom args, hostPathMounts, and devices config#206

Open
aclerici38 wants to merge 4 commits into
squat:mainfrom
aclerici38:chart/more-config
Open

feat(chart): add custom args, hostPathMounts, and devices config#206
aclerici38 wants to merge 4 commits into
squat:mainfrom
aclerici38:chart/more-config

Conversation

@aclerici38

Copy link
Copy Markdown
Contributor

adds configuration for args,hostPath mounts, and devices to the chart.

Mounts can be configured under hostPathMounts in the values. The current default of /dev remains unless a mount under hostPathMounts mounts /dev (if someone wants to set readOnly for example) or begins with /dev/. For example if someone adds /dev/tun the default /dev will be removed.

When devices is set the default devices args are replaced.

Chart version also updated to 0.2.0.

Let me know if this is acceptable, thanks!!

@squat

squat commented May 7, 2026

Copy link
Copy Markdown
Owner

Thank you @aclerici38! I need to run this locally to test that the generated manifests are actually produced with the right structure. It will probably make sense to add some kind of testing to the charts soon to validate that they generate correctly

@boneskewer69

Copy link
Copy Markdown

Hey, what is missing so this feature branch gets approved and merged into main? I would like to use the Helm chart from this feature branch. If there is anything to work on, I am willing to help.

@aclerici38

Copy link
Copy Markdown
Contributor Author

I believe just for @squat to get a chance to test it. Fwiw I've been running it for a minute in my homelab and all seems well https://github.com/aclerici38/home-ops/blob/main/kubernetes/apps/home/generic-device-plugin/hr.yaml

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants