[wolflowbj]

Signed-off-by: Zhou,Junde piter.zhou@163.com

Why I did it

support clounix asic for pegatron fn8656 product which will be deployed in internal data center verry soon.

How I did it

support clounix asic for pegatron fn8656 product which will be deployed in internal data center verry soon.

How to verify it

support clounix asic for pegatron fn8656 product which will be deployed in internal data center verry soon.

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

• 201811
• 201911
• 202006
• 202012
• 202106
• 202111
• 202205

Description for the changelog

Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU.

Link to config_db schema for YANG module changes

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

[lgtm-com]

This pull request introduces 35 alerts when merging 6f383cd into a00773a - view on LGTM.com

new alerts:

• 24 for Wrong number of arguments in a class instantiation
• 3 for Except block handles 'BaseException'
• 2 for Backspace escape in regular expression
• 1 for Unnecessary pass
• 1 for Use of the return value of a procedure
• 1 for Module imports itself
• 1 for 'import *' may pollute namespace
• 1 for Wrong number of arguments in a call
• 1 for Binding a socket to all network interfaces

[linux-foundation-easycla]

The committers listed above are authorized under a signed CLA.

• ✅ login: wolflowbj / name: Junde (1d46a8b, 94ace7b, 168ba1c)

[lgtm-com]

This pull request introduces 35 alerts when merging aa3e19e0bd12814d96b9458a6966d126c2a97c82 into a00773a - view on LGTM.com

new alerts:

• 24 for Wrong number of arguments in a class instantiation
• 3 for Except block handles 'BaseException'
• 2 for Backspace escape in regular expression
• 1 for Unnecessary pass
• 1 for Use of the return value of a procedure
• 1 for Module imports itself
• 1 for 'import *' may pollute namespace
• 1 for Wrong number of arguments in a call
• 1 for Binding a socket to all network interfaces

[lgtm-com]

This pull request introduces 35 alerts when merging 0db6014 into d8d0436 - view on LGTM.com

new alerts:

• 24 for Wrong number of arguments in a class instantiation
• 3 for Except block handles 'BaseException'
• 2 for Backspace escape in regular expression
• 1 for Unnecessary pass
• 1 for Use of the return value of a procedure
• 1 for Module imports itself
• 1 for 'import *' may pollute namespace
• 1 for Wrong number of arguments in a call
• 1 for Binding a socket to all network interfaces

[wolflowbj]

@lguohan clould you please help to merge for this pull request ? any comments for this? thanks.

[yxieca]

@wolflowbj can you please look into the LGTM alerts and fix accordingly?

[wolflowbj]

an you please look into the LGTM alerts and fix accordingly?

thanks for your quick response. I will fix it later.

[lgtm-com]

This pull request introduces 24 alerts when merging fa72b343b28579d22c7a39798c21c123b1a7bf77 into e15acb5 - view on LGTM.com

new alerts:

• 24 for Wrong number of arguments in a class instantiation

---

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

[wolflowbj]

an you please look into the LGTM alerts and fix accordingly?

thanks for your quick response. I will fix it later.
24 alerts are caused by the LGTM checked the function from broadcom and mellanox. it seems we shouldn't do this. how can we fix the issue, or ignore this alerts?
@yxieca any comments for the alerts? thanks.

[Battier]

@sujinmkang
Could you please kindly review this PR? Thanks in advance.

[Battier]

@sujinmkang
Hi Sujin,
Could you please kindly take a look this PR? Thanks for your helping.

[wolflowbj]

@yxieca yxieca thanks for review comments last time. and we failed to reach [sujinmkang]. could you please help to review it again or involve anyone who could help us. thanks.