Skip to content

server: valve: force eclipse servers to be included - #35

Closed
000exploit wants to merge 1 commit into
FWGS:mainfrom
000exploit:main
Closed

000exploit wants to merge 1 commit into
FWGS:mainfrom
000exploit:main

Conversation

@000exploit

@000exploit 000exploit commented Sep 27, 2026 •

Copy link
Copy Markdown
Contributor

Since my servers sometimes aren't resolvable from GitHub CI for some reason, it seems like forcing the host to always be available is a good idea. Tested the script locally and it resolves fine from my home network.

  [+]        valve  51.91.249.12:27017  ping=38.191ms  players=4
  [-]        valve  2.26.26.83:27015  silent
  [+]        valve  94.138.138.9:27015  ping=116.599ms  players=0
  [+]        valve  94.138.138.9:27016  ping=117.988ms  players=0
  [+]        valve  94.138.138.9:27017  ping=116.594ms  players=0
  [+]        valve  45.154.199.157:27015  ping=64.244ms  players=0 # <- this one
  [+]        valve  45.154.199.157:27016  ping=18.331ms  players=0 # <- and this too
  [+]        valve  89.221.220.61:27016  ping=45.345ms  players=0
  [+]        valve  72.34.103.90:27015  ping=170.227ms  players=0
  [-]        valve  57.129.154.247:27015  silent
  [+]        valve  147.224.170.114:27015  ping=117.89ms  players=0
  -> output/v1/servers/valve  (9 published)

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

@github-actions

Copy link
Copy Markdown
Contributor

Server list PR validation

❌ Not ready — 2 problems: 2 unreachables.

Probed 2 new or changed entries.

Gamedir Address Change Claimed Responder Host Notes
valve 45.154.199.157:27015 force False → True 49 (Xash) — — ❌ no response — server unreachable; will not be published until it answers
valve 45.154.199.157:27016 force False → True 49 (Xash) — — ❌ no response — server unreachable; will not be published until it answers

Probes can be flaky on the first try; the nightly publish workflow re-probes with a 48 h grace window, so a single ❌ here is not necessarily fatal. Re-push to re-run.

@a1batross

Copy link
Copy Markdown
Member

It does answer now from Blacksmith's VMs. In fact, thanks to them, we don't really need this force logic anymore.

@a1batross a1batross closed this Sep 27, 2026
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