Skip to content

Conversation

@choppsv1
Copy link
Contributor

This adds a new frr-host yang module with a single leaf value (so far) allow-reserved-ranges to move this config from CLI to mgmtd. This avoids a race in zebra between IP config and this setting.

@choppsv1 choppsv1 force-pushed the chopps/frr-host-yang-module branch 5 times, most recently from 3631c4b to 4eb0709 Compare June 13, 2025 06:14
- Everyone loads frr-host module to process host config

Signed-off-by: Christian Hopps <[email protected]>
@choppsv1 choppsv1 force-pushed the chopps/frr-host-yang-module branch 2 times, most recently from cf9662f to 06c3386 Compare June 13, 2025 10:07
choppsv1 added 3 commits June 13, 2025 06:20
First use of flag is to decide whether to initialize the `host` CLI handlers
in the daemon or not.

Mgmtd daemons process `frr-host` config from mgmtd which handles the CLI, others
still take it directly from VTY and so they need the DEFPY handlers.

Signed-off-by: Christian Hopps <[email protected]>
@choppsv1 choppsv1 force-pushed the chopps/frr-host-yang-module branch from 06c3386 to b2f5241 Compare June 13, 2025 10:20
Copy link
Member

@ton31337 ton31337 left a comment

Choose a reason for hiding this comment

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

LGTM

Need to add frr-host to the expected results when yang-library oper-state is
checked.

Signed-off-by: Christian Hopps <[email protected]>
@frrbot frrbot bot added the tests Topotests, make check, etc label Jun 13, 2025
@donaldsharp
Copy link
Member

code works for me now too.

@donaldsharp donaldsharp merged commit 31469b2 into FRRouting:master Jun 13, 2025
15 checks passed
@choppsv1 choppsv1 deleted the chopps/frr-host-yang-module branch June 13, 2025 17:23
@choppsv1 choppsv1 added this to the 10.4 milestone Jun 13, 2025
@choppsv1
Copy link
Contributor Author

@Mergifyio backport dev/10.4

@mergify
Copy link

mergify bot commented Jun 13, 2025

backport dev/10.4

✅ Backports have been created

Details

ton31337 added a commit that referenced this pull request Jun 15, 2025
Add frr-host yang module - fix bug with reserved IP range config (backport #19019)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants