Skip to content

Set the right initial permissions for the download dirs - #214

Merged
flavio merged 1 commit into
flavio:mainfrom
haoqixu:fix-dir-perms
Jul 9, 2025
Merged

Set the right initial permissions for the download dirs#214
flavio merged 1 commit into
flavio:mainfrom
haoqixu:fix-dir-perms

Conversation

@haoqixu

@haoqixu haoqixu commented Jul 4, 2025

Copy link
Copy Markdown
Contributor

This PR sets the right initial permissions for the download dirs.

Before this PR, if we git clone this repo and run go run ./cmd/kuberlr kubectl, we will see the following errors:

$ go run ./cmd/kuberlr kubectl
I0704 16:18:21.135572  681781 versioner.go:115] Right kubectl missing, downloading version 1.33.2
F0704 16:18:21.135867  681781 main.go:81] mkdir /home/<username>/.kuberlr/linux-amd64: permission denied
exit status 255
$ go run ./cmd/kuberlr kubectl
F0704 16:18:35.917452  682022 main.go:67] stat /home/<username>/.kuberlr/kuberlr.conf: permission denied
exit status 255

@flavio

flavio commented Jul 9, 2025

Copy link
Copy Markdown
Owner

ouch, that's a good catch. I fear this is a regression we introduced.

@flavio

flavio commented Jul 9, 2025

Copy link
Copy Markdown
Owner

thanks for the fix

@flavio
flavio merged commit 5fe1bab into flavio:main Jul 9, 2025
6 checks passed
@flavio

flavio commented Jul 9, 2025

Copy link
Copy Markdown
Owner

JFYI: this now part of the freshly tagged 0.6.1 release

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.

2 participants