diff options
| author | Tom Meadows <tom@tmlabs.co.uk> | 2025-10-01 12:24:21 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2025-10-01 12:24:21 +0100 |
| commit | af1114e896fd16378dbf8f0584b0d55ebd46930b (patch) | |
| tree | 6580fd7671125075456702b34e6c63f4331e97f3 /control/controlknobs/controlknobs.go | |
| parent | 9781b7c25cbaae314f3ca95741d20c6125a89531 (diff) | |
| download | tailscale-af1114e896fd16378dbf8f0584b0d55ebd46930b.tar.xz tailscale-af1114e896fd16378dbf8f0584b0d55ebd46930b.zip | |
cmd/k8s-proxy: importing feature/condregister on cmd/k8s-proxy (#17383)
https://github.com/tailscale/tailscale/pull/17346 moved the kube and aws
arn store initializations to feature/condregister, under the assumption
that anything using it would use kubestore.New. Unfortunately,
cmd/k8s-proxy makes use of store.New, which compares the `<prefix>:`
supplied in the provided `path string` argument against known stores. If
it doesn't find it, it fallsback to using a FileStore.
Since cmd/k8s-proxy uses store.New to try and initialize a kube store in
some cases (without importing feature/condregister), it silently creates
a FileStore and that leads to misleading errors further along in
execution.
This fixes this issue by importing condregister, and successfully
initializes a kube store.
Updates #12614
Signed-off-by: chaosinthecrd <tom@tmlabs.co.uk>
Diffstat (limited to 'control/controlknobs/controlknobs.go')
0 files changed, 0 insertions, 0 deletions