Skip to content

Conversation

@heri16
Copy link

@heri16 heri16 commented Nov 29, 2025

On many VPS providers, the IPv4 uplink network interface is eth0, while the IPv6 uplink may be on a different interface (e.g., eth1 or another device).

This PR:

  • Improves automatic detection of the uplink network interface for both IPv4 and IPv6.
  • Adds support for configurations where the IPv4 and IPv6 uplinks are on different interfaces, while remaining fully compatible with setups where they share the same interface.
  • Changes the expect environment variable name from NETWORK_DEVICE to IPV4_UPLINK_DEV and IPV6_UPLINK_DEV

This change is Reviewable

@vercel
Copy link

vercel bot commented Nov 29, 2025

@heri16 is attempting to deploy a commit to the nyx-network Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions
Copy link

Thank you for making this first PR

@serinko
Copy link
Contributor

serinko commented Dec 2, 2025

Hey, it looks good. Thank you for your PR.
You haven't ticked the env var but I see it added in the PR

Please redirect it from develop to feat/operators/ntm-upgrades.

This var is used here:
https://github.com/nymtech/nym/blob/develop/scripts/nym-node-setup/nym-node-cli.py#L578
https://github.com/nymtech/nym/blob/develop/scripts/nym-node-setup/nym-node-cli.py#L643

If you want you can toggle that too, otherwise I will fix that one

@heri16
Copy link
Author

heri16 commented Dec 2, 2025

For the record:
"ip -6 -o route show default" was not reliable.
On my VPS machine (Debian 13), it requires "print $3" while the original script uses "print $5".
See:

nonroot@localhost:~$ ip -6 -o route show default
default dev eth1.1 proto static metric 1024 pref medium

@heri16
Copy link
Author

heri16 commented Dec 2, 2025

https://github.com/nymtech/nym/blob/develop/scripts/nym-node-setup/nym-node-cli.py#L643

Should we have:

  • Option A: --uplink-dev-v4 and --uplink-dev-v6
  • Option B: --uplink-dev and --uplink-dev-v6 where v6 defaults to autodetection if unspecified?

@serinko

Also, I've found another issue with the quic_bridge_deployment.sh on machines such as mine.
Would you prefer a different PR? This line containing UPLINK_DEV is used by quic_bridge_deployment.sh: https://github.com/nymtech/nym/blob/develop/scripts/nym-node-setup/nym-node-cli.py#L577

@heri16 heri16 changed the base branch from develop to feat/operators/ntm-upgrades December 2, 2025 20:37
@heri16 heri16 requested a review from mfahampshire as a code owner December 2, 2025 21:08
Copy link
Author

@heri16 heri16 left a comment

Choose a reason for hiding this comment

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

Please review latest changes to this PR.

@heri16
Copy link
Author

heri16 commented Dec 2, 2025

:lgtm:

@serinko
Copy link
Contributor

serinko commented Dec 3, 2025

I would prefer this one as the program should ask minimum from the operator

* Option B: `--uplink-dev` and `--uplink-dev-v6` where v6 defaults to autodetection if unspecified?

If that issue on QUIC connects to this interface, feel free to add it to this PR.

@serinko serinko self-requested a review December 3, 2025 09:14
@serinko serinko added this to the Independent Release milestone Dec 3, 2025
@vercel
Copy link

vercel bot commented Dec 3, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Preview Comments Updated (UTC)
docs-nextra Ignored Ignored Preview Dec 3, 2025 9:14am

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