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

fix: maps axon ports correctly #269

Merged
merged 2 commits into from
Oct 14, 2024
Merged

fix: maps axon ports correctly #269

merged 2 commits into from
Oct 14, 2024

Conversation

grantdfoster
Copy link
Contributor

@grantdfoster grantdfoster commented Oct 14, 2024

Description

After release, noticed a bug on mainnet where the ports were not being correctly mapped to the axon, even though the Makefile was passing --axon.port correctly. The likely culprit is/was an upgrade to bittensor, and the bug was not caught until running multiple axons on mainnet.

Updated code for the miner and validator base:

self.axon = bt.axon(
  wallet=self.wallet, config=self.config, port=self.config.axon.port
)

By explicitly passing an axon port, we resolve the issue where an axon is trying to be served by the default ports, 0891 for miners and 0892 for validators.

Signed commits

  • Yes, I signed my commits.

@grantdfoster grantdfoster merged commit 207c9fb into main Oct 14, 2024
3 checks passed
@grantdfoster grantdfoster deleted the fix--axon-port-mapping branch October 14, 2024 23:36
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