Bug 1720232

Summary: RFE: Add /afs to the base filesystem package
Product: [Fedora] Fedora Reporter: David Howells <dhowells>
Component: filesystemAssignee: Ondrej Vasik <ovasik>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: jaltman, marc.c.dionne, ovasik, pknirsch
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: filesystem-3.12-1.fc31 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2019-06-20 13:57:46 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:
Attachments:
Description Flags
Patch to add /afs to the filesystem base
none
Patch to add /afs to a filesystem subpackage none

Description David Howells 2019-06-13 12:57:34 UTC
Created attachment 1580266 [details]
Patch to add /afs to the filesystem base

Description of problem:

Add /afs to the base filesystem package.  It has been approved by the Fedora packaging committee (bug 888):

    #888 How to install "/afs" directory for kafs-client package
    (geppetto, 16:06:25)
    ACTION: Allow /afs to be included in filesystem, or afs-filesystem
    if desired, for rawhide/f31 (+1:6, 0:0, -1:0) (geppetto, 16:19:07)
    Can use the scriptlet dir. creation hack for f30, but not f31+
    (geppetto, 16:19:36)

This is needed to make it possible to do the upcoming kafs-client package correctly.  The problem is that the kafs-client package can't simply declare /afs as a directory because then package upgrading breaks as dnf doesn't seem to be able to upgrade a package to a newer version if there's a direct directory spec.

Instead, various afs clients (including OpenAFS, for example) try to create the /afs directory from a post install script with mkdir - which is really the wrong solution.

I've included a patch to make the change.

Comment 1 David Howells 2019-06-13 12:59:51 UTC
Created attachment 1580267 [details]
Patch to add /afs to a filesystem subpackage

Here's an alternative patch that creates a subpackage (filesystem-afs) that just adds the /afs directory.  I would prefer the initial patch.

Comment 2 Ondrej Vasik 2019-06-20 13:57:46 UTC
I prefer the afs subpackage ... as I don't think this is something that needs to be installed by default for all systems. AFS clients should require this subpackage. Pushed the commit and built rawhide.