sonic-net / sonic-platform-daemons

Platform module daemons for SONiC
Other
25 stars 159 forks source link

[xcvrd] Use new logger implementation and enable runtime log level configuration #515

Closed Junchao-Mellanox closed 2 weeks ago

Junchao-Mellanox commented 4 months ago

Depends on https://github.com/sonic-net/sonic-buildimage/pull/19611

Description

Use new logger class SysLogger and enable runtime log level configuration

Motivation and Context

Use new logger class SysLogger and enable runtime log level configuration

How Has This Been Tested?

Manual test

Additional Information (Optional)

Junchao-Mellanox commented 4 months ago

The checker failure is expected. It should pass after https://github.com/sonic-net/sonic-buildimage/pull/19611 merged.

prgeor commented 2 months ago

@Junchao-Mellanox is this PR ready for review? I see in draft mode

Junchao-Mellanox commented 1 month ago

/azpw run Azure.sonic-platform-daemons

mssonicbld commented 1 month ago

/AzurePipelines run Azure.sonic-platform-daemons

azure-pipelines[bot] commented 1 month ago
Azure Pipelines successfully started running 1 pipeline(s).
Junchao-Mellanox commented 3 weeks ago

/azpw run Azure.sonic-platform-daemons

mssonicbld commented 3 weeks ago

/AzurePipelines run Azure.sonic-platform-daemons

azure-pipelines[bot] commented 3 weeks ago
Azure Pipelines successfully started running 1 pipeline(s).
Junchao-Mellanox commented 3 weeks ago

/azpw run Azure.sonic-platform-daemons

mssonicbld commented 3 weeks ago

/AzurePipelines run Azure.sonic-platform-daemons

azure-pipelines[bot] commented 3 weeks ago
Azure Pipelines successfully started running 1 pipeline(s).
Junchao-Mellanox commented 3 weeks ago

/azpw run Azure.sonic-platform-daemons

mssonicbld commented 3 weeks ago

/AzurePipelines run Azure.sonic-platform-daemons

azure-pipelines[bot] commented 3 weeks ago
Azure Pipelines successfully started running 1 pipeline(s).
Junchao-Mellanox commented 3 weeks ago

The UT failure is caused by dependency miss. The dependency https://github.com/sonic-net/sonic-buildimage/pull/19611 was merged on 10-27. But there is no vs.master build since 10-25: https://sonic-build.azurewebsites.net/ui/sonic/pipelines/142/builds?branchName=master.

Junchao-Mellanox commented 3 weeks ago

/azpw run Azure.sonic-platform-daemons

mssonicbld commented 3 weeks ago

/AzurePipelines run Azure.sonic-platform-daemons

azure-pipelines[bot] commented 3 weeks ago
Azure Pipelines successfully started running 1 pipeline(s).