Unverified Commit 02bda2c4 authored by Michael Daniels's avatar Michael Daniels
Browse files

ci/eval/compare: set NixOS test rebuild label on all PRs that do so 

I don't think the worry about confusion applies anymore because nowadays CI
will tell you if you target staging-nixos when you should be targeting staging.

On the other hand, I was confused why kernel updates didn't show up as
NixOS test rebuilds...
parent 098bec5a

ci/eval/compare/default.nix

+1 −6
Original line number Diff line number Diff line
@@ -188,12 +188,7 @@ let 
            kernel: rebuilds: lib.nameValuePair "10.rebuild-${kernel}-stdenv" (lib.elem "stdenv" rebuilds)

          ) rebuildsByKernel

          // {

            "10.rebuild-nixos-tests" =

              lib.elem "nixosTests.simple" (extractPackageNames diffAttrs.rebuilds)

              &&

                # Only set this label when no other label with indication for staging has been set.

                # This avoids confusion whether to target staging or batch this with kernel updates.

                lib.last (lib.sort lib.lessThan (lib.attrValues rebuildCountByKernel)) <= 500;

            "10.rebuild-nixos-tests" = lib.elem "nixosTests.simple" (extractPackageNames diffAttrs.rebuilds);

          };

      }

    );