frr icon indicating copy to clipboard operation
frr copied to clipboard

lib: add debugs for interface lib module events

Open mjstapp opened this issue 1 year ago • 8 comments

Add some debugs for interface lib events, similar to what is present with "debug vrf".

mjstapp avatar Mar 11 '24 15:03 mjstapp

quite right - updated with a yang model change.

lib/if.c is fully converted to NB, I don't think we should add new functionality using regular non-NB vty anymore.

mjstapp avatar Mar 12 '24 20:03 mjstapp

I wanted to provide some example of how I think debug should be implemented in NB/mgmtd, but unfortunately it's not possible to implement it before https://github.com/FRRouting/frr/pull/15594 is merged (RPCs are necessary for temporary debug settings from the "enable" node).

For now I opened a PR https://github.com/FRRouting/frr/pull/15646 that simplifies the debug module in frrlib and makes it easier to use. As NB can't be implemented properly right now, I'm fine with this PR without NB if it starts using the libfrr debug.

idryzhov avatar Apr 01 '24 20:04 idryzhov

This pull request has conflicts, please resolve those before we can evaluate the pull request.

github-actions[bot] avatar Jul 18 '24 16:07 github-actions[bot]

This PR is stale because it has been open 180 days with no activity. Comment or remove the autoclose label in order to avoid having this PR closed.

github-actions[bot] avatar Jan 15 '25 01:01 github-actions[bot]

Updated to newer master; still working on this, still think it's worth ... something

mjstapp avatar Jan 15 '25 15:01 mjstapp

This pull request has conflicts, please resolve those before we can evaluate the pull request.

github-actions[bot] avatar Apr 24 '25 12:04 github-actions[bot]

rebased; still trying to get this to work...

mjstapp avatar Apr 24 '25 19:04 mjstapp

This pull request has conflicts, please resolve those before we can evaluate the pull request.

github-actions[bot] avatar May 20 '25 15:05 github-actions[bot]