Skip to content

Conversation

@itsMrRobot
Copy link

Encountered PfxAdv option when working on Lab cEOS devices running 4.34.2F-43232954.4342F, added line 21 to account for that field as previous version encountered error without it

CLI output from device:
image

Error received without line:
image

Results after adding line to local template file:
image

Encountered PfxAdv option when working on Lab cEOS devices running 4.34.2F-43232954.4342F, added line 21 to account for that field as previous version encountered error without it
@mjbear
Copy link
Collaborator

mjbear commented Sep 6, 2025

Hi @itsMrRobot!

  1. Test data needs to be added so CI tests can run against this template to validate it works now and in the future. There are helper scripts to help with generating the structured test data from the raw CLI output.

Here are some tips on the development environment

Thank you!

@mjbear mjbear added the status: action required This issue requires additional information to be actionable label Sep 6, 2025
@mjbear mjbear changed the title Added line to include PfxAdv option Fix arista_eos sh ip bgp for PfxAdv column Sep 6, 2025
@mjbear
Copy link
Collaborator

mjbear commented Sep 21, 2025

@itsMrRobot
Please visit the PR on GitHub to review my guidance from two weeks ago (9/6).

If there are questions, please respond on the PR here.
Thank you!

@mjbear mjbear marked this pull request as draft September 21, 2025 18:06
@mjbear
Copy link
Collaborator

mjbear commented Oct 25, 2025

@itsMrRobot
Please review the September 6 requested changes.
If you have any questions, please respond back. We're here to assist.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: action required This issue requires additional information to be actionable

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants