Skip to content

[cherry-pick] [PR:10860] Loganalyzer regex correction for test_po_update#11184

Merged
wangxin merged 1 commit intosonic-net:202205from
sanjair-git:cherry-10860
Jan 5, 2024
Merged

[cherry-pick] [PR:10860] Loganalyzer regex correction for test_po_update#11184
wangxin merged 1 commit intosonic-net:202205from
sanjair-git:cherry-10860

Conversation

@sanjair-git
Copy link
Contributor

Description of PR

Cherry-Pick PR for 202205 from master #10860

Summary:
Fixes # (issue)

  • For PC tests 'test_po_update.py', seen the following LogAnalyzer mismatch related messages being reported as errors.
  • Issues seen in PC tests running on chassis with topo t2 multi-asic.
    E Match Messages: E Nov 11 20:41:16.136440 ixre-egl-board3 ERR syncd0#syncd: :- process_on_fdb_event: invalid OIDs in fdb notifications, NOT translating and NOT storing in ASIC DB E E Nov 11 20:41:16.136471 ixre-egl-board3 ERR syncd0#syncd: :- process_on_fdb_event: FDB notification was not sent since it contain invalid OIDs, bug?

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Back port request

  • 201911
  • 202012
  • 202205
  • 202305

Approach

What is the motivation for this PR?

  • Loganalyzer mismatch errors seen during PC test suites for test_po_update.py tests w.r.t syncd container messages per asic as shown above.

How did you do it?

  • Fix the existing regex to support syncd container syslog messages per asic.

How did you verify/test it?

  • Ran the PC test cases against a multi-asic line card in a T2 chassis.

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

@judyjoseph judyjoseph requested a review from wangxin January 3, 2024 18:13
@wangxin wangxin merged commit b1b28a7 into sonic-net:202205 Jan 5, 2024
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