Skip to content

Filter out unnecessary dataplane logs#21271

Merged
StormLiangMS merged 1 commit intosonic-net:masterfrom
wangxin:fix-dataplane-logger
Dec 1, 2025
Merged

Filter out unnecessary dataplane logs#21271
StormLiangMS merged 1 commit intosonic-net:masterfrom
wangxin:fix-dataplane-logger

Conversation

@wangxin
Copy link
Copy Markdown
Collaborator

@wangxin wangxin commented Nov 11, 2025

Description of PR

Summary:
Fixes # (issue)

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • New Test case
    • Skipped for non-supported platforms
  • Test case improvement

Back port request

  • 202205
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505

Approach

What is the motivation for this PR?

PR #21045 for fixing compatibility issues of docker-sonic-mgmt upgrade removed the hook for setting log levels of couple of loggers.

The code for setting log level of ansible loggers is no longer needed after previous ansible upgrade a few years ago.

However, setting log level for the 'dataplane' logger is still necessary.

How did you do it?

This change added back the code to set log level for the 'dataplane' logger to avoid unnessary logs when 'ptfadapter' is used in test script.

How did you verify/test it?

Any platform specific information?

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

Documentation

PR sonic-net#21045 for fixing compatibility issues of docker-sonic-mgmt upgrade removed the hook for setting log levels of couple of loggers.

The code for setting log level of ansible loggers is no longer needed after previous ansible upgrade a few years ago.

However, setting log level for the 'dataplane' logger is still necessary.

This change added back the code to set log level for the 'dataplane' logger to avoid unnessary logs when 'ptfadapter' is used in test script.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
@wangxin wangxin requested a review from a team as a code owner November 11, 2025 07:08
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

