Skip to content

Fixes related to whitespace for parsing "show interfaces accounting"#966

Merged
omehrabi merged 1 commit intoCiscoTestAutomation:mainfrom
jlmcgraw:20251029_iosxe_show_interfaces_accounting
Nov 3, 2025
Merged

Fixes related to whitespace for parsing "show interfaces accounting"#966
omehrabi merged 1 commit intoCiscoTestAutomation:mainfrom
jlmcgraw:20251029_iosxe_show_interfaces_accounting

Conversation

@jlmcgraw
Copy link
Copy Markdown
Contributor

@jlmcgraw jlmcgraw commented Oct 30, 2025

Description

  • iosxe
    • Modified class ShowInterfacesAccounting
      • Modified regex pattern to support interface descriptions with multiple spaces
      • Only right-strip whitespace from lines and modify regex for lines containing counters to add explicit leading
        whitespace

Motivation and Context

learn("interface") was crashing on output from an iosxe device where interface descriptions had multiple spaces

Impact (If any)

The output that was failing now parses successfully

Checklist:

  • I have updated the changelog.
  • I have updated the documentation (If applicable).
  • I have added tests to cover my changes (If applicable).
  • All new and existing tests passed.
  • All new code passed compilation.

@omehrabi omehrabi merged commit bad8497 into CiscoTestAutomation:main Nov 3, 2025
5 checks passed
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.

3 participants