Skip to content

package/netd: ensure /etc/net.d exists at boot#1444

Merged
troglobit merged 2 commits intomainfrom
another-netd-fixup
Mar 13, 2026
Merged

package/netd: ensure /etc/net.d exists at boot#1444
troglobit merged 2 commits intomainfrom
another-netd-fixup

Conversation

@troglobit
Copy link
Contributor

Description

Before netd starts, confd might want to create .conf file for it to run, so we sort of depend on /etc/net.d existing at boot.

Checklist

Tick relevant boxes, this PR is-a or has-a:

  • Bugfix
    • Regression tests
    • ChangeLog updates (for next release)
  • Feature
    • YANG model change => revision updated?
    • Regression tests added?
    • ChangeLog updates (for next release)
    • Documentation added?
  • Test changes
    • Checked in changed Readme.adoc (make test-spec)
    • Added new test to group Readme.adoc and yaml file
  • Code style update (formatting, renaming)
  • Refactoring (please detail in commit messages)
  • Build related changes
  • Documentation content changes
    • ChangeLog updated (for major changes)
  • Other (please describe):

Before netd starts confd might want to create .conf file for it to run,
so we sort of depend on /etc/net.d existing at boot.

Signed-off-by: Joachim Wiberg <troglobit@gmail.com>
@troglobit troglobit requested a review from mattiaswal March 12, 2026 21:16
Replace ERROR() with ERRNO() in all error paths following POSIX API
calls (fopen, rename, realloc, fmkpath, etc.), removing redundant
manual strerror(errno) formatting where present.

Signed-off-by: Joachim Wiberg <troglobit@gmail.com>
@troglobit
Copy link
Contributor Author

troglobit commented Mar 12, 2026

One test failed, on one test system, it's up to you @mattiaswal if we can merge this or not. Happy to report though, that cherry-picking the netd change to v26.02.x makes the BPi-R3 dance happy and carefree again! 🎉

@troglobit troglobit merged commit 5487344 into main Mar 13, 2026
8 of 9 checks passed
@troglobit troglobit deleted the another-netd-fixup branch March 13, 2026 06:15
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