-
Notifications
You must be signed in to change notification settings - Fork 29.7k
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
403 Forbidden for the most recent coverage reports #36040
Comments
I too get it - let me put it in the core repo. /cc @nodejs/website @nodejs/build may be? |
AFAIK there were some c8 changes recently? Maybe related? |
If I had to guess, it is/was a full disk. But that's totally a guess. Unrelated to c8, I'm pretty sure. Some successful new-c8 runs happened. Not seeing any builds at https://ci.nodejs.org/view/All/job/node-test-commit-coverage/ which is puzzling (unless that's not where we run this stuff anymore?). Kicking off a run there to see if it magically fixes itself because why not? https://ci.nodejs.org/view/All/job/node-test-commit-coverage/62/ |
I think https://ci.nodejs.org/job/node-test-commit-linux-coverage-daily/ is the job (nodejs/build#2468 recently updated the link in nodejs/build). Looks to be broken for the last few days -- haven't looked into it but maybe the job was broken by recent changes/cleanups to get the coverage working with GitHub Actions? |
Looks like the coverage runs OK but the transfer of the report to the server fails?
|
@richardlau maybe we can pair on getting that fixed up. We currently need to do periodic cleanup or the rsync times out. If we can do that together then maybe you can help improve as I've not been able to prioritise that. If you can find a hour in my calendar tomorrow we could do it together. |
@mhdawson unfortunately it looks like both of our calendars tomorrow are fairly full, although I may get an hour back if the Release WG meeting gets cancelled. I also don't appear to be able to ssh into
|
@richardlau I seemed to be able to login with the test key that I have ok. I rebooted the machine. Can you check if that resolves the problem and if not find a time in our calendar that will work even if its not tomorrow. |
@mhdawson I now can't connect to |
I've done a soft and a hard reboot of that machine and it's not coming back online, ruh roh |
One last thing to try is a power off/on. Doing that now |
Power on/off did not work. That machine was only being used for benchmarks Benchmark WG is no longer active and I have an open issue to de-charter so getting the machine working for the benchmarks does not make sense. There is also an effort to transition to be newer/better coverage data capture approach, it's just not quite ready yet. In the interim |
As previously mentioned We've also disabled the |
It's quite possible the job itself will need to be changed to take into account the changes in #35767. I'll take a look on Monday. |
Last few runs of https://ci.nodejs.org/job/node-test-commit-linux-coverage-daily/ have been green and the links for the last few days on https://coverage.nodejs.org/ appear to be working. I'll close this out but feel free to reopen if there's anything amiss. I've also tidied up the job a bit to stop installing and running |
Thanks for doing this! 🙂 |
Why am I getting a 403 Forbidden when I'm trying to see the most recent coverage reports?
The text was updated successfully, but these errors were encountered: