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

Detect IPv6 automatically, Rewrite kexec url #304

Merged
merged 1 commit into from
Apr 30, 2024

Conversation

mkg20001
Copy link
Contributor

@mkg20001 mkg20001 commented Apr 21, 2024

This uses ip r g 1.1.1.1 to test if there's any route to an IPv4 internet host

If that fails and the url is a github.com one it will change github.com to gh-v6.com,
as specified in the documentation

This saves the user from an extra manual step and makes nixos-anywhere better

The fallback behaviour if the ip binary cannot be found is to just assume ipv4 connectivity. I can invert that if wanted.

@mkg20001 mkg20001 force-pushed the automatic-ipv6-detection branch 4 times, most recently from 672c8ce to 560e73c Compare April 21, 2024 17:33
src/nixos-anywhere.sh Outdated Show resolved Hide resolved
src/nixos-anywhere.sh Outdated Show resolved Hide resolved
@Mic92
Copy link
Member

Mic92 commented Apr 29, 2024

cc @zimbatm because he is paying for the proxy.

@Mic92
Copy link
Member

Mic92 commented Apr 29, 2024

This pull request also needs to be rebased once #297 is merged.

@mkg20001 mkg20001 force-pushed the automatic-ipv6-detection branch 2 times, most recently from c98a11c to dbb3366 Compare April 29, 2024 21:32
This uses ip r g 1.1.1.1 to test if there's any route to an IPv4
internet host

If that fails and the url is a github.com one it will change github.com
to gh-v6.com,
as specified in the documentation

This saves the user from an extra manual step and makes nixos-anywhere
better

Co-authored-by: Jörg Thalheim <Mic92@users.noreply.github.com>
@Mic92
Copy link
Member

Mic92 commented Apr 30, 2024

@mergify queue

Copy link
Contributor

mergify bot commented Apr 30, 2024

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at 313ac87

@mergify mergify bot merged commit 313ac87 into nix-community:main Apr 30, 2024
24 checks passed
@mkg20001 mkg20001 deleted the automatic-ipv6-detection branch April 30, 2024 13:41
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