Skip to content

Conversation

@camfairchild
Copy link
Contributor

@camfairchild camfairchild commented Aug 17, 2023

This PR adds a new custom RPC call to grab a new storage of IPs associated with the current subnet validator-set.
This will help Servers correctly and quickly update their firewalls.
This will also allow Validators to associate multiple IP addresses with themselves

Waiting on subtensor PR (and subsequent chain upgrade for opentensor/subtensor#179)

@camfairchild camfairchild changed the base branch from staging to new_staging August 17, 2023 18:33
@camfairchild camfairchild force-pushed the add-associated-ip-info branch from cb4ffed to d18a665 Compare August 17, 2023 18:35
Copy link
Contributor

@ifrit98 ifrit98 left a comment

Choose a reason for hiding this comment

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

Looks good!

@ifrit98
Copy link
Contributor

ifrit98 commented Aug 18, 2023

Small thing, just reformat with black and we're solid.

@camfairchild
Copy link
Contributor Author

Waiting on subtensor PR opentensor/subtensor#179 and subtensor endpoint node upgrades

@mogmachine
Copy link
Contributor

Just a clarification ... this will expose app IPs publicly to miners/network?

@camfairchild
Copy link
Contributor Author

Just a clarification ... this will expose app IPs publicly to miners/network?

Exactly. The idea is you should accept connections from any IP in the list

@ifrit98
Copy link
Contributor

ifrit98 commented Sep 27, 2023

Already in revolution.

@ifrit98 ifrit98 closed this Sep 27, 2023
@ifrit98 ifrit98 deleted the add-associated-ip-info branch December 14, 2023 20:25
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