Skip to content

[Not4Review] Update frr side config to enable bmp#20857

Closed
FengPan-Frank wants to merge 2 commits intosonic-net:masterfrom
FengPan-Frank:frr_bmp
Closed

[Not4Review] Update frr side config to enable bmp#20857
FengPan-Frank wants to merge 2 commits intosonic-net:masterfrom
FengPan-Frank:frr_bmp

Conversation

@FengPan-Frank
Copy link
Contributor

Why I did it

Update frr side config to enable bmp

Work item tracking
  • Microsoft ADO (number only):27588893

How I did it

Update bgpd.conf.j2, as well as bgpd daemon parameter.

How to verify it

local tested.

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)


[program:bgpd]
command=/usr/lib/frr/bgpd -A 127.0.0.1 -M snmp
command=/usr/lib/frr/bgpd -A 127.0.0.1 -M snmp -M bmp
Copy link
Collaborator

Choose a reason for hiding this comment

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

-M bmp

If the feature is disabled, this change still has performance impact or maybe regression to FRR. Please consider FEATURE table for j2 rendering.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

moved to #20895 instead.

@FengPan-Frank FengPan-Frank changed the title Update frr side config to enable bmp [Not4Review] Update frr side config to enable bmp Dec 9, 2024
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