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

Support running Stratum 1 without Squid #68

Merged
merged 1 commit into from
May 22, 2024
Merged

Conversation

natefoo
Copy link
Member

@natefoo natefoo commented May 21, 2024

Likely a very niche case but Squid won't start on port 80 on a weird virtualization environment I have to run on one of my Stratum 1s, while httpd is fine.

I actually don't fully understand what Squid really gets you on a Stratum 1 since the docs have this to say:

Squid should be used as a frontend to Apache, configured as a reverse proxy. It is recommended to run it on the same machine as Apache instead of a separate machine, to reduce the number of points of failure. In that case caching can be disabled for the data (since there’s no need to store it again on the same disk), but caching is helpful for the responses to geo api calls. Using a squid is also helpful for participating in shared monitoring such as the WLCG Squid Monitor.

I'm not sure how taxing geo api calls are but that would seem to be the only benefit for us, and perhaps could be supported directly in Apache with mod_cache if needed.

@hexylena hexylena merged commit 2709f5f into main May 22, 2024
2 checks passed
@hexylena
Copy link
Member

Yeah i've never understood that either, squid and nginx/apache behaving as a reverse proxy fill a very similar hole for me.

@natefoo
Copy link
Member Author

natefoo commented May 25, 2024

Wow I wish I had a functional memory. Apparently past me was smarter and figured out how to make this work. At least I left notes for myself.

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 this pull request may close these issues.

2 participants