Skip to content

tools/dhcpv6-pd_ia: create /run/kea on startup#17674

Merged
benpicco merged 1 commit intoRIOT-OS:masterfrom
benpicco:tools/dhcpv6-pd_ia-/run/kea
Feb 17, 2022
Merged

tools/dhcpv6-pd_ia: create /run/kea on startup#17674
benpicco merged 1 commit intoRIOT-OS:masterfrom
benpicco:tools/dhcpv6-pd_ia-/run/kea

Conversation

@benpicco
Copy link
Copy Markdown
Contributor

Contribution description

I was wondering why the border router would not work on native witth USE_DHCPV6=1, then I remembered that I manually have to create /run/kea each time.

So let's add this to the dhcpv6-pd_ia.py script so it will 'just work' in the future.

It looks like on Arch something simimar was needed, but on Ubuntu /run is a tmpfs, so we have to create the directory after each reboot.

Testing procedure

make -C examples/gnrc_border_router USE_DHCPV6=1 all term

should get a global address on the ZEP interface.

Issues/PRs references

@benpicco benpicco requested a review from miri64 as a code owner February 17, 2022 17:19
@github-actions github-actions bot added the Area: tools Area: Supplementary tools label Feb 17, 2022
@benpicco benpicco added CI: skip compile test If set, CI server will run only non-compile jobs, but no compile jobs or their dependent jobs Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation labels Feb 17, 2022
@benpicco benpicco force-pushed the tools/dhcpv6-pd_ia-/run/kea branch 2 times, most recently from f302565 to 3a3e67b Compare February 17, 2022 17:28
@benpicco benpicco requested a review from JKRhb February 17, 2022 17:29
On Ubuntu KEA expects this directory to be present, but it is not
created automatically.
/run is also a tmpfs, so we have to create the directory after each
reboot.
@benpicco benpicco force-pushed the tools/dhcpv6-pd_ia-/run/kea branch from 3a3e67b to 5476d74 Compare February 17, 2022 17:34
Copy link
Copy Markdown
Member

@JKRhb JKRhb left a comment

Choose a reason for hiding this comment

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

LGTM. Tested it under Ubuntu and it worked as expected :)

@benpicco benpicco added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Feb 17, 2022
@benpicco benpicco enabled auto-merge February 17, 2022 17:50
@benpicco benpicco merged commit fed25b6 into RIOT-OS:master Feb 17, 2022
@benpicco benpicco deleted the tools/dhcpv6-pd_ia-/run/kea branch February 17, 2022 23:02
@OlegHahm OlegHahm added this to the Release 2022.04 milestone Apr 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: tools Area: Supplementary tools CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR CI: skip compile test If set, CI server will run only non-compile jobs, but no compile jobs or their dependent jobs Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants