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

Linkcheck tests fail intermittently on CI #1057

Closed
mortenpi opened this issue Jul 3, 2019 · 2 comments · Fixed by #1295
Closed

Linkcheck tests fail intermittently on CI #1057

mortenpi opened this issue Jul 3, 2019 · 2 comments · Fixed by #1295

Comments

@mortenpi
Copy link
Member

mortenpi commented Jul 3, 2019

The linkcheck tests in test/dochecks.jl fail intermittently on Travis CI.

Usually with:

┌ Warning: linkcheck 'google.com' status: 301, redirects to http://www.google.com/.
└ @ Documenter.DocChecks ~/build/JuliaDocs/Documenter.jl/src/DocChecks.jl:222
┌ Warning: `curl -sI --proto =http,https,ftp,ftps https://codecov.io/gh/invenia/LibPQ.jl --max-time 10 -o /dev/null --write-out '%{http_code} %{url_effective} %{redirect_url}'` failed:
│   exception = failed process: Process(`curl -sI --proto =http,https,ftp,ftps https://codecov.io/gh/invenia/LibPQ.jl --max-time 10 -o /dev/null --write-out '%{http_code} %{url_effective} %{redirect_url}'`, ProcessExited(28)) [28]
└ @ Documenter.DocChecks ~/build/JuliaDocs/Documenter.jl/src/DocChecks.jl:205
linkcheck: Test Failed at /Users/travis/build/JuliaDocs/Documenter.jl/test/docchecks.jl:23
  Expression: (doc.internal).errors == Set{Symbol}()
   Evaluated: Set(Symbol[:linkcheck]) == Set(Symbol[])
@iamed2
Copy link
Contributor

iamed2 commented Apr 9, 2020

Error code 28 is a timeout

I think we should either make that timeout configurable (and set it longer for CI) or just set it longer by default.

This is happening to me often on my nightly LibPQ tests.

@mortenpi
Copy link
Member Author

mortenpi commented Apr 9, 2020

Making it configurable seems reasonable, I'd be happy to take a PR for that.

In Documenter, it seems that the issue has resolved itself for now. We're also now linkchecking Documenter's own manual in a separate GitHub Actions job (#1255). My hope is that Actions timeout less, and if it doesn't, a separate workflow will be easier to ignore.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants