Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bookworm] Update platform-modules-dell for Bookworm #16735

Closed
saiarcot895 opened this issue Sep 28, 2023 · 3 comments
Closed

[Bookworm] Update platform-modules-dell for Bookworm #16735

saiarcot895 opened this issue Sep 28, 2023 · 3 comments
Labels
baseimage Bookworm DELL Triaged this issue has been triaged

Comments

@saiarcot895
Copy link
Contributor

Description

In the bookworm branch, platform-modules-dell has been disabled from the Broadcom build as it needs changes for Bookworm. This needs to be updated for Bookworm.

Steps to reproduce the issue:

Describe the results you received:

Describe the results you expected:

Output of show version:

(paste your output here)

Output of show techsupport:

(paste your output here or download and attach the file here )

Additional information you deem important (e.g. issue happens only occasionally):

@jeff-yin
Copy link
Collaborator

jeff-yin commented Oct 4, 2023

Please assign to @arunlk-dell @aravindmani-1 @srideepDell
(I'm not able to designate assignees)

@vmittal-msft vmittal-msft added Triaged this issue has been triaged DELL labels Oct 11, 2023
@saiarcot895
Copy link
Contributor Author

@jeff-yin Looks like I can't assign any of them to this task for some reason. They're not showing up in the list of people I can assign to.

yxieca pushed a commit that referenced this issue Oct 26, 2023
Why I did it
[Bookworm] Update platform-modules-dell for Bookworm #16735

How I did it
Modified platform driver to comply with bookworm kernel.
Removed MODULE_SUPPORTED_DEVICE wherever used.
Modified python build commands for building whl packages.

How to verify it
Verify whether all the platform bookworm debs are built.
make target/debs/bookworm/platform-modules-z9100_1.1_amd64.deb
Load the platform debian into the device and install it in bookworm image.
Verify the platform related CLI and the functionality
@saiarcot895
Copy link
Contributor Author

Fixed by #17003.

saiarcot895 pushed a commit to saiarcot895/sonic-buildimage that referenced this issue Nov 8, 2023
Why I did it
[Bookworm] Update platform-modules-dell for Bookworm sonic-net#16735

How I did it
Modified platform driver to comply with bookworm kernel.
Removed MODULE_SUPPORTED_DEVICE wherever used.
Modified python build commands for building whl packages.

How to verify it
Verify whether all the platform bookworm debs are built.
make target/debs/bookworm/platform-modules-z9100_1.1_amd64.deb
Load the platform debian into the device and install it in bookworm image.
Verify the platform related CLI and the functionality
saiarcot895 pushed a commit to saiarcot895/sonic-buildimage that referenced this issue Nov 21, 2023
Why I did it
[Bookworm] Update platform-modules-dell for Bookworm sonic-net#16735

How I did it
Modified platform driver to comply with bookworm kernel.
Removed MODULE_SUPPORTED_DEVICE wherever used.
Modified python build commands for building whl packages.

How to verify it
Verify whether all the platform bookworm debs are built.
make target/debs/bookworm/platform-modules-z9100_1.1_amd64.deb
Load the platform debian into the device and install it in bookworm image.
Verify the platform related CLI and the functionality
yxieca pushed a commit that referenced this issue Nov 22, 2023
Why I did it
[Bookworm] Update platform-modules-dell for Bookworm #16735

How I did it
Modified platform driver to comply with bookworm kernel.
Removed MODULE_SUPPORTED_DEVICE wherever used.
Modified python build commands for building whl packages.

How to verify it
Verify whether all the platform bookworm debs are built.
make target/debs/bookworm/platform-modules-z9100_1.1_amd64.deb
Load the platform debian into the device and install it in bookworm image.
Verify the platform related CLI and the functionality
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
baseimage Bookworm DELL Triaged this issue has been triaged
Projects
None yet
Development

No branches or pull requests

3 participants