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

IOSXR: ShowL2VPNBridgeDomainDetail parser enhancement request #761

Closed
agandl046 opened this issue Jun 12, 2023 · 16 comments
Closed

IOSXR: ShowL2VPNBridgeDomainDetail parser enhancement request #761

agandl046 opened this issue Jun 12, 2023 · 16 comments
Assignees

Comments

@agandl046
Copy link

Similar to the draft PR #744, I would like to get Flow Label flags configured (Tx=1,Rx=1), negotiated (Tx=1,Rx=1) parsed and added to the ShowL2VPNBridgeDomainDetail parser for Access PWs and VFIs. This is currently blocked by this bug in the parser: #760

Parser: https://github.com/CiscoTestAutomation/genieparser/blob/master/src/genie/libs/parser/iosxr/show_l2vpn.py#L1258

Example of output can be found in https://github.com/CiscoTestAutomation/genieparser/blob/master/src/genie/libs/parser/iosxr/tests/ShowL2vpnBridgeDomainDetail/cli/equal/golden7_output.txt

@iamsatyanarayan
Copy link

Hi @agandl046
I will be working on your ticket. I'll get back to you once I have any update on it.

@iamsatyanarayan
Copy link

Hi @agandl046
I am working on this parser issue. I'll let you know once I have any update on it.

@iamsatyanarayan
Copy link

Hi @agandl046 ,
Could you please provide any update on your issue ?

@agandl046
Copy link
Author

Hi @iamsatyanarayan,
I don't seem to have access to the PR you shared

@iamsatyanarayan
Copy link

Hi @agandlo46
PR has been merged. It will reflect in next release.

@agandl046
Copy link
Author

@iamsatyanarayan Where can I see this update to confirm it works with my outputs? Also, this draft PR #744 is for a similar change. Is it still being worked on?

@iamsatyanarayan
Copy link

iamsatyanarayan commented Sep 7, 2023

Hi @agandl046
Please wait until next release,

@agandl046
Copy link
Author

@iamsatyanarayan Is that a Cisco internal github? That page doesn't load

@iamsatyanarayan
Copy link

Hi @agandl046
Please wait until next release,

@iamsatyanarayan
Copy link

Hi @agandl046
Could you please share with me the piestack ticket link for this PR :- #744

@agandl046
Copy link
Author

Hi @iamsatyanarayan. I don't know if there was ever a ticket opened for that PR. I had originally reached out via email about that issue back in March and sent follow up emails in April/May/June with no response. The support email I had used was pyats-support-ext@cisco.com and Dave Wapstra was who originally responded.

@iamsatyanarayan
Copy link

HI,
Ok please give me some time . I will let you know after get confirmation from team.

@SohanTirpude
Copy link
Contributor

Hello @agandl046,

The newer version(23.9) of pyATS has been released, hence kindly update your pyATS and try again. Kindly let me know the status.

@SohanTirpude
Copy link
Contributor

Hello @agandl046,

Did you get a chance to test this fix? Kindly let me know.

@agandl046
Copy link
Author

Hi @SohanTirpude. The fix looks good. Thank you!

@SohanTirpude
Copy link
Contributor

Hello @agandl046,

Thank you very much for the confirmation, hence I am closing this ticket. If you are facing any other issues, kindly open a new ticket. Thank you again.

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

No branches or pull requests

4 participants