-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Action execution duration #29
Comments
20s sounds reasonable 🤔 . The bulk of the time is probably taken by installing Nix + packages.
Correct |
I noticed that install-nix-action is able to complete ~4s: https://github.com/cachix/install-nix-action/actions/runs/8198580795/job/22422336858 |
@dephiros I think for install-nix-action, the 4s is just installing Nix without any packages. Based on the screenshot it looks like ours also took around 5s (from 09:55:27 to 09:55:33). The rest of ~12s was spent on restoring the nix store cache from GitHub. Perhaps there's something we can do to optimize that cache restoration part 🤔 |
cc @kalbasit |
I just had this action take 17 minutes for the following packages: "packages": {
"_1password": "2.30.0",
"terraform": "1.9.8",
"ansible": "2.17.5",
"packer": "1.11.2",
"git": "2.46.1",
"path:./.flakes/azure-cli": "",
"python3": "3.12",
"ansible-lint": "latest"
}, {
description = "A flake to install azure-cli";
inputs = {
nixpkgs.url = "github:NixOS/nixpkgs/nixos-unstable";
flake-utils.url = "github:numtide/flake-utils";
};
outputs = { self, nixpkgs, flake-utils, }:
flake-utils.lib.eachDefaultSystem (system:
let pkgs = nixpkgs.legacyPackages.${system};
in {
formatter = pkgs.nixfmt;
packages = {
default = pkgs.azure-cli.withExtensions
[ pkgs.azure-cli-extensions.terraform ];
};
});
} Using the action as follows: - name: Install devbox
uses: jetify-com/[email protected]
with:
enable-cache: 'true' |
Running the action is taking about 20 seconds to complete.
does it sound reasonable? any suggestions how to make it faster?
Looking at the log, this the longest part of the action:
I tried changing the input
skip-nix-installation
tofalse
which fail the action,I guess this variable can help in case the github runner already has nix installed.
adding the
devbox.json
just in case:Thanks!
The text was updated successfully, but these errors were encountered: