Branch: refs/heads/master
  Home:   https://github.com/NixOS/nixpkgs
  Commit: fb46df8a9a4102e265f4b14af48a5df90d5b06c3
      
https://github.com/NixOS/nixpkgs/commit/fb46df8a9a4102e265f4b14af48a5df90d5b06c3
  Author: aszlig <asz...@redmoonstudios.org>
  Date:   2016-09-07 (Wed, 07 Sep 2016)

  Changed paths:
    M nixos/modules/services/networking/firewall.nix
    M nixos/tests/firewall.nix

  Log Message:
  -----------
  nixos: Fix ordering of firewall.service

Follow-up to the following commits:

  abdc5961c3cdf9f5893ea1e91ba08ff5089f53a4: Fix starting the firewall
  e090701e2d09aec3e8866ab9a8e53c37973ffeb4: Order before sysinit

Solely use sysinit.target here instead of multi-user.target because we
want to make sure that the iptables rules are applied *before* any
socket units are started.

The reason I've dropped the wantedBy on multi-user.target is that
sysinit.target is already a part of the dependency chain of
multi-user.target.

To make sure that this holds true, I've added a small test case to
ensure that during switch of the configuration the firewall.service is
considered as well.

Tested using the firewall NixOS test.

Signed-off-by: aszlig <asz...@redmoonstudios.org>
Cc: @edolstra


_______________________________________________
nix-commits mailing list
nix-comm...@lists.science.uu.nl
http://lists.science.uu.nl/mailman/listinfo/nix-commits

Reply via email to