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

Enable CONFIG_SCSI_MPT3SAS for x86_64 #166

Closed
wants to merge 1 commit into from

Conversation

ader1990
Copy link
Contributor

Storage devices that need CONFIG_SCSI_MPT3SAS to be enabled are fairly common on baremetals and I would like to add support for such devices.

@jacobweinstock
Copy link
Member

jacobweinstock commented Jun 1, 2023

Apologies, I will re-open this PR. I'm trying to the validation check to run.

@jacobweinstock
Copy link
Member

Hey @ader1990. Apologies, i think you will need to open a new PR for this. I can't seem to get the validation action to run. Very sorry.

@chrisdoherty4
Copy link
Member

chrisdoherty4 commented Jul 6, 2023

@jacobweinstock I wonder if the repository configuration is borked because this is happening across multiple PRs? It seems it wants the validation job but it doesn't list the workflow name as the other checks do.

@jacobweinstock
Copy link
Member

oh, doh 🤦🏼 . this is this issue: #145.

Storage devices that need CONFIG_SCSI_MPT3SAS to be enabled are fairly
common on baremetals and I would like to add support for such devices.

Signed-off-by: Adrian Vladu <avladu@cloudbasesolutions.com>
@jacobweinstock jacobweinstock mentioned this pull request May 21, 2024
3 tasks
jacobweinstock added a commit that referenced this pull request May 21, 2024
## Description

<!--- Please describe what this PR is going to change -->
Community member requested the need to have CONFIG_SCSI_MPT3SAS enabled.
#166

## Why is this needed

<!--- Link to issue you have raised -->

Fixes: #166 

## How Has This Been Tested?
<!--- Please describe in detail how you tested your changes. -->
<!--- Include details of your testing environment, and the tests you ran
to -->
<!--- see how your change affects other areas of the code, etc. -->


## How are existing users impacted? What migration steps/scripts do we
need?

<!--- Fixes a bug, unblocks installation, removes a component of the
stack etc -->
<!--- Requires a DB migration script, etc. -->


## Checklist:

I have:

- [ ] updated the documentation and/or roadmap (if required)
- [ ] added unit or e2e tests
- [ ] provided instructions on how to upgrade
@jacobweinstock
Copy link
Member

Hey @ader1990, apologies for the very long delay here. We've recently updated the build process and added additional kernels. 6.6, for example. #226 gets your PR merged into main. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants