Skip to content
This repository has been archived by the owner on Aug 16, 2023. It is now read-only.

gpu_operator_set_repo-config: new role to set spec.driver.repoConfig #124

Merged
merged 4 commits into from
May 10, 2021

Conversation

kpouget
Copy link
Collaborator

@kpouget kpouget commented Apr 28, 2021

This PR add a new toolbox command toolbox/gpu-operator/set_repo-config.sh <repofile|--rhel-beta> to tell the GPU Operator driver Pod to use a custom DNF repo list or RHEL beta repositories.

It should solve OCP 4.8 testing, at least when the self-signed certificate is removed from the chain, or we change the repo URL

@kpouget
Copy link
Collaborator Author

kpouget commented Apr 28, 2021

/test gpu-operator-e2e

@kpouget
Copy link
Collaborator Author

kpouget commented Apr 29, 2021

/approve

@openshift-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: kpouget

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci-robot openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Apr 29, 2021
@kpouget kpouget force-pushed the custom-repo branch 3 times, most recently from 260f946 to fa7a4bd Compare May 4, 2021 11:46
@kpouget
Copy link
Collaborator Author

kpouget commented May 4, 2021

/test gpu-operator-e2e

@kpouget kpouget force-pushed the custom-repo branch 3 times, most recently from 19cfb85 to fa3d0dc Compare May 6, 2021 09:31
@kpouget kpouget requested review from omertuc and removed request for ArangoGutierrez May 6, 2021 14:39
@openshift-ci openshift-ci bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label May 6, 2021
@openshift-ci openshift-ci bot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label May 7, 2021
@kpouget
Copy link
Collaborator Author

kpouget commented May 7, 2021

/test gpu-operator-e2e

1 similar comment
@kpouget
Copy link
Collaborator Author

kpouget commented May 7, 2021

/test gpu-operator-e2e

@omertuc
Copy link
Contributor

omertuc commented May 10, 2021

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label May 10, 2021
@openshift-merge-robot openshift-merge-robot merged commit c40b1ab into openshift-psap:master May 10, 2021
kpouget added a commit to kpouget/ci-artifacts that referenced this pull request May 11, 2021
kpouget added a commit to kpouget/ci-artifacts that referenced this pull request May 11, 2021
kpouget added a commit to kpouget/ci-artifacts that referenced this pull request May 11, 2021
@kpouget kpouget deleted the custom-repo branch May 17, 2021 15:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. progress/ready-for-review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants