-
Notifications
You must be signed in to change notification settings - Fork 7.8k
ci: update caddy job to use GitHub Releases #13296
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! It might still make sense to use the timeout values from GH-13295.
7fbc224
to
e99eda3
Compare
Added timeouts, thank you! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change is clearly correct, but I wonder if we should use the gh
CLI binary to download the release (https://cli.github.com/manual/gh_release_download), because I'm not 100% sure if unauthenticated access to the release API might be rate-limited.
In any case this change should be applied to the earliest branch that has the setup-caddy action.
Example: |
e99eda3
to
2981748
Compare
Definitely makes more sense to use |
I tried with gh release -R caddyserver/caddy download --pattern 'caddy_*_linux_amd64.tar.gz' -O - | sudo tar -xz -C /usr/bin caddy In GH Actions, I'm seeing this:
I think it'd be more involved to deal with tokens, so I'd say lets go with the direct links for now. As far as I see from HTTP headers and online, there is no hard limit to those URLs; only |
Not really. That
|
The caddyserver.com download page is unreliable, and it also recommends to use GitHub releases instead.
2981748
to
fe791f1
Compare
Ah perfect! I made a quick test and it worked. Submitting said changes for review. TIL, thank you @TimWolla. |
Now merged into PHP-8.1+. This is a clear improvement, thank you! |
Hmm, did this ever work? Apparently, it fails all the time: https://github.com/php/php-src/actions/runs/10493648028/job/29068097535#step:8:20 The issue had been reported by @Disservin, and I only now had a look. According to https://stackoverflow.com/questions/68951888/gh-is-not-recognized-error-while-trying-to-create-github-repo-from-command, we would need to install |
It seems to be not working on But yeah you are right, lack of |
Related: phpGH-13296. ASAN builds use a different base image on GitHub Actions, which does not have the `gh` binary installed. This adds `gh` to the `apt install` list, to make sure it's available on all builds.
Related: phpGH-13296. ASAN builds use a different base image on GitHub Actions, which does not have the `gh` binary installed. This adds `gh` to the `apt install` list, to make sure it's available on all builds.
The caddyserver.com download page is unreliable, and it also recommends to use GitHub releases instead.
Replaces #13295.