Skip to content

fix huge console log generated by get_multiple_flows#9047

Merged
XuChen-MSFT merged 1 commit intosonic-net:202012from
XuChen-MSFT:xuchen3/202012/remove-huge-console-log
Jul 20, 2023
Merged

fix huge console log generated by get_multiple_flows#9047
XuChen-MSFT merged 1 commit intosonic-net:202012from
XuChen-MSFT:xuchen3/202012/remove-huge-console-log

Conversation

@XuChen-MSFT
Copy link
Copy Markdown
Contributor

@XuChen-MSFT XuChen-MSFT commented Jul 20, 2023

Description of PR

Summary:
Fixes # (issue)

Type of change

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

Back port request

  • 201911
  • 202012
  • 202205

Approach

What is the motivation for this PR?

observed lots console log in qos sai test:


"Trying 2 => 3, src_ip:192.0.0.249 dstip:192.168.0.5",
"Trying 2 => 3, src_ip:192.0.0.249 dstip:192.168.0.5",
"Trying 2 => 3, src_ip:192.0.0.249 dstip:192.168.0.5",
"Trying 2 => 3, src_ip:192.0.0.249 dstip:192.168.0.5",
"Trying 2 => 3, src_ip:192.0.0.249 dstip:192.168.0.5",
"Trying 2 => 3, src_ip:192.0.0.249 dstip:192.168.0.5",
"Trying 2 => 3, src_ip:192.0.0.249 dstip:192.168.0.5",
"Trying 2 => 3, src_ip:192.0.0.249 dstip:192.168.0.5",
"Trying 2 => 3, src_ip:192.0.0.249 dstip:192.168.0.5",
"Trying 2 => 3, src_ip:192.0.0.249 dstip:192.168.0.5",
"Trying 2 => 3, src_ip:192.0.0.249 dstip:192.168.0.5",
"Trying 2 => 3, src_ip:192.0.0.249 dstip:192.168.0.5",
"Trying 2 => 3, src_ip:192.0.0.249 dstip:192.168.0.5",
"Trying 2 => 3, src_ip:192.0.0.249 dstip:192.168.0.5",
"Trying 2 => 3, src_ip:192.0.0.249 dstip:192.168.0.5",
"Trying 2 => 3, src_ip:192.0.0.249 dstip:192.168.0.5",
"Trying 2 => 3, src_ip:192.0.0.249 dstip:192.168.0.5",
"Trying 2 => 3, src_ip:192.0.0.249 dstip:192.168.0.5",
"Trying 2 => 3, src_ip:19"]}

How did you do it?

move this debug log to ptf's syslog

How did you verify/test it?

Any platform specific information?

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

Documentation

wsycqyz
wsycqyz approved these changes Jul 20, 2023
View reviewed changes
Copy link
Copy Markdown
Contributor

@wsycqyz wsycqyz left a comment

Choose a reason for hiding this comment

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

LGTM

@XuChen-MSFT XuChen-MSFT merged commit da84cf6 into sonic-net:202012 Jul 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wsycqyz wsycqyz wsycqyz approved these changes

@wangxin wangxin Awaiting requested review from wangxin

@StormLiangMS StormLiangMS Awaiting requested review from StormLiangMS

@jcaiMR jcaiMR Awaiting requested review from jcaiMR

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@XuChen-MSFT @wsycqyz