sonic-platform-daemons icon indicating copy to clipboard operation
sonic-platform-daemons copied to clipboard

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

Open Junchao-Mellanox opened this issue 1 year ago • 1 comments
trafficstars

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 avatar Jul 18 '24 02:07 Junchao-Mellanox

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

Junchao-Mellanox avatar Jul 19 '24 01:07 Junchao-Mellanox

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

prgeor avatar Sep 18 '24 02:09 prgeor

/azpw run Azure.sonic-platform-daemons

Junchao-Mellanox avatar Oct 12 '24 08:10 Junchao-Mellanox

/AzurePipelines run Azure.sonic-platform-daemons

mssonicbld avatar Oct 12 '24 08:10 mssonicbld

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Oct 12 '24 08:10 azure-pipelines[bot]

/azpw run Azure.sonic-platform-daemons

Junchao-Mellanox avatar Oct 28 '24 01:10 Junchao-Mellanox

/AzurePipelines run Azure.sonic-platform-daemons

mssonicbld avatar Oct 28 '24 01:10 mssonicbld

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Oct 28 '24 01:10 azure-pipelines[bot]

/azpw run Azure.sonic-platform-daemons

Junchao-Mellanox avatar Oct 29 '24 08:10 Junchao-Mellanox

/AzurePipelines run Azure.sonic-platform-daemons

mssonicbld avatar Oct 29 '24 08:10 mssonicbld

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Oct 29 '24 08:10 azure-pipelines[bot]

/azpw run Azure.sonic-platform-daemons

Junchao-Mellanox avatar Oct 29 '24 09:10 Junchao-Mellanox

/AzurePipelines run Azure.sonic-platform-daemons

mssonicbld avatar Oct 29 '24 09:10 mssonicbld

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Oct 29 '24 09:10 azure-pipelines[bot]

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 avatar Oct 29 '24 10:10 Junchao-Mellanox

/azpw run Azure.sonic-platform-daemons

Junchao-Mellanox avatar Oct 30 '24 03:10 Junchao-Mellanox

/AzurePipelines run Azure.sonic-platform-daemons

mssonicbld avatar Oct 30 '24 03:10 mssonicbld

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Oct 30 '24 03:10 azure-pipelines[bot]