Skip to content

Add support for new end-of-RIB elem#60

Open
alistairking wants to merge 3 commits into
CAIDA:masterfrom
alistairking:a6r--eor
Open

Add support for new end-of-RIB elem#60
alistairking wants to merge 3 commits into
CAIDA:masterfrom
alistairking:a6r--eor

Conversation

@alistairking

Copy link
Copy Markdown
Member

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR updates pybgpstream to recognize and expose the new libbgpstream BGPSTREAM_ELEM_TYPE_END_OF_RIB element type (paired with CAIDA/libbgpstream#258), and documents the corresponding elemtype filter value.

Changes:

  • Add BGPSTREAM_ELEM_TYPE_END_OF_RIB handling in BGPElem_get_fields, exposing the element prefix.
  • Update API docs to include endofrib as a valid elemtype filter value.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
src/_pybgpstream_bgpelem.c Adds type-specific field support for the new END_OF_RIB element type.
docs/api__pybgpstream.rst Documents endofrib as an elemtype filter option.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/_pybgpstream_bgpelem.c Outdated

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

Comment thread src/_pybgpstream_bgpelem.c

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

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