StormLiangMS pushed a commit that referenced this pull request Nov 13, 2025
….04 (#21045) (#21271) (#21289)

* Compatibility fixes for docker-sonic-mgmt based on Ubuntu 24.04 (#21045)

What is the motivation for this PR?
This PR is to continue the effort made by @yutongzhang-microsoft in #18339

The current docker-sonic-mgmt image is based on Ubuntu 20.04 which is end of support now. PR sonic-net/sonic-buildimage#24306 upgraded the base image of docker-sonic-mgmt to Ubuntu 24.04. Together, version of most packages are upgraded too. The upgrade introduced lots of compatibility issues. This PR is to fix all the compatibility issues.

All the fixes are backward compatible. The code change woks with both current and new docker-sonic-mgmt.

How did you do it?
Fix the snmp code caused by pysnmp upgrade.
Fix json dump of ansible result caused by pytest-ansible upgrade.
How did you verify/test it?
Take advantage of the current sonic-mgmt PR testing.
Verified that the code change works with new docker-sonic-mgmt in #20851
Verified that the code change works with the current docker-sonic-mgmt in this PR.

* Filter out unnecessary dataplane logs

PR #21045 for fixing compatibility issues of docker-sonic-mgmt upgrade removed the hook for setting log levels of couple of loggers.

The code for setting log level of ansible loggers is no longer needed after previous ansible upgrade a few years ago.

However, setting log level for the 'dataplane' logger is still necessary.

This change added back the code to set log level for the 'dataplane' logger to avoid unnessary logs when 'ptfadapter' is used in test script.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>

---------

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
StormLiangMS pushed a commit that referenced this pull request Nov 17, 2025
….04 (#21045) (#21271) (#21285)

* Compatibility fixes for docker-sonic-mgmt based on Ubuntu 24.04 (#21045)

What is the motivation for this PR?
This PR is to continue the effort made by @yutongzhang-microsoft in #18339

The current docker-sonic-mgmt image is based on Ubuntu 20.04 which is end of support now. PR sonic-net/sonic-buildimage#24306 upgraded the base image of docker-sonic-mgmt to Ubuntu 24.04. Together, version of most packages are upgraded too. The upgrade introduced lots of compatibility issues. This PR is to fix all the compatibility issues.

All the fixes are backward compatible. The code change woks with both current and new docker-sonic-mgmt.

How did you do it?
Fix the snmp code caused by pysnmp upgrade.
Fix json dump of ansible result caused by pytest-ansible upgrade.
How did you verify/test it?
Take advantage of the current sonic-mgmt PR testing.
Verified that the code change works with new docker-sonic-mgmt in #20851
Verified that the code change works with the current docker-sonic-mgmt in this PR.

* Filter out unnecessary dataplane logs

PR #21045 for fixing compatibility issues of docker-sonic-mgmt upgrade removed the hook for setting log levels of couple of loggers.

The code for setting log level of ansible loggers is no longer needed after previous ansible upgrade a few years ago.

However, setting log level for the 'dataplane' logger is still necessary.

This change added back the code to set log level for the 'dataplane' logger to avoid unnessary logs when 'ptfadapter' is used in test script.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>

* Remove unused import

---------

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
wangxin added a commit that referenced this pull request Nov 19, 2025
….04 (#21045) (#21271) (#21288)

* Compatibility fixes for docker-sonic-mgmt based on Ubuntu 24.04 (#21045)

What is the motivation for this PR?
This PR is to continue the effort made by @yutongzhang-microsoft in #18339

The current docker-sonic-mgmt image is based on Ubuntu 20.04 which is end of support now. PR sonic-net/sonic-buildimage#24306 upgraded the base image of docker-sonic-mgmt to Ubuntu 24.04. Together, version of most packages are upgraded too. The upgrade introduced lots of compatibility issues. This PR is to fix all the compatibility issues.

All the fixes are backward compatible. The code change woks with both current and new docker-sonic-mgmt.

How did you do it?
Fix the snmp code caused by pysnmp upgrade.
Fix json dump of ansible result caused by pytest-ansible upgrade.
How did you verify/test it?
Take advantage of the current sonic-mgmt PR testing.
Verified that the code change works with new docker-sonic-mgmt in #20851
Verified that the code change works with the current docker-sonic-mgmt in this PR.

* Filter out unnecessary dataplane logs

PR #21045 for fixing compatibility issues of docker-sonic-mgmt upgrade removed the hook for setting log levels of couple of loggers.

The code for setting log level of ansible loggers is no longer needed after previous ansible upgrade a few years ago.

However, setting log level for the 'dataplane' logger is still necessary.

This change added back the code to set log level for the 'dataplane' logger to avoid unnessary logs when 'ptfadapter' is used in test script.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
wangxin added a commit that referenced this pull request Nov 19, 2025
….04 (#21045) (#21271) (#21287)

* Compatibility fixes for docker-sonic-mgmt based on Ubuntu 24.04 (#21045)

What is the motivation for this PR?
This PR is to continue the effort made by @yutongzhang-microsoft in #18339

The current docker-sonic-mgmt image is based on Ubuntu 20.04 which is end of support now. PR sonic-net/sonic-buildimage#24306 upgraded the base image of docker-sonic-mgmt to Ubuntu 24.04. Together, version of most packages are upgraded too. The upgrade introduced lots of compatibility issues. This PR is to fix all the compatibility issues.

All the fixes are backward compatible. The code change woks with both current and new docker-sonic-mgmt.

How did you do it?
Fix the snmp code caused by pysnmp upgrade.
Fix json dump of ansible result caused by pytest-ansible upgrade.
How did you verify/test it?
Take advantage of the current sonic-mgmt PR testing.
Verified that the code change works with new docker-sonic-mgmt in #20851
Verified that the code change works with the current docker-sonic-mgmt in this PR.

* Filter out unnecessary dataplane logs

PR #21045 for fixing compatibility issues of docker-sonic-mgmt upgrade removed the hook for setting log levels of couple of loggers.

The code for setting log level of ansible loggers is no longer needed after previous ansible upgrade a few years ago.

However, setting log level for the 'dataplane' logger is still necessary.

This change added back the code to set log level for the 'dataplane' logger to avoid unnessary logs when 'ptfadapter' is used in test script.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
wangxin added a commit that referenced this pull request Nov 20, 2025
….04 (#21045) (#21271) (#21286)

* Compatibility fixes for docker-sonic-mgmt based on Ubuntu 24.04 (#21045)

What is the motivation for this PR?
This PR is to continue the effort made by @yutongzhang-microsoft in #18339

The current docker-sonic-mgmt image is based on Ubuntu 20.04 which is end of support now. PR sonic-net/sonic-buildimage#24306 upgraded the base image of docker-sonic-mgmt to Ubuntu 24.04. Together, version of most packages are upgraded too. The upgrade introduced lots of compatibility issues. This PR is to fix all the compatibility issues.

All the fixes are backward compatible. The code change woks with both current and new docker-sonic-mgmt.

How did you do it?
Fix the snmp code caused by pysnmp upgrade.
Fix json dump of ansible result caused by pytest-ansible upgrade.
How did you verify/test it?
Take advantage of the current sonic-mgmt PR testing.
Verified that the code change works with new docker-sonic-mgmt in #20851
Verified that the code change works with the current docker-sonic-mgmt in this PR.

* Filter out unnecessary dataplane logs

PR #21045 for fixing compatibility issues of docker-sonic-mgmt upgrade removed the hook for setting log levels of couple of loggers.

The code for setting log level of ansible loggers is no longer needed after previous ansible upgrade a few years ago.

However, setting log level for the 'dataplane' logger is still necessary.

This change added back the code to set log level for the 'dataplane' logger to avoid unnessary logs when 'ptfadapter' is used in test script.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
@wangxin
Copy link
Copy Markdown
Collaborator Author

wangxin commented Nov 26, 2025

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@wangxin
Copy link
Copy Markdown
Collaborator Author

wangxin commented Nov 26, 2025

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Copy Markdown
Collaborator

@StormLiangMS StormLiangMS left a comment

Choose a reason for hiding this comment

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

LGTM

@StormLiangMS StormLiangMS added the Request for 202511 branch Request to backport a change to 202511 branch label Dec 1, 2025
@StormLiangMS StormLiangMS merged commit f657f7f into sonic-net:master Dec 1, 2025
20 checks passed
opcoder0 pushed a commit to opcoder0/sonic-mgmt that referenced this pull request Dec 8, 2025
PR sonic-net#21045 for fixing compatibility issues of docker-sonic-mgmt upgrade removed the hook for setting log levels of couple of loggers.

The code for setting log level of ansible loggers is no longer needed after previous ansible upgrade a few years ago.

However, setting log level for the 'dataplane' logger is still necessary.

This change added back the code to set log level for the 'dataplane' logger to avoid unnessary logs when 'ptfadapter' is used in test script.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
dcaugher pushed a commit to dcaugher/sonic-mgmt that referenced this pull request Dec 8, 2025
PR sonic-net#21045 for fixing compatibility issues of docker-sonic-mgmt upgrade removed the hook for setting log levels of couple of loggers.

The code for setting log level of ansible loggers is no longer needed after previous ansible upgrade a few years ago.

However, setting log level for the 'dataplane' logger is still necessary.

This change added back the code to set log level for the 'dataplane' logger to avoid unnessary logs when 'ptfadapter' is used in test script.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
Signed-off-by: Dan Caugherty <dcaugher@cisco.com>
nissampa pushed a commit to nissampa/sonic-mgmt_dpu_test that referenced this pull request Dec 9, 2025
PR sonic-net#21045 for fixing compatibility issues of docker-sonic-mgmt upgrade removed the hook for setting log levels of couple of loggers.

The code for setting log level of ansible loggers is no longer needed after previous ansible upgrade a few years ago.

However, setting log level for the 'dataplane' logger is still necessary.

This change added back the code to set log level for the 'dataplane' logger to avoid unnessary logs when 'ptfadapter' is used in test script.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
Signed-off-by: Nishanth Sampath Kumar <nissampa@cisco.com>
selldinesh pushed a commit to selldinesh/sonic-mgmt that referenced this pull request Dec 11, 2025
PR sonic-net#21045 for fixing compatibility issues of docker-sonic-mgmt upgrade removed the hook for setting log levels of couple of loggers.

The code for setting log level of ansible loggers is no longer needed after previous ansible upgrade a few years ago.

However, setting log level for the 'dataplane' logger is still necessary.

This change added back the code to set log level for the 'dataplane' logger to avoid unnessary logs when 'ptfadapter' is used in test script.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
Signed-off-by: selldinesh <dinesh.sellappan@keysight.com>
echuawu pushed a commit to echuawu/sonic-mgmt that referenced this pull request Dec 12, 2025
PR sonic-net#21045 for fixing compatibility issues of docker-sonic-mgmt upgrade removed the hook for setting log levels of couple of loggers.

The code for setting log level of ansible loggers is no longer needed after previous ansible upgrade a few years ago.

However, setting log level for the 'dataplane' logger is still necessary.

This change added back the code to set log level for the 'dataplane' logger to avoid unnessary logs when 'ptfadapter' is used in test script.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
saravanan-nexthop pushed a commit to saravanan-nexthop/sonic-mgmt that referenced this pull request Dec 15, 2025
PR sonic-net#21045 for fixing compatibility issues of docker-sonic-mgmt upgrade removed the hook for setting log levels of couple of loggers.

The code for setting log level of ansible loggers is no longer needed after previous ansible upgrade a few years ago.

However, setting log level for the 'dataplane' logger is still necessary.

This change added back the code to set log level for the 'dataplane' logger to avoid unnessary logs when 'ptfadapter' is used in test script.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
Signed-off-by: Saravanan <saravanan@nexthop.ai>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 16, 2025
PR sonic-net#21045 for fixing compatibility issues of docker-sonic-mgmt upgrade removed the hook for setting log levels of couple of loggers.

The code for setting log level of ansible loggers is no longer needed after previous ansible upgrade a few years ago.

However, setting log level for the 'dataplane' logger is still necessary.

This change added back the code to set log level for the 'dataplane' logger to avoid unnessary logs when 'ptfadapter' is used in test script.

Signed-off-by: Xin Wang <xiwang5@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
PR sonic-net#21045 for fixing compatibility issues of docker-sonic-mgmt upgrade removed the hook for setting log levels of couple of loggers.

The code for setting log level of ansible loggers is no longer needed after previous ansible upgrade a few years ago.

However, setting log level for the 'dataplane' logger is still necessary.

This change added back the code to set log level for the 'dataplane' logger to avoid unnessary logs when 'ptfadapter' is used in test script.

Signed-off-by: Xin Wang <xiwang5@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
PR sonic-net#21045 for fixing compatibility issues of docker-sonic-mgmt upgrade removed the hook for setting log levels of couple of loggers.

The code for setting log level of ansible loggers is no longer needed after previous ansible upgrade a few years ago.

However, setting log level for the 'dataplane' logger is still necessary.

This change added back the code to set log level for the 'dataplane' logger to avoid unnessary logs when 'ptfadapter' is used in test script.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 21, 2025
PR sonic-net#21045 for fixing compatibility issues of docker-sonic-mgmt upgrade removed the hook for setting log levels of couple of loggers.

The code for setting log level of ansible loggers is no longer needed after previous ansible upgrade a few years ago.

However, setting log level for the 'dataplane' logger is still necessary.

This change added back the code to set log level for the 'dataplane' logger to avoid unnessary logs when 'ptfadapter' is used in test script.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
vrajeshe pushed a commit to Akshath-17/sonic-mgmt that referenced this pull request Jan 4, 2026
PR sonic-net#21045 for fixing compatibility issues of docker-sonic-mgmt upgrade removed the hook for setting log levels of couple of loggers.

The code for setting log level of ansible loggers is no longer needed after previous ansible upgrade a few years ago.

However, setting log level for the 'dataplane' logger is still necessary.

This change added back the code to set log level for the 'dataplane' logger to avoid unnessary logs when 'ptfadapter' is used in test script.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
Signed-off-by: Venkata Gouri Rajesh Etla <vrajeshe@cisco.com>
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Jan 13, 2026
PR sonic-net#21045 for fixing compatibility issues of docker-sonic-mgmt upgrade removed the hook for setting log levels of couple of loggers.

The code for setting log level of ansible loggers is no longer needed after previous ansible upgrade a few years ago.

However, setting log level for the 'dataplane' logger is still necessary.

This change added back the code to set log level for the 'dataplane' logger to avoid unnessary logs when 'ptfadapter' is used in test script.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
yifan-nexthop pushed a commit to nexthop-ai/sonic-mgmt that referenced this pull request Jan 14, 2026
PR sonic-net#21045 for fixing compatibility issues of docker-sonic-mgmt upgrade removed the hook for setting log levels of couple of loggers.

The code for setting log level of ansible loggers is no longer needed after previous ansible upgrade a few years ago.

However, setting log level for the 'dataplane' logger is still necessary.

This change added back the code to set log level for the 'dataplane' logger to avoid unnessary logs when 'ptfadapter' is used in test script.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
Signed-off-by: YiFan Wang <yifan@nexthop.ai>
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Jan 20, 2026
PR sonic-net#21045 for fixing compatibility issues of docker-sonic-mgmt upgrade removed the hook for setting log levels of couple of loggers.

The code for setting log level of ansible loggers is no longer needed after previous ansible upgrade a few years ago.

However, setting log level for the 'dataplane' logger is still necessary.

This change added back the code to set log level for the 'dataplane' logger to avoid unnessary logs when 'ptfadapter' is used in test script.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202511: #22014

PriyanshTratiya pushed a commit to PriyanshTratiya/sonic-mgmt that referenced this pull request Jan 21, 2026
PR sonic-net#21045 for fixing compatibility issues of docker-sonic-mgmt upgrade removed the hook for setting log levels of couple of loggers.

The code for setting log level of ansible loggers is no longer needed after previous ansible upgrade a few years ago.

However, setting log level for the 'dataplane' logger is still necessary.

This change added back the code to set log level for the 'dataplane' logger to avoid unnessary logs when 'ptfadapter' is used in test script.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
Signed-off-by: Priyansh Tratiya <ptratiya@microsoft.com>
lakshmi-nexthop pushed a commit to lakshmi-nexthop/sonic-mgmt that referenced this pull request Jan 28, 2026
PR sonic-net#21045 for fixing compatibility issues of docker-sonic-mgmt upgrade removed the hook for setting log levels of couple of loggers.

The code for setting log level of ansible loggers is no longer needed after previous ansible upgrade a few years ago.

However, setting log level for the 'dataplane' logger is still necessary.

This change added back the code to set log level for the 'dataplane' logger to avoid unnessary logs when 'ptfadapter' is used in test script.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
Signed-off-by: Lakshmi Yarramaneni <lakshmi@nexthop.ai>
ytzur1 pushed a commit to ytzur1/sonic-mgmt that referenced this pull request Jan 29, 2026
PR sonic-net#21045 for fixing compatibility issues of docker-sonic-mgmt upgrade removed the hook for setting log levels of couple of loggers.

The code for setting log level of ansible loggers is no longer needed after previous ansible upgrade a few years ago.

However, setting log level for the 'dataplane' logger is still necessary.

This change added back the code to set log level for the 'dataplane' logger to avoid unnessary logs when 'ptfadapter' is used in test script.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
ytzur1 pushed a commit to ytzur1/sonic-mgmt that referenced this pull request Feb 2, 2026
PR sonic-net#21045 for fixing compatibility issues of docker-sonic-mgmt upgrade removed the hook for setting log levels of couple of loggers.

The code for setting log level of ansible loggers is no longer needed after previous ansible upgrade a few years ago.

However, setting log level for the 'dataplane' logger is still necessary.

This change added back the code to set log level for the 'dataplane' logger to avoid unnessary logs when 'ptfadapter' is used in test script.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
Signed-off-by: Yael Tzur <ytzur@nvidia.com>
abhishek-nexthop pushed a commit to nexthop-ai/sonic-mgmt that referenced this pull request Feb 6, 2026
PR sonic-net#21045 for fixing compatibility issues of docker-sonic-mgmt upgrade removed the hook for setting log levels of couple of loggers.

The code for setting log level of ansible loggers is no longer needed after previous ansible upgrade a few years ago.

However, setting log level for the 'dataplane' logger is still necessary.

This change added back the code to set log level for the 'dataplane' logger to avoid unnessary logs when 'ptfadapter' is used in test script.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
rraghav-cisco pushed a commit to rraghav-cisco/sonic-mgmt that referenced this pull request Feb 13, 2026
PR sonic-net#21045 for fixing compatibility issues of docker-sonic-mgmt upgrade removed the hook for setting log levels of couple of loggers.

The code for setting log level of ansible loggers is no longer needed after previous ansible upgrade a few years ago.

However, setting log level for the 'dataplane' logger is still necessary.

This change added back the code to set log level for the 'dataplane' logger to avoid unnessary logs when 'ptfadapter' is used in test script.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
Signed-off-by: Raghavendran Ramanathan <rraghav@cisco.com>
anilal-amd pushed a commit to anilal-amd/anilal-forked-sonic-mgmt that referenced this pull request Feb 19, 2026
PR sonic-net#21045 for fixing compatibility issues of docker-sonic-mgmt upgrade removed the hook for setting log levels of couple of loggers.

The code for setting log level of ansible loggers is no longer needed after previous ansible upgrade a few years ago.

However, setting log level for the 'dataplane' logger is still necessary.

This change added back the code to set log level for the 'dataplane' logger to avoid unnessary logs when 'ptfadapter' is used in test script.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
Signed-off-by: Zhuohui Tan <zhuohui.tan@amd.com>
abhishek-nexthop pushed a commit to nexthop-ai/sonic-mgmt that referenced this pull request Mar 17, 2026
PR sonic-net#21045 for fixing compatibility issues of docker-sonic-mgmt upgrade removed the hook for setting log levels of couple of loggers.

The code for setting log level of ansible loggers is no longer needed after previous ansible upgrade a few years ago.

However, setting log level for the 'dataplane' logger is still necessary.

This change added back the code to set log level for the 'dataplane' logger to avoid unnessary logs when 'ptfadapter' is used in test script.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
Signed-off-by: Abhishek <abhishek@nexthop.ai>
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Mar 27, 2026
PR sonic-net#21045 for fixing compatibility issues of docker-sonic-mgmt upgrade removed the hook for setting log levels of couple of loggers.

The code for setting log level of ansible loggers is no longer needed after previous ansible upgrade a few years ago.

However, setting log level for the 'dataplane' logger is still necessary.

This change added back the code to set log level for the 'dataplane' logger to avoid unnessary logs when 'ptfadapter' is used in test script.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants