Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rootless causes invalid buildkitd mount path to be used #111

Merged
merged 1 commit into from
Jul 17, 2023

Conversation

steved
Copy link
Contributor

@steved steved commented Jul 17, 2023

The error looked like this:

time="2023-07-17T19:20:12Z" level=warning msg="currently, only the default worker can be used."
time="2023-07-17T19:20:12Z" level=warning msg="TLS is disabled for unix:///run/user/%!s(float64=1000)/buildkit/buildkitd.sock"
buildkitd: mkdir /run/user/%!s(float64=1000): permission denied
[rootlesskit:child ] error: command [buildkitd --config /etc/buildkit/buildkitd.toml] exited: exit status 1
[rootlesskit:parent] error: child exited: exit status 1

Before:

$ helm template . -s templates/buildkit/configmap.yaml
[snip]
address = [ "tcp://0.0.0.0:1234", "unix:///run/user/%!s(float64=1000)/buildkit/buildkitd.sock" ]

$ helm template . -s templates/buildkit/configmap.yaml --set buildkit.rootlessUser=1234
[snip]
address = [ "tcp://0.0.0.0:1234", "unix:///run/user/%!s(int64=1234)/buildkit/buildkitd.sock" ]

After:

$ helm template . -s templates/buildkit/configmap.yaml
[snip]
address = [ "tcp://0.0.0.0:1234", "unix:///run/user/1000/buildkit/buildkitd.sock" ]

$ helm template . -s templates/buildkit/configmap.yaml --set buildkit.rootlessUser=1234
[snip]
address = [ "tcp://0.0.0.0:1234", "unix:///run/user/1234/buildkit/buildkitd.sock" ]

@steved steved requested review from Secretions and fraenkel July 17, 2023 19:32
@steved steved merged commit 8e47fed into main Jul 17, 2023
@steved steved deleted the steved/fix-rootless branch July 17, 2023 22:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants