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

Configure default DNS resolver in 6lbr example #21117

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

OlegHahm
Copy link
Member

@OlegHahm OlegHahm commented Jan 2, 2025

Contribution description

For convenience, it would be nice to have DNS resolution enabled by default in the examples. Hence, this patch configures a default DNS resolver on the 6lbr example and propagates its via RDNSS option in RAs to the nodes.

Once again the tradeoff is some increased ROM and RAM usage.

Testing procedure

  1. Setup a border router (for instance on the nrf52840dongle), for instance, via BOARD=nrf52840dongle make -C examples/gnrc_border_router clean all flash ULINK=cdc-ecm.
  2. Setup a second 6lowpan capable device with, for instance, gnrc_networking while enabling USEMODULE += sock_dns and USEMODULE += gnrc_ipv6_nib_dns.
  3. Check if pinging a hostname works, e.g., ping riot-os.org.

Issues/PRs references

Can be tested with #21116.

For convenience, it would be nice to have DNS resolution enabled by
default in the examples. Hence, this patch configures a default DNS
resolver on the 6lbr example and propagates its via RDNSS option in RAs
to the nodes.
@github-actions github-actions bot added the Area: examples Area: Example Applications label Jan 2, 2025
@benpicco benpicco requested a review from miri64 January 2, 2025 13:57
@mguetschow mguetschow added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Jan 2, 2025
@riot-ci
Copy link

riot-ci commented Jan 2, 2025

Murdock results

✔️ PASSED

e668ac4 examples/gnrc_border_router: set DNS resolver

Success Failures Total Runtime
19 0 19 01m:42s

Artifacts

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: examples Area: Example Applications CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants