Skip to content

feat(antminer): support stock firmware pool configuration#185

Merged
b-rowan merged 8 commits into256foundation:masterfrom
cfilipescu:feat/antminer-pool-config-from-dan-master
Mar 19, 2026
Merged

feat(antminer): support stock firmware pool configuration#185
b-rowan merged 8 commits into256foundation:masterfrom
cfilipescu:feat/antminer-pool-config-from-dan-master

Conversation

@cfilipescu
Copy link
Collaborator

Summary

  • add Antminer stock firmware pool configuration support through SupportsPoolsConfig::set_pools_config
  • improve Antminer pool discovery by collecting pool data from both RPC and miner config sources, with miner config as fallback when RPC pool data is empty
  • refactor pool parsing in the Antminer v2020 backend to reduce duplication and keep pool mapping logic consistent

Validation

  • cargo check -q

@cfilipescu cfilipescu force-pushed the feat/antminer-pool-config-from-dan-master branch from bf877b3 to d60eb89 Compare March 18, 2026 00:28
@b-rowan
Copy link
Member

b-rowan commented Mar 18, 2026

Main issue here is that the parsing for get_miner_conf.cgi is being put into the get_pools data gathering endpoint, when we actually don't want it there, and it should be moved into get_pools_config. The default implementation for get_pools_config is just there to allow using it if the proper way to get the pool config hasn't been defined.

@cfilipescu cfilipescu requested a review from b-rowan March 18, 2026 17:47
DanNicolau and others added 8 commits March 19, 2026 13:19
Stock Bitmain firmware stores pool configuration in get_miner_conf.cgi
endpoint, which persists even when miners are idle. The RPC 'pools'
command only returns runtime status and can be empty when pools haven't
connected yet.

This change queries both endpoints and falls back to the configuration
when RPC returns no pools, ensuring pool migration works reliably during
firmware installation.
Add password field to PoolData struct and parse the 'pass' key from
get_miner_conf.cgi when falling back to configuration-based pool
detection. All other backends set password to None as they read from
runtime RPC status which does not expose passwords.
@cfilipescu cfilipescu force-pushed the feat/antminer-pool-config-from-dan-master branch from e919b44 to f00428e Compare March 19, 2026 17:19
@b-rowan b-rowan merged commit 574574d into 256foundation:master Mar 19, 2026
2 checks passed
@cfilipescu cfilipescu deleted the feat/antminer-pool-config-from-dan-master branch March 19, 2026 17:22
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.

3 participants