Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: NixOS/nixpkgs
base: 071b49b38081
Choose a base ref
...
head repository: NixOS/nixpkgs
compare: 3e65f88623ee
Choose a head ref
  • 2 commits
  • 1 file changed
  • 2 contributors

Commits on Aug 30, 2020

  1. nixosTests.systemd-networkd: fix test flakiness

    The original idea for this test was, on top of providing a networkd
    test, to provide newcomers with a sample configuration they could use
    to get started with networkd.
    
    That's precisely why we were doing this systemd tmpfile dance in the
    first place. It was a convenient way to create a runtime file with a
    specific mode and owner.
    
    Sadly, this tmpfile rule made the test flaky. There's a race condition
    between the wireguard interface configured by systemd-networkd and
    systemd-tmpfiles-setup.
    
    Sometimes, networkd is going to try loading the wireguard private key
    file *before* the said file gets created by systemd-tmpfiles.
    
    A perfect solution here would be to create a "After" dependency
    between wg0.netdev and systemd-tmpfiles-setup.service. Sadly, it is
    currently impossible to create such a dependency between a
    networkd-specific unit and a service.
    
    We're removing this tmp file in favor of pointing networkd directly to
    the Nix store. This is clearly something that shouldn't be done in the
    real world for a private file: the store is world-readable. However,
    this is the only way I found to fix this test flakiness for now.
    picnoir committed Aug 30, 2020
    Copy the full SHA
    f63c842 View commit details
    Browse the repository at this point in the history
  2. Merge pull request #96690 from NinjaTrappeur/nin-networkd-test-fix-fl…

    …akiness
    
    nixosTests.systemd-networkd: fix test flakiness
    Ma27 committed Aug 30, 2020
    Copy the full SHA
    3e65f88 View commit details
    Browse the repository at this point in the history