Skip to content

feat: add local caps and peer-group to watchEventPeer#3410

Closed
Ivan-Pokhabov wants to merge 1 commit into
osrg:masterfrom
Ivan-Pokhabov:feat_add_params_for_watch_event
Closed

feat: add local caps and peer-group to watchEventPeer#3410
Ivan-Pokhabov wants to merge 1 commit into
osrg:masterfrom
Ivan-Pokhabov:feat_add_params_for_watch_event

Conversation

@Ivan-Pokhabov
Copy link
Copy Markdown
Contributor

No description provided.

@Ivan-Pokhabov Ivan-Pokhabov force-pushed the feat_add_params_for_watch_event branch from 4aa61f6 to fe860d6 Compare May 11, 2026 16:47
Comment thread pkg/server/grpc_server.go Outdated
LocalPort: p.Transport.LocalPort,
RemotePort: p.Transport.RemotePort,
},
AfiSafis: afiSafis,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This sets the result of negotiatedRFList(). However, ListPath() sets the result of locally configured families.
So it's inconsistent.
I think that it would be better to let the field alone. You can get the info from LocalCaps and RemoteCaps.

@Ivan-Pokhabov Ivan-Pokhabov force-pushed the feat_add_params_for_watch_event branch from fe860d6 to 006afb4 Compare May 24, 2026 16:15
@Ivan-Pokhabov Ivan-Pokhabov changed the title feat: add local caps, peer-group and families to watchEventPeer feat: add local caps and peer-group to watchEventPeer May 24, 2026
@Ivan-Pokhabov Ivan-Pokhabov force-pushed the feat_add_params_for_watch_event branch from 006afb4 to 37e3964 Compare May 24, 2026 16:27
@fujita
Copy link
Copy Markdown
Member

fujita commented May 25, 2026

Pushed, thanks.

@fujita fujita closed this May 25, 2026
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.

2 participants