Skip to content

Workaround to allow openzfs-zfs-dkms install on Ubuntu #15503

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

Merged
merged 1 commit into from
Nov 8, 2023
Merged

Workaround to allow openzfs-zfs-dkms install on Ubuntu #15503

merged 1 commit into from
Nov 8, 2023

Conversation

AllKind
Copy link
Contributor

@AllKind AllKind commented Nov 7, 2023

As shown in #15404#issuecomment-1765002181, Ubuntu kernel has 'Provides: zfs-dkms', which will cause uninstall of the kernel, when attempting to install openzfs-zfs-dkms.
As a workaround remove the 'Conflicts: zfs-dkms' definition from the debian control file.

Motivation and Context

Fixes: #15404 (comment)
It's a workaround, but for now it will allow Ubuntu users to install openzfs-zfs-dkms.
IMHO it should go into 2.2.1. That's why I rushed this quickly.

Description

Only the Conflicts: zfs-dkms definition is removed.

Personally I think the new naming schema should be reverted to the old one, which would solve the current problem.
Because nothing can prevent a distro (Ubuntu) to switch to the new (openzfs-xxx) naming schema and the problem that motivated the change, to prevent what rincebrain said:

The entire point of the prefix was to ensure you couldn't end up with a mix of the two package sets or have to worry about the versioning causing weird hard to debug problems.

would be the same again.
But that's something for you people to decide. Or to find a better solution.

How Has This Been Tested?

Didn't test that personally. Users in the linked issue did that.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Performance enhancement (non-breaking change which improves efficiency)
  • Code cleanup (non-breaking change which makes code smaller or more readable)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Library ABI change (libzfs, libzfs_core, libnvpair, libuutil and libzfsbootenv)
  • Documentation (a change to man pages or other documentation)

Checklist:

@behlendorf behlendorf added the Status: Accepted Ready to integrate (reviewed, tested) label Nov 7, 2023
@AllKind AllKind mentioned this pull request Nov 8, 2023
13 tasks
As shown in #15404#issuecomment-1765002181, Ubuntu kernel has
'Provides: zfs-dkms', which will cause uninstall of the kernel, when
attempting to install openzfs-zfs-dkms.
As a workaround remove the 'Conflicts: zfs-dkms' definition from
the debian control file.

Signed-off-by: Mart Frauenlob <[email protected]>
@behlendorf behlendorf merged commit 3a81bf4 into openzfs:master Nov 8, 2023
ixhamza pushed a commit to truenas/zfs that referenced this pull request Nov 20, 2023
As shown in openzfs#15404#issuecomment-1765002181, Ubuntu kernel has
'Provides: zfs-dkms', which will cause uninstall of the kernel, when
attempting to install openzfs-zfs-dkms.
As a workaround remove the 'Conflicts: zfs-dkms' definition from
the debian control file.

Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Mart Frauenlob <[email protected]>
Closes openzfs#15503
lundman pushed a commit to openzfsonwindows/openzfs that referenced this pull request Dec 12, 2023
As shown in openzfs#15404#issuecomment-1765002181, Ubuntu kernel has
'Provides: zfs-dkms', which will cause uninstall of the kernel, when
attempting to install openzfs-zfs-dkms.
As a workaround remove the 'Conflicts: zfs-dkms' definition from
the debian control file.

Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Mart Frauenlob <[email protected]>
Closes openzfs#15503
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Accepted Ready to integrate (reviewed, tested)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

zfs-2.2.0 building native-deb-utils target fails early on
2 participants