Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Netconf support IPv6 #2114

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.

Sign up for GitHub

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

Jump to bottom
Merged
KristianJerpetjon merged 11 commits into from
Apr 3, 2019
Merged

Netconf support IPv6 #2114

KristianJerpetjon merged 11 commits into from
Apr 3, 2019

Conversation

@AndreasAakesson
Copy link
Contributor

@AndreasAakesson AndreasAakesson commented Apr 2, 2019

So config.json can now hold IPv6 configuration as well.
Instead of adding 6 members, they now support a list of addresses, both v4 and v6. / can also be used for netmask/prefix.

Example:

    {
      "iface": 0,
      "config": [ "static" ],
      "address": [
        "fe80::1337/64",
        "10.0.0.42/24",
        "fe80::42",
      ],
      "gateway": [
        "10.0.0.1",
        "fe80::1"
      ],
      "dns": [
        "4.4.4.4",
        "fe80::1"
      ]
    }

It's also possible to pass more than one config method (then use array): dhcp, dhcp4, dhcp6 (unsupported), static, static4, static6, and the good ol' dhcp-with-fallback. static in this case means both static4 and static6.

KristianJerpetjon
KristianJerpetjon approved these changes Apr 3, 2019
View reviewed changes
Copy link
Contributor

@KristianJerpetjon KristianJerpetjon left a comment

Choose a reason for hiding this comment

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

LGTM

@KristianJerpetjon KristianJerpetjon merged commit fd34f73 into includeos:dev Apr 3, 2019
@AndreasAakesson AndreasAakesson deleted the netconf branch April 23, 2019 08:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@KristianJerpetjon KristianJerpetjon KristianJerpetjon approved these changes

@alfreb alfreb Awaiting requested review from alfreb

@fwsGonzo fwsGonzo Awaiting requested review from fwsGonzo

@ingve ingve Awaiting requested review from ingve

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@AndreasAakesson @KristianJerpetjon