Initial commit for introducing the SONiC Platform Development Env (Re…

[geans-pin]

What I did
This PR is a resubmit of PR#3778

How I did it
The PDE silicon test harness and platform test harness can be found in
src/sonic-platform-pdk-pde

How to verify it
You must have both this PR and PR below.
sonic-net/sonic-platform-pdk-pde#28

Enable PDE by un-commenting line in rules/config
INCLUDE_PDE=y

Currently only Broadcom support has been added.

Description for the changelog
Adding support for Platform Development Environment (PDE)

[lgtm-com]

This pull request introduces 4 alerts when merging 138a7fe41449ce3814d5f85e260ba542eacf3091 into b2286a2 - view on LGTM.com

new alerts:

• 4 for Unused import

[lgtm-com]

This pull request introduces 4 alerts when merging c152419 into 0bc0f98 - view on LGTM.com

new alerts:

• 4 for Unused import

[azure-pipelines]

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

[adyeung]

@jleveque Pls help review

[lgtm-com]

This pull request introduces 4 alerts when merging 1945c12 into 2cd61bc - view on LGTM.com

new alerts:

• 4 for Unused import

[lgtm-com]

This pull request introduces 4 alerts when merging 38592f9 into 237849a - view on LGTM.com

new alerts:

• 4 for Unused import

[lgtm-com]

This pull request introduces 4 alerts when merging d0f6bb2 into 1a3cab4 - view on LGTM.com

new alerts:

• 4 for Unused import

[geans-pin]

jleveque,

I have fixed the review comment. However, the kvm test is fail in the checker which seems not related with the fix.
Can you re-run the pipeline build ?

Geans

[geans-pin]

/azure-pipelines

[geans-pin]

/azpw run

[mssonicbld]

/AzurePipelines run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[geans-pin]

/azpw run

[mssonicbld]

/AzurePipelines run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[qiluo-msft]

Sorry for misleading. I notice an unexpected file src/sonic-platform-pde/.gitkeep
Please double check the page https://git-scm.com/book/en/v2/Git-Tools-Submodules

---

In reply to: 873021616

[geans-pin]

Thanks qiluo, Here we are.. Just committed the fix

[qiluo-msft]

The submodule looks good now. Please check with other reviewers.

[geans-pin]

xumia, jleveque, lguohan,

All of the reviewed comments are fixed and build are Pass.
Are we ready for the merge ?

Geans

[anshuv-mfst]

Hi @xumia , @qiluo-msft - can you please approve and merge by/before 7/9, this PR is needed in 202106 release.

[adyeung]

@xumia, @qiluo-msft, @lguohan, @anshuv-mfst, pls help approve and merge

[geans-pin]

/azpw run

[mssonicbld]

/AzurePipelines run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[lguohan]

[MikeHcChen]

Recently, I tried to build PDE environment with 202111 branch codes.
I added "INCLUDE_PDE = y" and "PDDF_SUPPORT = y" in "sonic-buildimage/rules/config" file and
revised "sonic-buildimage/dockers/docker-pde/Dockerfile.j2" as following:
--- a/dockers/docker-pde/Dockerfile.j2
+++ b/dockers/docker-pde/Dockerfile.j2
@@ -1,4 +1,4 @@
-FROM {{ docker_pde_load_image }}
+FROM docker-config-engine-buster

Then I tried to build PDE using commands "make init;make configure PLATFORM=broadcom; make target/sonic-broadcom-pde.bin". I found I can get docker-pde.gz (PDE docker) and sonic-platform-pde_1.0_amd64.deb, but cannot get sonic-broadcom-pde.bin. The error logs like:

make: *** No rule to make target 'target/sonic-broadcom-pde.bin'. Stop.
Makefile.work:304: recipe for target 'target/sonic-broadcom-pde.bin' failed
make[1]: *** [target/sonic-broadcom-pde.bin] Error 2
make[1]: Leaving directory '/home/mike/sonic_202111_branch/sonic-buildimage'
Makefile:30: recipe for target 'target/sonic-broadcom-pde.bin' failed
make: *** [target/sonic-broadcom-pde.bin] Error 2

Is there anything I should add to make the PDE image in SONiC?
Any idea is welcome. Thanks.