Skip to content
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

source-url documentation is inconsistent on whether it is to the nix-installer binary or to a script that downloads it #107

Closed
lf- opened this issue Jul 10, 2024 · 5 comments

Comments

@lf-
Copy link

lf- commented Jul 10, 2024

image

It seems that for whatever reason using https://install.lix.systems/lix as source-url on macOS returns permission denied like so: https://github.com/hazelweakly/nixos-configs/actions/runs/9877071030/job/27278362955

Further, @grahamc tweeted about using it with the binary package. So I think it actually is for the binary package. But then you have to inject the platform into that string, which is kind of ugly. https://x.com/grhmc/status/1787469491430989912

I am not sure why the permission denied error happened. The part that's especially confusing with that is that the lix-installer.sh does not include the string "nix-installer" anywhere which means that whatever broke here was actually in nix-installer-action as far as I can tell.

@grahamc
Copy link
Member

grahamc commented Jul 10, 2024

Thanks for pointing out the inconsistency. It does need to be the executable, not the script. Our library for making github actions includes automatically caching the downloaded executable into the GitHub Actions Cache, which uses the name of the action as the final binary name: https://github.com/DeterminateSystems/nix-installer-action/blob/main/src/index.ts#L80

The permission denied error is a bit surprising... I'm taking a look...

@grahamc
Copy link
Member

grahamc commented Jul 10, 2024

I sent a PR to fix the inconsistency (#108) -- thank you! I looked but ran out of time researching the permission denied error. I did verify that the cached artifact has an etag which matches the md5sum of the lix script:

##[debug]Checking the tool cache for https://install.lix.systems/lix at "37b484b605499fdebf3e1c88d2ec4336"

...

grahamchristensen@Grahams-MacBook-Pro ~ % md5 ~/Downloads/lix
MD5 (/Users/grahamchristensen/Downloads/lix) = 37b484b605499fdebf3e1c88d2ec4336

so that is good news. I would expect this to work correctly, and would be glad to know why it doesn't...

@hazelweakly
Copy link

Looks like things work when the source-url is set to a platform specific binary, interesting. Although I did have one spurious error that was a timeout issue (unrelated, just a network glitch. $WORK has been having this off and on as well in other places)

https://github.com/hazelweakly/nixos-configs/actions/runs/9879620636/job/27286337171

@grahamc
Copy link
Member

grahamc commented Jul 10, 2024

Gotcha. Can you share a log where there was a timeout? We try pretty hard to have retries built in and enabled in all the relevant spots.

@grahamc
Copy link
Member

grahamc commented Jul 15, 2024

I'm going to go ahead and close this. Feel free to re-open if you have more info!

@grahamc grahamc closed this as completed Jul 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants