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

nixos: Add config for defining docker sidecars #35

Merged
merged 16 commits into from Aug 29, 2023
Merged

Conversation

antifuchs
Copy link
Collaborator

@antifuchs antifuchs commented Aug 28, 2023

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.

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 antifuchs force-pushed the docker-sidecar branch 4 times, most recently from a557ca6 to ca27e88 Compare August 29, 2023 18:20
It's not going to work (SO MANY bugs with non-root and even rootless
containers), so let's skip all that.
@antifuchs antifuchs marked this pull request as ready for review August 29, 2023 18:40
@antifuchs antifuchs added this pull request to the merge queue Aug 29, 2023
Merged via the queue into main with commit 3c08485 Aug 29, 2023
11 checks passed
@antifuchs antifuchs deleted the docker-sidecar branch August 29, 2023 18:55
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.

Ship a nixos module for registering oci-container sidecars
1 participant