[Mellanox] get_error_descripton should return Not supported for modules that does not support this API

[Junchao-Mellanox]

Why I did it

Not all xcvr API support get_error_description, for example sff8636. For those API types, get_error_description should return "Not supported".

Work item tracking

• Microsoft ADO (number only):

How I did it

get_error_description should return "Not supported".

How to verify it

Manual test
unit test

Which release branch to backport (provide reason below if selected)

• 202411

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

[mssonicbld]

/azp run Azure.sonic-buildimage

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[Junchao-Mellanox]

/azpw ms_conflict

[Junchao-Mellanox]

Hi @StormLiangMS , could you please help check the ms conflict?

[liat-grozovik]

/azpw ms_conflict

[liat-grozovik]

Hi @StormLiangMS , could you please help check the ms conflict?

@StormLiangMS kindly reminder as we are blocked for this PR

[liat-grozovik]

@Junchao-Mellanox is it needed only for master/202411? why not 202405?

[Junchao-Mellanox]

Also needs for 202405 since the previous PR was in 202405.

[liat-grozovik]

/azpw run ms_conflict

[kperumalbfn]

@liushilongbuaa could you pls check the ms_conflict failure?

[liushilongbuaa]

/azpw ms_conflict

[mssonicbld]

Cherry-pick PR to 202411: #21537

[mssonicbld]

Cherry-pick PR to 202405: #21541

[bingwang-ms]

@Junchao-Mellanox There should be another PR to update the expectation in sonic-mgmt test. https://github.com/sonic-net/sonic-mgmt/blob/db2df148ed793d5292acc60588120d13d3b6fade/tests/platform_tests/sfp/test_sfputil.py#L365

[Junchao-Mellanox]

@Junchao-Mellanox There should be another PR to update the expectation in sonic-mgmt test. https://github.com/sonic-net/sonic-mgmt/blob/db2df148ed793d5292acc60588120d13d3b6fade/tests/platform_tests/sfp/test_sfputil.py#L365

Yes, someone from Nvidia will handle it.

[nhe-NV]

@Junchao-Mellanox There should be another PR to update the expectation in sonic-mgmt test. https://github.com/sonic-net/sonic-mgmt/blob/db2df148ed793d5292acc60588120d13d3b6fade/tests/platform_tests/sfp/test_sfputil.py#L365

Yes, someone from Nvidia will handle it.

It is handled in the PR: sonic-net/sonic-mgmt#16573