forked from NixOS/nixpkgs
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
ci/eval: re-implement compare in nix
- Loading branch information
1 parent
bf6b383
commit 57c1f54
Showing
5 changed files
with
193 additions
and
182 deletions.
There are no files selected for viewing
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,53 @@ | ||
{ | ||
lib, | ||
jq, | ||
runCommand, | ||
writeText, | ||
supportedSystems, | ||
... | ||
}: | ||
{ beforeResultDir, afterResultDir }: | ||
let | ||
inherit (import ./utils.nix { inherit lib; }) | ||
diff | ||
groupByKernel | ||
extractPackageNames | ||
getLabels | ||
uniqueStrings | ||
; | ||
|
||
getAttrs = dir: builtins.fromJSON (builtins.readFile "${dir}/outpaths.json"); | ||
beforeAttrs = getAttrs beforeResultDir; | ||
afterAttrs = getAttrs afterResultDir; | ||
|
||
diffAttrs = diff beforeAttrs afterAttrs; | ||
|
||
changed-paths = | ||
let | ||
rebuilds = uniqueStrings (diffAttrs.added ++ diffAttrs.changed); | ||
|
||
rebuildsByKernel = groupByKernel rebuilds; | ||
rebuildCountByKernel = lib.mapAttrs ( | ||
kernel: kernelRebuilds: lib.length kernelRebuilds | ||
) rebuildsByKernel; | ||
in | ||
writeText "changed-paths.json" ( | ||
builtins.toJSON { | ||
attrdiff = lib.mapAttrs (_: v: extractPackageNames v) diffAttrs; | ||
inherit rebuildsByKernel rebuildCountByKernel; | ||
labels = getLabels rebuildCountByKernel; | ||
} | ||
); | ||
in | ||
runCommand "compare" | ||
{ | ||
nativeBuildInputs = [ jq ]; | ||
} | ||
'' | ||
mkdir $out | ||
cp ${changed-paths} $out/changed-paths.json | ||
jq -r -f ${./generate-step-summary.jq} < ${changed-paths} > $out/step-summary.md | ||
# TODO: Compare eval stats | ||
'' |
File renamed without changes.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,130 @@ | ||
{ lib, ... }: | ||
rec { | ||
# Borrowed from https://github.com/NixOS/nixpkgs/pull/355616 | ||
uniqueStrings = list: builtins.attrNames (builtins.groupBy lib.id list); | ||
|
||
_processSystemPath = | ||
packageSystemPath: | ||
let | ||
# python312Packages.torch.aarch64-linux -> ["python312Packages" "torch" "aarch64-linux"] | ||
# splittedPath = lib.splitString "." attrName; | ||
splittedPath = lib.splitString "." packageSystemPath; | ||
|
||
# ["python312Packages" "torch" "aarch64-linux"] -> ["python312Packages" "torch"] | ||
packagePath = lib.sublist 0 (lib.length splittedPath - 1) splittedPath; | ||
in | ||
{ | ||
# "python312Packages.torch" | ||
name = lib.concatStringsSep "." packagePath; | ||
|
||
# "aarch64-linux" | ||
system = lib.last splittedPath; | ||
}; | ||
|
||
# Turns | ||
# [ | ||
# "hello.aarch64-linux" | ||
# "hello.x86_64-linux" | ||
# "hello.aarch64-darwin" | ||
# "hello.x86_64-darwin" | ||
# "bye.x86_64-darwin" | ||
# "bye.aarch64-darwin" | ||
# ] | ||
# | ||
# into | ||
# | ||
# [ | ||
# "hello" | ||
# "bye" | ||
# ] | ||
extractPackageNames = | ||
packageSystemPaths: | ||
builtins.attrNames ( | ||
builtins.removeAttrs (builtins.groupBy ( | ||
packageSystemPath: (_processSystemPath packageSystemPath).name | ||
) packageSystemPaths) [ "" ] | ||
); | ||
|
||
# Computes a diff between two attrs | ||
# { | ||
# added: [ <keys only in the second object> ], | ||
# removed: [ <keys only in the first object> ], | ||
# changed: [ <keys with different values between the two objects> ], | ||
# } | ||
# | ||
diff = | ||
let | ||
filterKeys = cond: attrs: lib.attrNames (lib.filterAttrs cond attrs); | ||
in | ||
old: new: { | ||
added = filterKeys (n: _: !(old ? ${n})) new; | ||
removed = filterKeys (n: _: !(new ? ${n})) old; | ||
changed = filterKeys ( | ||
n: v: | ||
# Filter out attributes that don't exist anymore | ||
(new ? ${n}) | ||
|
||
# Filter out attributes that are the same as the new value | ||
&& (v != (new.${n})) | ||
) old; | ||
}; | ||
|
||
# Turns | ||
# [ | ||
# "hello.aarch64-linux" | ||
# "hello.x86_64-linux" | ||
# "hello.aarch64-darwin" | ||
# "hello.x86_64-darwin" | ||
# "bye.x86_64-darwin" | ||
# "bye.aarch64-darwin" | ||
# ] | ||
# | ||
# into | ||
# | ||
# { | ||
# linux = [ | ||
# "hello" | ||
# ]; | ||
# darwin = [ | ||
# "hello" | ||
# "bye" | ||
# ]; | ||
# } | ||
groupByKernel = | ||
systemPaths: | ||
let | ||
systemPaths' = builtins.map _processSystemPath systemPaths; | ||
|
||
filterKernel = | ||
kernel: | ||
builtins.attrNames ( | ||
builtins.groupBy (systemPath: systemPath.name) ( | ||
builtins.filter (systemPath: lib.hasSuffix kernel systemPath.system) systemPaths' | ||
) | ||
); | ||
in | ||
lib.genAttrs [ "linux" "darwin" ] filterKernel; | ||
|
||
getLabels = lib.mapAttrs ( | ||
kernel: rebuildCount: | ||
let | ||
number = | ||
if rebuildCount == 0 then | ||
"0" | ||
else if rebuildCount <= 10 then | ||
"1-10" | ||
else if rebuildCount <= 500 then | ||
"101-500" | ||
else if rebuildCount <= 1000 then | ||
"501-1000" | ||
else if rebuildCount <= 2500 then | ||
"1001-2500" | ||
else if rebuildCount <= 5000 then | ||
"2501-5000" | ||
else | ||
"5001+"; | ||
|
||
in | ||
"10.rebuild-${kernel}: ${number}" | ||
); | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters