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

gnrc_ipv6_nib: use correct DHCPv6 client module name #18402

Merged

Conversation

benpicco
Copy link
Contributor

@benpicco benpicco commented Aug 5, 2022

Contribution description

MODULE_GNRC_DHCPV6_CLIENT_IA_PD does not exist, the correct module name is MODULE_DHCPV6_CLIENT_IA_PD.

Testing procedure

I noticed that when running examples/gnrc_border_router with UPLINK=wifi I would not get a global address on the upstream interface because RTR_ADV was enabled on it.

With the config properly set, RTR_ADV is only enabled on the downstream interface after a delegated prefix was received.

Issues/PRs references

follow-up to #18360

@benpicco benpicco requested a review from miri64 August 5, 2022 10:40
@github-actions github-actions bot added Area: Kconfig Area: Kconfig integration Area: network Area: Networking Area: sys Area: System labels Aug 5, 2022
@benpicco benpicco added Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Aug 5, 2022
`MODULE_GNRC_DHCPV6_CLIENT_IA_PD` does not exist, the correct module name
is `MODULE_DHCPV6_CLIENT_IA_PD`.
@benpicco benpicco force-pushed the CONFIG_GNRC_IPV6_NIB_ADV_ROUTER-fix branch from b4dd65f to f6cef27 Compare August 5, 2022 10:52
@miri64
Copy link
Member

miri64 commented Aug 5, 2022

Exhibit A for why I don't ACK my own changes or trust my own testing 😝

@benpicco benpicco added the Process: needs backport Integration Process: The PR is required to be backported to a release or feature branch label Aug 5, 2022
@benpicco
Copy link
Contributor Author

benpicco commented Aug 5, 2022

Continuous improvement is better than delayed perfection 😉

@benpicco benpicco added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR and removed CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Aug 5, 2022
@benpicco benpicco merged commit 3f1ac8a into RIOT-OS:master Aug 6, 2022
@benpicco benpicco deleted the CONFIG_GNRC_IPV6_NIB_ADV_ROUTER-fix branch August 6, 2022 20:39
@maribu maribu added this to the Release 2022.10 milestone Oct 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Kconfig Area: Kconfig integration Area: network Area: Networking Area: sys Area: System CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer Process: needs backport Integration Process: The PR is required to be backported to a release or feature branch Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants