This repository was archived by the owner on Jan 3, 2023. It is now read-only.
Use LAZY_INSTALL to avoid all drivers install to all platforms. Remove the bf_kdrv, bf_tun and test to avoid install WNC package failed. #6
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
- What I did
All platform drivers are installed into the wedge100bf-32x device and cause a bunch of errors. However, in SONiC architecture, the platform driver is independent and only the specific platform device driver will be installed. The issue should not happen. The root cause is that the Barefoot private github install all platform drivers in platform/barefoot/one-image.mk. This could be modified by using LAZY_INSTALL like https://github.com/Azure/sonic-buildimage/blob/master/platform/broadcom/one-image.mk. And this LAZY_INSTALL procedure will check machine.conf platform and install specific driver.
- How I did it
Use LAZY_INSTALL procedure to avoid installing all platform drivers in one platform.
Modify WNC package for avoid install failed as follows:
This is becuase the debian package of other platform driver (ex: wnc) has conflict files and cannot be installed into the wedge100bf-32x platform.
The platform-modules-bfn-montara directory is still have install error as follows:
But in this modules it's just install three files (bf_kdrv, bf_tun and test). It I deleted these files will make error. So I keep this packages exist and ignore the install error. You can determine how to fix it or not.
- How to verify it
We have built a fixed image and load into the wedge100bf-32x and Ingrasys s9180-32x. They work fine.
- Description for the changelog
- A picture of a cute animal (not mandatory but encouraged)