charts icon indicating copy to clipboard operation
charts copied to clipboard

Common Library dependency moved - headscale

Open NerdyShawn opened this issue 5 months ago • 5 comments

Chart Name

headscale

Chart Version

0.16.0

Bug Description

Common library dependency chart I believe has moved.

🔴 https://bjw-s.github.io/helm-charts

🟢 https://bjw-s-labs.github.io/helm-charts

values.yaml

using defaults with persistence on for config,

Relevant log output

k logs -n headscale deployments/headscale-headscale 
Defaulted container "headscale-headscale" out of: headscale-headscale, config (init)
2025-07-22T01:38:54Z FTL home/runner/work/headscale/headscale/cmd/headscale/cli/root.go:55 > Error loading config error="fatal error reading config file: Config File \"config\" Not Found in \"[/etc/headscale /root/.headscale /]\""

NerdyShawn avatar Jul 22 '25 01:07 NerdyShawn

You're right that the common library has been moved. I'll need to update the dependency URLs in this repo, but that won't affect installation of any of these charts. Helm bundles dependency charts when a chart is built, so that only breaks my CI and not your installation!

gabe565 avatar Aug 01 '25 19:08 gabe565

It seems to me like /etc/headscale/config.yaml doesn't exist maybe?

gabe565 avatar Aug 01 '25 19:08 gabe565

That sounds about right, I haven't checked on this and if I recall we had an issue with doing a pretty vanilla install of the chart where the config template that was sourced made a pretty empty config. I'll have to test again later but that might be true.

I want to say it was something with the initContainer putting the config down it needed, but it was pretty empty.

NerdyShawn avatar Aug 01 '25 19:08 NerdyShawn

You're right. By default the chart writes a mostly empty file with a comment explaining that envs are preferred, so the file should have existed. I need to update these charts anyways so I'll see if I can replicate.

gabe565 avatar Aug 02 '25 01:08 gabe565

ahh that makes more sense to me now.

When I initially deployed it with minimal values set I assumed it would have installed successfully.

flux get hr -n headscale 
NAME     	REVISION           	SUSPENDED	READY	MESSAGE                                                                                                                           
headscale	0.16.0+ff837f225371	False    	False	Helm install failed for release headscale/headscale-headscale with chart [email protected]+ff837f225371: context deadline exceeded	

The logs do mention config not being found as expected

k logs -n headscale -l app.kubernetes.io/name=headscale
Defaulted container "headscale-headscale" out of: headscale-headscale, config (init)
2025-08-02T02:43:44Z FTL home/runner/work/headscale/headscale/cmd/headscale/cli/root.go:49 > Error loading config error="fatal error reading config file: Config File \"config\" Not Found in \"[/etc/headscale /root/.headscale /]\""

I still need to dive into configuring more later. The init container does say its giving it an empty config, thats on me for not reading that the environment variables still needed

k logs -n headscale -l app.kubernetes.io/name=headscale -c config
Writing empty config

Perhaps an improvement idea could be the init container can drop the link to the config section in the doc's as well, that way the next person who is as excited as I was to install it without reading the doc's can figure that out quicker than I did. 😅

NerdyShawn avatar Aug 02 '25 03:08 NerdyShawn