Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Lower log level for message DUT type file doesn't exist #16119

Merged
merged 1 commit into from
Dec 18, 2024

Conversation

bingwang-ms
Copy link
Collaborator

Description of PR

Summary:
This PR is to lower the log level for the message DUT type file doesn't exist from ERROR to INFO.
The log message is annoying in the test result, and the log message is not monitored.
image

Type of change

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

Back port request

  • 202012
  • 202205
  • 202305
  • 202311
  • 202405

Approach

What is the motivation for this PR?

This PR is to lower the log level for the message DUT type file doesn't exist from ERROR to INFO.

How did you do it?

Change the log level.

How did you verify/test it?

The change is verified by running a test on a physical testbed.

Any platform specific information?

No.

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

Not a new test case.

Documentation

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@bingwang-ms
Copy link
Collaborator Author

@xwjiang-ms Can you help review? Thanks

Copy link
Contributor

@xwjiang-ms xwjiang-ms left a comment

Choose a reason for hiding this comment

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

LGTM

@bingwang-ms bingwang-ms merged commit 16ddeff into sonic-net:master Dec 18, 2024
19 checks passed
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Dec 18, 2024
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202405: #16129

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.

3 participants