-
Notifications
You must be signed in to change notification settings - Fork 13
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
nixos: Add config for defining docker sidecars #35
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This should allow using it in multiple places.
Now, a function generates the commandline args (except for -stateDir, which is by necessity different).
These _very likely_ won't work, but it's a start.
antifuchs
force-pushed
the
docker-sidecar
branch
from
August 29, 2023 15:20
60daa53
to
0ba118a
Compare
antifuchs
force-pushed
the
docker-sidecar
branch
from
August 29, 2023 15:31
4363db7
to
6363f8f
Compare
antifuchs
force-pushed
the
docker-sidecar
branch
from
August 29, 2023 15:32
6363f8f
to
5533518
Compare
antifuchs
force-pushed
the
docker-sidecar
branch
4 times, most recently
from
August 29, 2023 18:20
a557ca6
to
ca27e88
Compare
This is strongly inspired by NixOS/nixpkgs#138423
antifuchs
force-pushed
the
docker-sidecar
branch
from
August 29, 2023 18:23
ca27e88
to
16d77cf
Compare
It's not going to work (SO MANY bugs with non-root and even rootless containers), so let's skip all that.
antifuchs
force-pushed
the
docker-sidecar
branch
from
August 29, 2023 18:33
4fcaeeb
to
9ddf604
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This fixes #28, when done.
There is one really unfortunate bit about this: You can't run this rootless or as a non-root user, really (due to several problems on nixos's oci-containers side: NixOS/nixpkgs#138423, NixOS/nixpkgs#207050, containers/podman#18903 and a fun bug in podman too: containers/podman#19800).
...but otherwise, this works really well for me, at least under podman.