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

Fix: allow integer values in ext_community_list_raw field for Arista BGP… #306

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

cooperwinser
Copy link

@cooperwinser cooperwinser commented Dec 9, 2024

Description

Fix: allow integer values in ext_community_list_raw field for Arista BGP parsing

This fix resolves validation errors by modifying the parsing logic to accept both integer and string types for the 'ext_community_list_raw' field in Arista device BGP route data.

Related Issues

Fixes #305

Motivation and Context

The current implementation strictly requires string inputs for ext_community_list_raw, causing validation errors when integer values are received from Arista devices. We were experiencing this issue ourselves and these changes appear to have resolved the issue.

Tests

  • Verified parsing of ext_community_list_raw with integer values: 595497215590410, 595497215590424, etc.
  • Confirmed successful BGP route table parsing on Arista EOS devices.
  • Confirmed error messages in original issue no longer appear.

@cooperwinser cooperwinser reopened this Dec 9, 2024
@cooperwinser cooperwinser changed the title Fix: allow integer values in extCommunityListRaw field for Arista BGP… Fix: allow integer values in ext_community_list_raw field for Arista BGP… 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
1 participant