Ignore error during config reload in BGP/QOS/FPC test cases#19869
Merged
bingwang-ms merged 2 commits intosonic-net:masterfrom Jul 29, 2025
Merged
Ignore error during config reload in BGP/QOS/FPC test cases#19869bingwang-ms merged 2 commits intosonic-net:masterfrom
bingwang-ms merged 2 commits intosonic-net:masterfrom
Conversation
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Collaborator
|
Instead of adding the log pattern for each test cases, can we ignore it globally in https://github.com/sonic-net/sonic-mgmt/blob/master/tests/common/plugins/loganalyzer/loganalyzer_common_ignore.txt ? |
Contributor
Author
I warry about add to global list may make us ignore some real issue, can we use this ignore_loganalyzer for now? if there are more similer issue in future we can add to global list. |
bingwang-ms
approved these changes
Jul 29, 2025
Collaborator
|
@yejianquan Can you help with cherry-pick? Thanks |
Collaborator
|
@liuh-80 PR conflicts with 202505 branch |
Collaborator
|
Hi @liuh-80 , could you fix the conflict and create pr to 202505 branch? Thanks |
nissampa
pushed a commit
to nissampa/sonic-mgmt_dpu_test
that referenced
this pull request
Aug 7, 2025
…t#19869) * Ignore error during config reload * Fix format issue --------- Co-authored-by: sonicbld <sonicbld@microsoft.com>
liuh-80
added a commit
to liuh-80/sonic-mgmt
that referenced
this pull request
Aug 12, 2025
…t#19869) * Ignore error during config reload * Fix format issue --------- Co-authored-by: sonicbld <sonicbld@microsoft.com>
Contributor
Author
11 tasks
11 tasks
yejianquan
pushed a commit
that referenced
this pull request
Aug 14, 2025
…20198) Ignore error during config reload in BGP/QOS/FPC test cases Cherry-pick for #19869 Why I did it BGP/QOS/FPC test case failed because following error: E 2025 Jul 28 04:41:57.469604 str2-msn2700-spy-1 ERR iptables: tac_connect_single: connection to 10.64.246.145:49 failed: Network is unreachable These test case reload_config but not set ignore_loganalyzer parameter. Because reload config will restart networking service, which will cause TACACS server unreachable during networking service shutdown. Work item tracking Microsoft ADO (number only): How I did it Set reload_config ignore_loganalyzer parameter in BGP/QOS/FPC test cases. How to verify it Pass all test case. Tested branch (Please provide the tested image version) Description for the changelog Ignore error during config reload in BGP/QOS/FPC test cases co-authorized by: jianquanye@microsoft.com
yejianquan
added
Included in 202505 branch
and removed
Cherry Pick Conflict_202505
labels
ashutosh-agrawal
pushed a commit
to ashutosh-agrawal/sonic-mgmt
that referenced
this pull request
Aug 14, 2025
…t#19869) * Ignore error during config reload * Fix format issue --------- Co-authored-by: sonicbld <sonicbld@microsoft.com>
vidyac86
pushed a commit
to vidyac86/sonic-mgmt
that referenced
this pull request
Oct 23, 2025
…t#19869) * Ignore error during config reload * Fix format issue --------- Co-authored-by: sonicbld <sonicbld@microsoft.com>
opcoder0
pushed a commit
to opcoder0/sonic-mgmt
that referenced
this pull request
Dec 8, 2025
…t#19869) * Ignore error during config reload * Fix format issue --------- Co-authored-by: sonicbld <sonicbld@microsoft.com> Signed-off-by: opcoder0 <110003254+opcoder0@users.noreply.github.com>
gshemesh2
pushed a commit
to gshemesh2/sonic-mgmt
that referenced
this pull request
Dec 16, 2025
…t#19869) * Ignore error during config reload * Fix format issue --------- Co-authored-by: sonicbld <sonicbld@microsoft.com> Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
AharonMalkin
pushed a commit
to AharonMalkin/sonic-mgmt
that referenced
this pull request
Dec 16, 2025
…t#19869) * Ignore error during config reload * Fix format issue --------- Co-authored-by: sonicbld <sonicbld@microsoft.com> Signed-off-by: Aharon Malkin <amalkin@nvidia.com>
gshemesh2
pushed a commit
to gshemesh2/sonic-mgmt
that referenced
this pull request
Dec 21, 2025
…t#19869) * Ignore error during config reload * Fix format issue --------- Co-authored-by: sonicbld <sonicbld@microsoft.com> Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
venu-nexthop
pushed a commit
to venu-nexthop/sonic-mgmt
that referenced
this pull request
Jan 13, 2026
…t#19869) * Ignore error during config reload * Fix format issue --------- Co-authored-by: sonicbld <sonicbld@microsoft.com>
gshemesh2
pushed a commit
to gshemesh2/sonic-mgmt
that referenced
this pull request
Jan 26, 2026
…t#19869) * Ignore error during config reload * Fix format issue --------- Co-authored-by: sonicbld <sonicbld@microsoft.com> Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
ytzur1
pushed a commit
to ytzur1/sonic-mgmt
that referenced
this pull request
Feb 2, 2026
…t#19869) * Ignore error during config reload * Fix format issue --------- Co-authored-by: sonicbld <sonicbld@microsoft.com> Signed-off-by: Yael Tzur <ytzur@nvidia.com>
venu-nexthop
pushed a commit
to venu-nexthop/sonic-mgmt
that referenced
this pull request
Mar 27, 2026
…t#19869) * Ignore error during config reload * Fix format issue --------- Co-authored-by: sonicbld <sonicbld@microsoft.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Ignore error during config reload in BGP/QOS/FPC test cases
Why I did it
BGP/QOS/FPC test case failed because following error:
E 2025 Jul 28 04:41:57.469604 str2-msn2700-spy-1 ERR iptables: tac_connect_single: connection to 10.64.246.145:49 failed: Network is unreachable
These test case reload_config but not set ignore_loganalyzer parameter.
Because reload config will restart networking service, which will cause TACACS server unreachable during networking service shutdown.
Work item tracking
How I did it
Set reload_config ignore_loganalyzer parameter in BGP/QOS/FPC test cases.
How to verify it
Pass all test case.
Which release branch to backport (provide reason below if selected)
Tested branch (Please provide the tested image version)
Description for the changelog
Ignore error during config reload in BGP/QOS/FPC test cases
Link to config_db schema for YANG module changes
A picture of a cute animal (not mandatory but encouraged)