DellEMC: N3248PXE Initial platform commit

[arunlk-dell]

Why I did it

Added support for the device N3248PXE

How I did it

Implemented the support for the platform N3248PXE
n3248pxe_unit_test_log.txt

Switch Vendor: DellEMC
* Switch SKU: N3248PXE
* ASIC Vendor: Broadcom
* SONiC Image: sonic-broadcom.bin

How to verify it

Verified the show platform commands

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

• 201811
• 201911
• 202006
• 202012
• 202106

Description for the changelog

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

[lgtm-com]

This pull request introduces 61 alerts when merging 16b7c20 into 5ff87e1 - view on LGTM.com

new alerts:

• 22 for Unused import
• 17 for Except block handles 'BaseException'
• 7 for Wrong number of arguments in a class instantiation
• 5 for Variable defined multiple times
• 3 for Unused local variable
• 3 for Result of integer division may be truncated
• 1 for Module imports itself
• 1 for Unreachable code
• 1 for 'import *' may pollute namespace
• 1 for Wrong name for an argument in a class instantiation

[lgtm-com]

This pull request introduces 25 alerts when merging e9d0963 into 086254b - view on LGTM.com

new alerts:

• 7 for Wrong number of arguments in a class instantiation
• 6 for Unused import
• 4 for Except block handles 'BaseException'
• 3 for Result of integer division may be truncated
• 2 for Variable defined multiple times
• 1 for Module imports itself
• 1 for 'import *' may pollute namespace
• 1 for Wrong name for an argument in a class instantiation

[lgtm-com]

This pull request introduces 13 alerts when merging a39e2e0 into 086254b - view on LGTM.com

new alerts:

• 7 for Wrong number of arguments in a class instantiation
• 3 for Result of integer division may be truncated
• 1 for Module imports itself
• 1 for 'import *' may pollute namespace
• 1 for Wrong name for an argument in a class instantiation

[jeff-yin]

@sujinmkang @prgeor Can you review? Thanks

[lgtm-com]

This pull request introduces 13 alerts when merging 4170a69 into 8171bfe - view on LGTM.com

new alerts:

• 7 for Wrong number of arguments in a class instantiation
• 3 for Result of integer division may be truncated
• 1 for Module imports itself
• 1 for 'import *' may pollute namespace
• 1 for Wrong name for an argument in a class instantiation

[jeff-yin]

@arunlk-dell are the remaining LGTM alerts false-positives?
@lguohan can someone re-trigger the pipeline checks? I don't think we have permissions. I checked a while back and they had been all passing. Not sure what happened.

[arunlk-dell]

@jeff-yin Yes, the remaining LGTM alerts are false positive and the wrong name/number of the argument alerts are due to different platform definitions.

[srideepDell]

/AzurePipelines run

[azure-pipelines]

Commenter does not have sufficient privileges for PR 8562 in repo Azure/sonic-buildimage

[arunlk-dell]

/azp run

[azure-pipelines]

Commenter does not have sufficient privileges for PR 8562 in repo Azure/sonic-buildimage

[lgtm-com]

This pull request introduces 13 alerts when merging 42f5b72 into db529af - view on LGTM.com

new alerts:

• 7 for Wrong number of arguments in a class instantiation
• 3 for Result of integer division may be truncated
• 1 for Module imports itself
• 1 for 'import *' may pollute namespace
• 1 for Wrong name for an argument in a class instantiation

[arunlk-dell]

@prgeor @sujinmkang... Can you review the changes? Thanks

[arunlk-dell]

@sujinmkang .....
Have addressed the LGTM alerts, the wrong name/number of the argument alerts are due to different platform definitions.
And 'result of integer division may be truncated' alert can be ignored in python3
Can you merge the pull request.

[lgtm-com]

This pull request introduces 13 alerts when merging 9e9a265 into 62a1f5e - view on LGTM.com

new alerts:

• 7 for Wrong number of arguments in a class instantiation
• 3 for Result of integer division may be truncated
• 1 for Module imports itself
• 1 for 'import *' may pollute namespace
• 1 for Wrong name for an argument in a class instantiation