Skip to content

Conversation

twiest
Copy link
Contributor

@twiest twiest commented Sep 3, 2023

Add install-zfs-dkms-module example

  • Add install-zfs-dkms-module example Containerfile and readme
  • Add install-zfs-dkms-module example to main readme
  • Add install-zfs-dkms-module github workflow file
  • Test build install-zfs-dkms-module example

@twiest twiest force-pushed the install-zfs-dkms-module branch from 0e6b1c8 to ef51469 Compare September 3, 2023 18:15
@twiest
Copy link
Contributor Author

twiest commented Sep 3, 2023

Looks like this is failing because fedora-cores stable has kernel version 6.4.7 and the ZFS package requires a maximum kernel version of 6.3.999.

ZFS does this on purpose because the kernel symbols can change and they have to ensure that they work with each version of the kernel.

This will automatically start working once the ZFS package is updated to allow for kernel 6.4.x.

Until then, I'm going to mark this as a draft PR.

@twiest twiest marked this pull request as draft September 3, 2023 19:03
@jmarrero
Copy link
Member

jmarrero commented Sep 5, 2023

Nice, this is a lot simpler than building the module zfs from scratch like we did for: https://github.com/coreos/layering-examples/blob/main/build-zfs-module/Containerfile.

@jmarrero
Copy link
Member

jmarrero commented Sep 5, 2023

I guess we only run CI when someone changes the example, so merging this would not throw a bunch of failures at us every time fedora updates the kernel to a version not supported by ZFS.

- [x] Add install-zfs-dkms-module example Containerfile and readme
- [x] Add install-zfs-dkms-module example to main readme
- [x] Add install-zfs-dkms-module github workflow file
- [x] Test build install-zfs-dkms-module example
@twiest twiest force-pushed the install-zfs-dkms-module branch from ef51469 to 4aa7de3 Compare September 7, 2023 21:39
@andrewgdunn
Copy link

This works now with the recent release from the upstream zfs project. Thanks for the awesome example.

@jmarrero jmarrero marked this pull request as ready for review November 21, 2023 15:45
@jmarrero
Copy link
Member

@twiest anything new and cool you have done to your Containerfile that you want to update here before I merge it?

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.

4 participants