-
Notifications
You must be signed in to change notification settings - Fork 936
fix(dogfood/contents): fix resource metadata under nix #16349
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
Conversation
Change-Id: Ie978c62ce4164386e55cedd6d7ceae71b07e217e Signed-off-by: Thomas Kosiewski <tk@coder.com>
EDIT: I was building |
|
61d4348
to
387cafb
Compare
I added a script in init.d to start the docker daemon with the proper group set up in the background.
Ca-certificates do get installed, the explicit SSL_CERT_FILE is most likely not getting preserved when running |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dockerd starts successfully under sysbox 0.6.6. There's a separate issue with the build scripts:
[nix-shell:~/coder]$ make build
getopt: unrecognized option '--version'
BusyBox v1.36.1 () multi-call binary.
Usage: getopt [OPTIONS] [--] OPTSTRING PARAMS
TIL that busybox has its own slightly different version of getopt
🙃
387cafb
to
d52008c
Compare
Yep, I removed busybox entirely and pulled in ncurses for When running |
c89b06d
to
3bcc415
Compare
|
Change-Id: I530de9066ea94ab54488de6e83ed64e7d44a1d72 Signed-off-by: Thomas Kosiewski <tk@coder.com>
3bcc415
to
b1277e9
Compare
I'll go ahead and merge this PR. I will need to make a follow-up PR, as we have some tests depending on hard-coded OS values (mostly clistat package relying on Elastic's library, which in turn expects OS information in |
No description provided.