Skip to content

Migrate static configs - #1674

Merged
mattiaswal merged 3 commits into
mainfrom
migrate-static-configs
Sep 30, 2026
Merged

mattiaswal merged 3 commits into
mainfrom
migrate-static-configs

Conversation

@rical

@rical rical commented Sep 30, 2026

Copy link
Copy Markdown
Contributor

Description

Add make migrate-configs to bring static configurations, e.g., the
per-product factory-config.cfg in Infix and in spins, up to date with
the current confd version.

Also add a new unit test that ensures all factory configurations are up to date
and bump all factory configs to make it pass.

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

Run all static *-config.cfg files in BR2_EXTERNAL through
confd migrate. Introduce a STATIC flag for static configs.
There's things static configs shall not include, such as key
material, which might end up shared across an install base.

Signed-off-by: Richard Alpe <richard@bit42.se>
Signed-off-by: Richard Alpe <richard@bit42.se>
Migrated using make migrate-configs.
The netconf.enabled=false leaf it inserted was removed, the configs
had already been converted by hand and NETCONF should remain enabled.

Signed-off-by: Richard Alpe <richard@bit42.se>
@mattiaswal
mattiaswal merged commit ee54f1c into main Sep 30, 2026
9 checks passed
@mattiaswal
mattiaswal deleted the migrate-static-configs branch September 30, 2026 11:34
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.

3 participants