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

remove IPADDR, which was making Rob feel ill #417

Merged
merged 3 commits into from
Jun 23, 2018

Conversation

rob-solana
Copy link
Contributor

@rob-solana rob-solana commented Jun 23, 2018

IPADDR is simple, which is nice, but not exactly what we need for testnet, where NAT'd
folks need to join in, need to advertize themselves as on the interweb.

myip() helps, but there's some TODOs: fullnode-config probably needs to
be told where it lives in the real world (machine interfaces tell us dick),
or incorporate something like the "ifconfig.co" code in myip.sh

  IPADDR is simple, but not exactly what we need for testnet, where NAT'd
  folks need to join in, need to advertize themselves as on the interweb.

  myip() helps, but there's some TODOs: fullnode-config probably needs to
  be told where it lives in the real world (machine interfaces tell us dick),
  or incorporate something like the "ifconfig.co" code in myip.sh
Copy link
Contributor

@garious garious left a comment

Choose a reason for hiding this comment

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

Loving the small, frequent PRs, thanks!

@aeyakovenko
Copy link
Member

@rob-solana don't you think we will need physical vs virtual IPs? Basically fullnode-config should ask the network for its addr, figure out if its actually reachable on the ports it thinks it has open, but tell the actual node to bind to 0, or whatever the interface can connect tot the network.

@rob-solana
Copy link
Contributor Author

this PR allows the user to decide whether he's going to run a network on his internal network or talk to the real world. the default choice is to use the interface that presents in the real world (via http://ifconfig.co)

@garious
Copy link
Contributor

garious commented Jun 23, 2018

rebase and shellcheck? Feel free to automerge on successful CI

@mvines
Copy link
Member

mvines commented Jun 23, 2018

(The publish-snap ci job is still red so auto merge will fail. Fixing this today...)

@rob-solana rob-solana merged commit 55fc647 into solana-labs:master Jun 23, 2018
@rob-solana rob-solana deleted the fix-IPADDR branch June 23, 2018 23:00
vkomenda pushed a commit to vkomenda/solana that referenced this pull request Aug 29, 2021
Bumps [node-fetch](https://github.com/bitinn/node-fetch) from 2.6.0 to 2.6.1.
- [Release notes](https://github.com/bitinn/node-fetch/releases)
- [Changelog](https://github.com/node-fetch/node-fetch/blob/master/docs/CHANGELOG.md)
- [Commits](node-fetch/node-fetch@v2.6.0...v2.6.1)

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
godmodegalactus pushed a commit to blockworks-foundation/solana that referenced this pull request Jan 5, 2024
Use cluster info functions for tpu (solana-labs#345) (solana-labs#347)
Use git rev-parse for git sha
Remove blacklisted tx from message_hash_to_transaction (Backport solana-labs#374) (solana-labs#376)
Updates scripts for easy local setup. (solana-labs#383)
Backports sim_bundle improvements (solana-labs#407)
backports clone derivation 416 (solana-labs#417)
Backport solana-labs#419: add upsert to AccountOverrides (solana-labs#420)

backports solana-labs#430 v1.16: update jito-programs sha (solana-labs#431)

[JIT-1661] Faster Autosnapshot (solana-labs#406)

Fix Buildkite warnings (solana-labs#437)

Backport solana-labs#446 to v1.16 (solana-labs#447)

backport 428, runtime plugin (solana-labs#429)

v1.16: Backport solana-labs#449 (solana-labs#450)
brooksprumo pushed a commit to brooksprumo/solana that referenced this pull request Mar 25, 2024
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.

4 participants