Skip to content

[Nexthop] Add BGP thrift definitions#1038

Open
manoharan-nexthop wants to merge 1 commit intofacebook:mainfrom
nexthop-ai:manoharan.bgp-show-thrift
Open

[Nexthop] Add BGP thrift definitions#1038
manoharan-nexthop wants to merge 1 commit intofacebook:mainfrom
nexthop-ai:manoharan.bgp-show-thrift

Conversation

@manoharan-nexthop
Copy link
Copy Markdown
Contributor

Pre-submission checklist

  • I've ran the linters locally and fixed lint errors related to the files I modified in this PR. You can install the linters by running pip install -r requirements-dev.txt && pre-commit install
  • pre-commit run

Summary

Adds the thrift interface definitions for BGP (Border Gateway Protocol) functionality. These thrift files define the data structures and service interfaces needed for BGP operations.

Thrift Files Added:

  • fboss/bgp/if/bgp_attr.thrift - BGP attribute definitions
  • fboss/bgp/if/bgp_config.thrift - BGP configuration structures
  • fboss/bgp/if/bgp_thrift.thrift - Main BGP service interface
  • fboss/bgp_policy/if/bgp_policy.thrift - BGP policy types
  • fboss/bgp_policy/if/rib_policy.thrift - RIB policy definitions
  • fboss/routing/if/vip_service_config.thrift - Virtual IP service config
  • fboss/routing/policy/if/policy_thrift.thrift - Routing policy definitions
  • fboss/cli/fboss2/commands/show/bgp/summary/bgp_summary.thrift - BGP summary types

Build System Updates:

  • CMakeLists.txt - Added thrift compilation targets for all BGP thrift files
  • BUCK files - Added thrift library definitions for Meta-internal builds

Test Plan

This commit adds the thrift interface definitions for BGP (Border Gateway Protocol)
functionality. These thrift files define the data structures and service interfaces
needed for BGP operations.

Thrift Files Added:
- fboss/bgp/if/bgp_attr.thrift - BGP attribute definitions
- fboss/bgp/if/bgp_config.thrift - BGP configuration structures
- fboss/bgp/if/bgp_thrift.thrift - Main BGP service interface
- fboss/bgp_policy/if/bgp_policy.thrift - BGP policy types
- fboss/bgp_policy/if/rib_policy.thrift - RIB policy definitions
- fboss/routing/if/vip_service_config.thrift - Virtual IP service config
- fboss/routing/policy/if/policy_thrift.thrift - Routing policy definitions
- fboss/cli/fboss2/commands/show/bgp/summary/bgp_summary.thrift - BGP summary types

Build System Updates:
- CMakeLists.txt - Added thrift compilation targets for all BGP thrift files
- BUCK files - Added thrift library definitions for Meta-internal builds

The thrift files define:
- BGP state machine states and peer information
- Route attributes (AS path, communities, next hop, etc.)
- BGP configuration structures
- Policy and filtering definitions
- Service interface for BGP operations

These definitions will be used by the BGP CLI commands and BGP service
implementation.
@manoharan-nexthop manoharan-nexthop requested review from a team as code owners March 26, 2026 18:25
@meta-cla meta-cla bot added the CLA Signed label Mar 26, 2026
@manoharan-nexthop manoharan-nexthop changed the title Add BGP thrift definitions [Nexthop] Add BGP thrift definitions Mar 26, 2026
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Can you please add a .md file or inline comments of how the configuration commands will render?

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.

2 participants