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

feat: swap to negativo17 as nvidia driver source #170

Merged
merged 2 commits into from
Apr 15, 2024
Merged

feat: swap to negativo17 as nvidia driver source #170

merged 2 commits into from
Apr 15, 2024

Conversation

bsherman
Copy link
Contributor

@bsherman bsherman commented Apr 13, 2024

This reworks all the bits in akmods for nvidia:

  • removes nvidia driver version as only latest is supported
  • adds negativo17-fedora-nvidia repo to ublue-os-nvidia-addons RPM
  • removes extra metadata from the nvidia-vars create in image

This PR will require a corresponding PR in hwe to ensure proper repos are being used and the new package names are installed since negativo17 has some different package naming.

This reworks all the bits in akmods for nvidia:
- removes nvidia driver version as only latest is supported
- adds negativo17-fedora-nvidia repo to ublue-os-nvidia-addons RPM
- removes extra metadata from the nvidia-vars create in image
@bsherman bsherman linked an issue Apr 13, 2024 that may be closed by this pull request
@bsherman bsherman marked this pull request as ready for review April 14, 2024 05:13
@bsherman bsherman requested a review from castrojo as a code owner April 14, 2024 05:13
@KyleGospo KyleGospo enabled auto-merge April 14, 2024 05:21
@KyleGospo KyleGospo added this pull request to the merge queue Apr 14, 2024
@bsherman bsherman removed this pull request from the merge queue due to a manual request Apr 14, 2024
@bsherman
Copy link
Contributor Author

Despite initially clean builds which looked good (and tested good locally) when we got approvals, it seems like the timing matched up with a kernel update which is causing version skew with rpm repos.

Probably wait until Sunday afternoon to try merging again.

@bsherman bsherman added this pull request to the merge queue Apr 15, 2024
@bsherman bsherman removed this pull request from the merge queue due to a manual request Apr 15, 2024
@bsherman bsherman added this pull request to the merge queue Apr 15, 2024
Merged via the queue into main with commit b64843b Apr 15, 2024
20 checks passed
@bsherman bsherman deleted the nega-nv branch April 15, 2024 04:45
bsherman added a commit that referenced this pull request Apr 15, 2024
bsherman added a commit that referenced this pull request Apr 15, 2024
@bsherman bsherman restored the nega-nv branch April 15, 2024 09:39
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.

switch to negativo17 for nvidia akmod
3 participants