Bug 1056192

Summary: kernel-PAE-modules-extra no longer provides kernel-modules-extra
Product: [Fedora] Fedora Reporter: Bruno Wolff III <bruno>
Component: xl2tpdAssignee: Paul Wouters <pwouters>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: bruno, didierg-divers, robatino
Target Milestone: ---   
Target Release: ---   
Hardware: i686   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-05-15 12:53:16 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1058331    
Attachments:
Description Flags
Patch to switch to using Requires on individual kernel modules none

Description Bruno Wolff III 2014-01-21 16:35:13 UTC
Description of problem:
kernel-PAE-modules-extra no longer provides kernel-modules-extra, so that if you are using a PAE kernel, dlm will pull in non-PAE kernel packages.

Depending on the intent of requiring kernel-modules-extra, requiring installonlypkg(kernel-module) may be a suitable replacement.

Version-Release number of selected component (if applicable):
xl2tpd-0:1.3.1-14.fc20.i686

Comment 1 Didier G 2014-01-26 10:09:26 UTC
Upgrading to fc20 using fedup on a computer running PAE Kernel, xl2tpd requires no-PAE kernel to install.

Comment 2 Bruno Wolff III 2014-01-26 19:37:03 UTC
I got some more feedback on this from the Fedora kernel guys and we probably need to live with this for a while. (I'll add more details later.) Mostly it is just a bit of disk space being wasted, which shouldn't be a significant problem for most people.

Comment 3 Bruno Wolff III 2014-01-27 15:05:29 UTC
installonlypkg(kernel-module) is not suitable as a requires replacement. We should wait for the *.ko autoprovides feature to make changes.

(If we use a requires on installonlypkg(kernel-module) it could potentially bring back the problem that prompted the original problem. Also that provides is special and it is a bad idea to overload its use.)

Comment 4 Andre Robatino 2014-02-10 16:16:40 UTC
Just saw this with the latest F20 kernel update. Three packages including xl2tpd (which the other two require) were removed on a 32-bit PAE machine. Trying to install them pulls in the non-PAE kernel and kernel-modules-extra.

Comment 5 Paul Wouters 2014-05-13 01:58:24 UTC
Bruno: any idea on what the real fix should be for this?

xl2tpd requires kernel-modules-extras for the l2tp_ppp.ko module.

Comment 6 Bruno Wolff III 2014-05-13 05:09:09 UTC
There isn't a perfect fix because you can have multiple kernels installed and for the package system any of the sets can provide the module, but you really want the running kernel set to provide it. However requiring the kernel module should work pretty well in practice. If you look at the tracking bug 1058331, you will see some more info. Josh has set up a test copr and seems pretty close to enabling auto provides for kernel modules.

Comment 7 Bruno Wolff III 2014-05-14 15:20:13 UTC
Auto provides for kernel modules has been enabled in kernel-3.15.0-0.rc5.git2.8.fc21, which should show up in tomorrow's rawhide barring unexpected issues.
You can now use contructs like kmod(joydev.ko) to make sure the joystick driver is installed, instead of requiring kernel-modules-extra. This also makes packages proofed against modules moving between kernel-modules and kernel-modules-extra.

Comment 8 Josh Boyer 2014-05-14 18:27:58 UTC
Created attachment 895578 [details]
Patch to switch to using Requires on individual kernel modules

The attached patch switches xl2tpd from using a Requires on kernel-modules-extra to using a requires on the individual kernel modules now that bug 1058331 is in rawhide.

Please review and let me know if it's OK to commit and build.

Comment 9 Paul Wouters 2014-05-14 19:39:33 UTC
looks okay. go ahead or I'll do it as soon as this appears in rawhide and the fedora branches.

Comment 10 Josh Boyer 2014-05-15 12:53:16 UTC
Pushed and building.  Thanks for the review Paul.