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

Supporting out-of-tree drivers on OpenShift #357

Closed
wants to merge 29 commits into from

Conversation

zvonkok
Copy link

@zvonkok zvonkok commented Jun 2, 2020

Support out-of-tree drivers on OpenShift for enabling custom hardware and third-party kernel modules on container and non-container based operating systems.

In OpenShift v3.x out-of-tree drivers could be easily installed on the nodes, since the node was a full RHEL node with a subscription and needed tools were installed easily with yum.
In OpenShift v4.x this changed with the introduction of RHCOS. There are currently two different documented ways to enable out-of-tree drivers. One way is using an SRO based operator such as the nvidia GPU operator and the other is using kmods-via-containers.
We want to come up with a unified solution that works for our customers across RHCOS and RHEL. The solution should also help customers that are currently on RHEL7 to consider moving to RHCOS which is fully managed and easier to support in OpenShift.

(As a side note we had DriverContainers running with OCP-3.11 on AtomicHost)

@zvonkok zvonkok changed the title Supporting out-of-tree drivers on BareMetal & OpenShift [WIP]: Supporting out-of-tree drivers on BareMetal & OpenShift Jun 2, 2020
@openshift-ci-robot openshift-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jun 2, 2020
@zvonkok
Copy link
Author

zvonkok commented Jun 2, 2020

@zshi-redhat PTAL

@zvonkok
Copy link
Author

zvonkok commented Jun 2, 2020

@ashcrow
Copy link
Member

ashcrow commented Jun 3, 2020

/cc @darkmuggle

@zvonkok
Copy link
Author

zvonkok commented Jul 7, 2020

Depends on: #384 #389 #363

Zvonko Kaiser and others added 7 commits March 8, 2021 21:34
Co-authored-by: David Gray <40244437+dagrayvid@users.noreply.github.com>
Co-authored-by: David Gray <40244437+dagrayvid@users.noreply.github.com>
Co-authored-by: David Gray <40244437+dagrayvid@users.noreply.github.com>
Co-authored-by: David Gray <40244437+dagrayvid@users.noreply.github.com>
Co-authored-by: David Gray <40244437+dagrayvid@users.noreply.github.com>
Co-authored-by: David Gray <40244437+dagrayvid@users.noreply.github.com>
Co-authored-by: David Gray <40244437+dagrayvid@users.noreply.github.com>
hackmd-deploy and others added 3 commits March 19, 2021 13:04
driver-toolki + kABI section added.
driver-toolki + kABI section added.
Zvonko Kaiser and others added 2 commits March 19, 2021 14:11
Co-authored-by: David Gray <40244437+dagrayvid@users.noreply.github.com>
@openshift-ci
Copy link
Contributor

openshift-ci bot commented May 5, 2021

@zvonkok: The following test failed, say /retest to rerun all failed tests:

Test name Commit Details Rerun command
ci/prow/markdownlint 0185308 link /test markdownlint

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Copy link
Contributor

@SchSeba SchSeba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

small comment

value: "SECRET_ACCESS_KEY"
driverContainer:
source:
git:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

some of the environments that need OOT drivers are in disconnected environments will it be reasonable to add the ability to use a container image as a container for the driver source?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See CR section artifacts: from Claim, Image or HostPath

@zvonkok zvonkok closed this Jul 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet