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

URL checking is unreliable #184

Closed
graeme-a-stewart opened this issue Nov 13, 2017 · 3 comments · Fixed by #880
Closed

URL checking is unreliable #184

graeme-a-stewart opened this issue Nov 13, 2017 · 3 comments · Fixed by #880

Comments

@graeme-a-stewart
Copy link
Member

As discussed in #183 with @hegner and @jouvin , we now have so many external URLs in the HSF website we observe a significant failure rate from

  • Websites that are really in downtime for some ~hours
  • Temporary timeouts

Immediate retries would help with the later. A nightly check might help with the former.

However, we should be aware that over complicating the infrastructure for doing this would not be a good solution.

@hegner
Copy link
Member

hegner commented Nov 13, 2017

We should implement a check that only looks at URLs in the respective pull-request. To ensure long-term stability, I would suggest a weekly (not nightly) test. EP/SFT can provide it for the time being.

@jouvin
Copy link
Contributor

jouvin commented Nov 13, 2017

It is not so easy to find the URLs changed in the pull request, this will require a lot of parsing and I am not convinced that this is a good idea. As said somewhere else, I'd start by redoing the check (in the script used) in case of a failure. My experience with failing tests in our web site is that rerunning the test has always resulted in a successful test, except when there was really a problem. Trying this approach for a few weeks should be enough to see if it is an improvement or not.

@graeme-a-stewart
Copy link
Member Author

@JavierCVilla just pointed out this alternative link checker:

https://github.com/endymion/link-checker

Might be worth looking at.

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

Successfully merging a pull request may close this issue.

3 participants