Skip to content

Move yang models to confd #1013

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
merged 2 commits into from
Apr 11, 2025
Merged

Move yang models to confd #1013

merged 2 commits into from
Apr 11, 2025

Conversation

mattiaswal
Copy link
Contributor

@mattiaswal mattiaswal commented Apr 7, 2025 •
edited
Loading

They are still installed in romfs by netopeer2, sysrepo and other packages, but the sysrepoctl is handled from confd.

This fix #994

Description

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):

@mattiaswal mattiaswal force-pushed the move-yang-models-to-confd branch 2 times, most recently from b3d3d93 to d9c3bb7 Compare April 8, 2025 05:57
@mattiaswal mattiaswal requested a review from troglobit April 8, 2025 06:03
@mattiaswal mattiaswal force-pushed the move-yang-models-to-confd branch 3 times, most recently from 1ff197c to 3ab4229 Compare April 8, 2025 09:17
@mattiaswal mattiaswal added the ci:main Build default defconfig, not minimal label Apr 8, 2025
@mattiaswal
srload: Add support for multiple search directories
d0277de
@mattiaswal
confd: Install *all* YANG modules from confd
092964a 
Sysrepo, netopeer etc are still responsible to install the modules in
target directory, this will make that we will discover quickly if something
is updated.

Also this enables rm -rf output/target/etc/sysrepo/data/* && make confd-rebuild
This will reinstall all modules again.
@mattiaswal mattiaswal force-pushed the move-yang-models-to-confd branch from 3ab4229 to 092964a Compare April 11, 2025 06:22
troglobit
troglobit approved these changes Apr 11, 2025
View reviewed changes
Copy link
Contributor

@troglobit troglobit left a comment

Choose a reason for hiding this comment

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

🚀

@mattiaswal mattiaswal merged commit cc03a59 into main Apr 11, 2025
6 checks passed
@mattiaswal mattiaswal deleted the move-yang-models-to-confd branch April 11, 2025 11:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@troglobit troglobit troglobit approved these changes

Assignees
No one assigned
Labels
ci:main Build default defconfig, not minimal
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

confd: Install all YANG modules from netopeer2, sysrepo from confd
2 participants
@mattiaswal @troglobit