Skip to content
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.

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

mgmtd: add notification selection to front-end API #16184

Merged
merged 8 commits into from
Jun 13, 2024

Conversation

choppsv1
Copy link
Contributor

@choppsv1 choppsv1 commented Jun 6, 2024

  • Add new notification select message for front-end clients
  • Add new native session-req and session-reply replacements for front-end protobuf messages
  • Extend fe_client to support new native notify select and session-req messages
  • Add topotest
  • Add string array encoding/decode to native messages

@choppsv1 choppsv1 added the freeze Show the warning before merging as failed label Jun 6, 2024
choppsv1 added 6 commits June 6, 2024 22:38
- `darr_free_free` to `darr_free` each element prior to `darr_free`
  the array.
- `darr_free_func` to call `func` on each element prior to `darr_free`
  the array.

Signed-off-by: Christian Hopps <[email protected]>
@choppsv1 choppsv1 force-pushed the chopps/fe-notify-select branch from b30acaa to 56ce198 Compare June 7, 2024 02:40
@github-actions github-actions bot added rebase PR needs rebase size/XL and removed size/L labels Jun 11, 2024
choppsv1 added 2 commits June 11, 2024 10:37
This addition allows for a limited native-message-only front-end
interaction.

Signed-off-by: Christian Hopps <[email protected]>
Use this to test new native message format for creating sessions.

Signed-off-by: Christian Hopps <[email protected]>
@choppsv1 choppsv1 force-pushed the chopps/fe-notify-select branch from 8e13e96 to 27e3694 Compare June 11, 2024 14:37
@choppsv1 choppsv1 removed the freeze Show the warning before merging as failed label Jun 11, 2024
@Jafaral Jafaral changed the title add notification selection to front-end API mgmtd: add notification selection to front-end API Jun 13, 2024
@Jafaral Jafaral merged commit 2e02bd2 into FRRouting:master Jun 13, 2024
11 of 12 checks passed
@choppsv1 choppsv1 deleted the chopps/fe-notify-select branch June 13, 2024 05:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants