Skip to content
This repository was archived by the owner on Nov 6, 2023. It is now read-only.

Fixcppcheck1.osuosl.org:8000 - #20152

Merged
zoracon merged 5 commits into
EFForg:masterfrom
guillaume-uH57J9:guillaume-uH57J9-patch-1
Nov 30, 2021
Merged

Fixcppcheck1.osuosl.org:8000#20152
zoracon merged 5 commits into
EFForg:masterfrom
guillaume-uH57J9:guillaume-uH57J9-patch-1

Conversation

@guillaume-uH57J9

Copy link
Copy Markdown
Contributor

Fixes #20151

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • Existing Ruleset

@zoracon zoracon left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove this domain since it can't be resolved
oprod-controller-openpower.osuosl.org

Remove oprod-controller1.osuosl.org because it does not (longer) resolves
@guillaume-uH57J9

Copy link
Copy Markdown
Contributor Author

@zoracon with pleasure. done!

@guillaume-uH57J9

guillaume-uH57J9 commented Nov 24, 2021

Copy link
Copy Markdown
Contributor Author

The one automated test that fails is because of a timeout when accessing https://oprod-controller1.osuosl.org/
cf https://app.travis-ci.com/github/EFForg/https-everywhere/jobs/549573382

The change I have submitted does not touch this subdomain, and is not causing this error.
Please consider accepting it.

I did check and this URL load for me https://oprod-controller1.osuosl.org/ so I guess it was either temporarily unavailable during the test, or there may be a different kind of network issue between that subdomain and the CI server.

Edit: now that I think of it, I suspect there is rate limiting in effect on this server. I recently had connection failure with another subdomain after making several queries. It's possible CI is running into rate limiting on the remote server

guillaume-uH57J9 and others added 2 commits November 27, 2021 12:53
Remove oprod-controller-openpower.osuosl.org because it does not (longer) resolves
@guillaume-uH57J9

Copy link
Copy Markdown
Contributor Author

I made a typo in a previous commit message (confusion between hostnames) so I amended it

@zoracon
zoracon merged commit 88adc10 into EFForg:master Nov 30, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

osuosl.org, unable to load daca@home results page

2 participants