ros-infrastructure / buildfarm_deployment

Apache License 2.0
30 stars 39 forks source link

"Proxy setup broken" after master deployment #212

Closed gavanderhoorn closed 5 years ago

gavanderhoorn commented 5 years ago

After a successful deployment of a master instance, Jenkins complains that: It appears that your reverse proxy set up is broken.

This is on an Ubuntu 16.04 amd64 VM with a default Ubuntu Server install, working DNS and public hostnames. Both pre- and post-jep-200 version of buildfarm_deployment.

gavanderhoorn commented 5 years ago

@RonaldEnsing: FYI.

gavanderhoorn commented 5 years ago

The Jenkins says my reverse proxy setup is broken page discusses this and provides some hints on what to check / how to address this. The last post (by Gary Phillip) helped.

For this particular host, I had to add the following settings to /etc/apache2/sites-available25-master.conf:

AllowEncodedSlashes NoDecode

and append nocanon to ProxyPass / http://localhost:8080/.

I've also (but these appear not to be strictly necessary):

@nuclearsandwich: the proxy warning has popped up on all deployments for me, would it be an idea to add this to the Apache setup in modules/profile/manifests/jenkins/master.pp?

nuclearsandwich commented 5 years ago

@nuclearsandwich: the proxy warning has popped up on all deployments for me, would it be an idea to add this to the Apache setup in modules/profile/manifests/jenkins/master.pp?

Yeah this has been present on all deployments (including the original Trusty canonical buildfarm) and because it never interfered with operation I let it be. But if there's a way to resolve the error message while still being flexible enough to work without too much manual config, I would merrily review the PR.

gavanderhoorn commented 5 years ago

because it never interfered with operation I let it be

yes, it doesn't seem to really impact operations, but a warning/error is a warning/error, and I've noticed people wondering whether something is really broken.

I'll submit a PR.

nuclearsandwich commented 5 years ago

Resolved by @gavanderhoorn in #215. Thanks!