Skip to content

Implement new TC for SNMP ifErrors/ifDiscard (#14916)#653

Merged
r12f merged 2 commits intoAzure:202412from
illia-kotvitskyi:snmp_errors_discards
Aug 28, 2025
Merged

Implement new TC for SNMP ifErrors/ifDiscard (#14916)#653
r12f merged 2 commits intoAzure:202412from
illia-kotvitskyi:snmp_errors_discards

Conversation

@illia-kotvitskyi
Copy link
Copy Markdown

  • Implement new TC for SNMP ifErrors/ifDiscard

  • move parse_rif_counters to a common utilities

the motivation is to avoid cross-feature dependency

  • align test_snmp_interfaces_error_discard with recent updates

@illia-kotvitskyi illia-kotvitskyi force-pushed the snmp_errors_discards branch 2 times, most recently from 4800644 to bc7b3db Compare August 13, 2025 22:24
@illia-kotvitskyi
Copy link
Copy Markdown
Author

@r12f can I ask you for a review?

@r12f
Copy link
Copy Markdown
Contributor

r12f commented Aug 15, 2025

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@illia-kotvitskyi
Copy link
Copy Markdown
Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Commenter does not have sufficient privileges for PR 653 in repo Azure/sonic-mgmt.msft

@r12f
Copy link
Copy Markdown
Contributor

r12f commented Aug 19, 2025

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

* Implement new TC for SNMP ifErrors/ifDiscard

* move parse_rif_counters to a common utilities

the motivation is to avoid cross-feature dependency

* align test_snmp_interfaces_error_discard with recent updates

---------

Co-authored-by: Illia Kotvitskyi <ikotvitskyi@nvidia.com>
@illia-kotvitskyi
Copy link
Copy Markdown
Author

/azpw run Azure.sonic-mgmt

@illia-kotvitskyi
Copy link
Copy Markdown
Author

@r12f need your help to rerun the CI
Previously failed due to infra issues not related to my change

@r12f
Copy link
Copy Markdown
Contributor

r12f commented Aug 22, 2025

/azp run Azure.sonic-mgmt

@azure-pipelines
Copy link
Copy Markdown

No pipelines are associated with this pull request.

@r12f
Copy link
Copy Markdown
Contributor

r12f commented Aug 24, 2025

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@r12f r12f merged commit 8dbb968 into Azure:202412 Aug 28, 2025
6 of 14 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.

2 participants