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

Do not cache Valve server-status #93

Closed
wants to merge 2 commits into from

Conversation

Candunc
Copy link

@Candunc Candunc commented Sep 20, 2019

Looking through the logs I noticed that Valve has a server-status url on their servers which seems to report the load of said server, and that Monolithic/Generic caches the response by default

For example, http://cache1-sea1.steamcontent.com/server-status returns

{
	"csid"		"3"
	"load"		"10"
	"cell"		"31"
}

This is a very minor feature, but it could potentially speed up cache misses by honoring what appears to be Valve's automatic load balancing.

I don't have a monolithic cache set up to test this against at the moment, but I should report back sometime this weekend.

Looking through the logs I noticed that Valve has a server-status url on their servers which seems to report the load of said server, and that Monolithic/Generic caches the response by default

For example, http://cache1-sea1.steamcontent.com/server-status returns 

```"status"
{
	"csid"		"3"
	"load"		"10"
	"cell"		"31"
}
```

This is a very minor feature, but it could potentially speed up cache misses by honoring what appears to be Valve's automatic load balancing.
Copy link
Member

@astrolox astrolox left a comment

Choose a reason for hiding this comment

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

This looks good to me, but I'd like to test it out before approving it.

VibroAxe added a commit to lancachenet/monolithic that referenced this pull request Jul 20, 2021
Implements lancachenet/generic#93 as this is no longer a valid comit after switcheroo
VibroAxe added a commit to lancachenet/monolithic that referenced this pull request Oct 9, 2021
@VibroAxe
Copy link
Member

VibroAxe commented Oct 9, 2021

Merged via lancachenet/monolithic#133

@VibroAxe VibroAxe closed this Oct 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants