Skip to content

Fix: Capture the whole error message with netmiko_regexp - #3950

Merged
ipspace merged 1 commit into
devfrom
netmiko-regexp
Oct 15, 2026
Merged

ipspace merged 1 commit into
devfrom
netmiko-regexp

Conversation

@ipspace

@ipspace ipspace commented Oct 15, 2026

Copy link
Copy Markdown
Owner

The netmiko error regexp is updated for EOS, IOS, and IOS XR to capture the whole error message.

The netmiko error regexp is updated for EOS, IOS, and IOS XR to
capture the whole error message.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🔵 Needs a closer look

The IOS regexp must be adjusted to capture the complete error line, including the leading % .

0 open findings

What changed in this PR

Updates Netmiko error regexps for EOS, IOS, and IOS XR, with documentation and an IOS testing topology.

Changes:

  • Expands device-specific error matching.
  • Adds IOS Netmiko test scenarios.
  • Documents manual configuration testing.
File Description
tests/​config/​README.md Documents manual Netmiko testing.
tests/​config/​ios-netmiko.yml Adds IOS error-condition topology.
netsim/​devices/​xr.yml Expands IOS XR error matching.
netsim/​devices/​ios.yml Extends IOS error matching; leading % remains omitted.
netsim/​devices/​eos.yml Expands EOS error matching.

🧠 Review effort: Lite


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@ipspace
ipspace merged commit b271e09 into dev Oct 15, 2026
9 checks passed
@ipspace
ipspace deleted the netmiko-regexp branch October 15, 2026 16:35
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