Skip to content

[202505] Ignore error during config reload in BGP/QOS/FPC test cases#20198

Merged
yejianquan merged 1 commit intosonic-net:202505from
liuh-80:dev/liuh/cherry-pick-19869-202505
Aug 14, 2025
Merged

[202505] Ignore error during config reload in BGP/QOS/FPC test cases#20198
yejianquan merged 1 commit intosonic-net:202505from
liuh-80:dev/liuh/cherry-pick-19869-202505

Conversation

@liuh-80
Copy link
Copy Markdown
Contributor

@liuh-80 liuh-80 commented Aug 12, 2025

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.

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

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)

…t#19869)

* Ignore error during config reload

* Fix format issue

---------

Co-authored-by: sonicbld <sonicbld@microsoft.com>
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@liuh-80 liuh-80 requested review from opcoder0 and yejianquan August 12, 2025 01:44
@liuh-80 liuh-80 marked this pull request as ready for review August 12, 2025 09:34
@liuh-80 liuh-80 requested a review from StormLiangMS as a code owner August 12, 2025 09:34
Copy link
Copy Markdown
Collaborator

@yejianquan yejianquan left a comment

Choose a reason for hiding this comment

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

LGTM

@yejianquan yejianquan merged commit 4bc8138 into sonic-net:202505 Aug 14, 2025
17 checks passed
@liuh-80 liuh-80 deleted the dev/liuh/cherry-pick-19869-202505 branch August 15, 2025 02:26
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