Skip to content
This repository was archived by the owner on Aug 6, 2020. It is now read-only.

nspawn: chown() the legacy hierarchy when it's used in a container - #23

Merged
fpletz merged 1 commit into
NixOS:nixos-v239from
flokli:nixos-v239-nspawn
Sep 24, 2018
Merged

fpletz merged 1 commit into
NixOS:nixos-v239from
flokli:nixos-v239-nspawn

Conversation

@flokli

@flokli flokli commented Sep 23, 2018 •

Copy link
Copy Markdown
Member

This fixes nspawn container with older systemd failing to start.

(cherry picked from commit 6132e1c)

See:
systemd#10104
NixOS/nixpkgs#47253

@Mic92

Mic92 commented Sep 23, 2018

Copy link
Copy Markdown
Member

@globin @fpletz I tested this locally with:

systemd.package = pkgs.systemd.overrideAttrs (old: {
  patches = [
    (pkgs.fetchpatch {
      url = "https://github.com/NixOS/systemd/commit/6f7a429e9b0ed2bc1e7e5c24ea5bc70d72c36962.patch";
      sha256 = "0qxghr0vzfkxsb61xmzy5x5lfws6fyvxpcggq4hp406waa3ng1jc";
    })
  ];
 });

and an ubuntu container. Please merge.

@fpletz
fpletz merged commit 31859dd into NixOS:nixos-v239 Sep 24, 2018
@fpletz

fpletz commented Sep 24, 2018

Copy link
Copy Markdown
Member

Thanks!

flokli added a commit to flokli/nixpkgs that referenced this pull request Sep 24, 2018
This fixes nspawn containers with older systemd inside currently failing
to start.

See:
NixOS/systemd#23
systemd/systemd#10104
NixOS#47253
Mic92 pushed a commit to NixOS/nixpkgs that referenced this pull request Sep 24, 2018
This fixes nspawn containers with older systemd inside currently failing
to start.

See:
NixOS/systemd#23
systemd/systemd#10104
#47253
Mic92 pushed a commit to NixOS/nixpkgs that referenced this pull request Sep 24, 2018
This fixes nspawn containers with older systemd inside currently failing
to start.

See:
NixOS/systemd#23
systemd/systemd#10104
#47253
(cherry picked from commit c3cc34f)
@flokli
flokli deleted the nixos-v239-nspawn branch September 24, 2018 17:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants