feat: huge beszel hub and agent helm chart update#1582
feat: huge beszel hub and agent helm chart update#1582cloudwithdan wants to merge 8 commits intohenrygd:mainfrom
Conversation
d87ceb7 to
aa2a849
Compare
|
Do we have an estimation of this PR being merged and chart to release? |
|
Thanks so much for this! And sorry for the delayed reply, I've been mostly off the computer for a few weeks and still catching up on a lot of stuff. I took a quick look and it's definitely a huge improvement. I don't have time to go over it line by line at the moment, so I ran the patch through gemini. It picked out few potential issues I'm wondering if we can address before merging. If any of this is wrong or not necessary, just let me know :) 1. Fix the "TO DO" in the Hub READMEThe 2. Restore the Docker Socket ExamplePatch 3 accidentally removed the Docker socket volume mount example from the Fix: It should be added back to the README and ideally included as a commented-out section in the 3. Add Validation for the SSH Key (env.KEY)The agent requires an SSH public key to work. Currently, if a user forgets to set Fix: Adding a simple 4. Consistent Variable NamingThe documentation uses Fix: Standardize on |
|
Henry, sorry for late contribution but I had to jump in since a lot of people are asking for the helm chart.
Mounting docker socket is not needed since this is a setup for kubernetes. |
|
Hi @cloudwithdan Also for the agent: maybe we could add an "existingSecret" section. The Helm “existing secrets” pattern lets you point a chart to a pre‑created Kubernetes Secret instead of generating one from values. Charts typically expose a field like existingSecret, and when it’s set, templates skip secret creation and simply reference the named Secret. This keeps sensitive data out of values.yaml and allows external secret managers to control the lifecycle. something like this: env:
PORT: "45876"
# Hub URL - OPTIONAL (e.g., http://beszel-hub:8090)
HUB_URL: ""
# Agent name in the Hub - OPTIONAL (defaults to node name)
SYSTEM_NAME: ""
# For GPU support (henrygd/beszel-agent-nvidia only)
# NVIDIA_VISIBLE_DEVICES: "all"
# NVIDIA_DRIVER_CAPABILITIES: "compute,video,utility"
secret:
existingSecret: "" # my-beszel-token
tokenKey: "token"
sshKey: "key"Then within the deployment/daemonset (if the existingSecret value is not empty) mount that secret instead. Also skip the creation of the secret. Also the HUB_URL could be default the one you specified: "http://beszel-hub:8090" so that is really optional. right now the default value is just empty - which makes the agent log out an error, that it can't connect to the hub. |
|
@tim-krehan yes totally, thank you for your suggestion, that really is the best helm chart practice. I will update the helm chart, since I am migrating ingress-nginx to Traefik Gateway API, I will need the httproute myself, I will also add support for Gateway API. 🚀 |
tim-krehan
left a comment
There was a problem hiding this comment.
looks good! Secret works, small typo in the httproute file.
Co-authored-by: Tim Krehan <[email protected]>
Co-authored-by: Tim Krehan <[email protected]>

📃 Description
This PR updates and adds a new helm chart for
beszel-agentand updates the existingbeszel-hub🪵 Changelog
➕ Added
beszel-hubbeszel-agentrelease-pleasegithub actionsRemoved
beszel-agentfrom .gitignore since it was not letting me togit addthe helm chart