From patchwork Fri Mar 31 21:56:53 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Scott Mayhew X-Patchwork-Id: 9657311 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 62A5E602BD for ; Fri, 31 Mar 2017 21:57:00 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 49C612817F for ; Fri, 31 Mar 2017 21:57:00 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 3E5A02850F; Fri, 31 Mar 2017 21:57:00 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.9 required=2.0 tests=BAYES_00,RCVD_IN_DNSWL_HI autolearn=ham version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id AC56F2817F for ; Fri, 31 Mar 2017 21:56:59 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755453AbdCaV44 (ORCPT ); Fri, 31 Mar 2017 17:56:56 -0400 Received: from mx1.redhat.com ([209.132.183.28]:45194 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755388AbdCaV44 (ORCPT ); Fri, 31 Mar 2017 17:56:56 -0400 Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.13]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 818D42FB90F; Fri, 31 Mar 2017 21:56:55 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mx1.redhat.com 818D42FB90F Authentication-Results: ext-mx04.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com Authentication-Results: ext-mx04.extmail.prod.ext.phx2.redhat.com; spf=pass smtp.mailfrom=smayhew@redhat.com DKIM-Filter: OpenDKIM Filter v2.11.0 mx1.redhat.com 818D42FB90F Received: from coeurl.usersys.redhat.com (ovpn-121-127.rdu2.redhat.com [10.10.121.127]) by smtp.corp.redhat.com (Postfix) with ESMTP id 455B584730; Fri, 31 Mar 2017 21:56:55 +0000 (UTC) Received: by coeurl.usersys.redhat.com (Postfix, from userid 1000) id D7B6A205C0; Fri, 31 Mar 2017 17:56:54 -0400 (EDT) From: Scott Mayhew To: steved@redhat.com Cc: neilb@suse.com, linux-nfs@vger.kernel.org Subject: [RFC nfs-utils PATCH 1/2] idmapd: move the pipefs-directory config option to nfs.conf Date: Fri, 31 Mar 2017 17:56:53 -0400 Message-Id: <20170331215654.31570-2-smayhew@redhat.com> In-Reply-To: <20170331215654.31570-1-smayhew@redhat.com> References: <20170331215654.31570-1-smayhew@redhat.com> X-Scanned-By: MIMEDefang 2.79 on 10.5.11.13 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.28]); Fri, 31 Mar 2017 21:56:55 +0000 (UTC) Sender: linux-nfs-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-nfs@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP Changed idmapd to read its value for the pipefs-directory from /etc/nfs.conf rather than /etc/idmapd.conf. All other configurations related to id mapping still reside in /etc/idmapd.conf for now. Removed the -c option, since it would be confusing as whether it should override nfs.conf, idmapd.conf, or both. Signed-off-by: Scott Mayhew --- nfs.conf | 3 +++ systemd/nfs.conf.man | 9 +++++++++ utils/idmapd/idmapd.c | 35 ++++++++++++++--------------------- utils/idmapd/idmapd.man | 19 ++++++++++++++++++- 4 files changed, 44 insertions(+), 22 deletions(-) diff --git a/nfs.conf b/nfs.conf index 81ece06..4359904 100644 --- a/nfs.conf +++ b/nfs.conf @@ -17,6 +17,9 @@ # cred-cache-directory= # preferred-realm= # +#[idmapd] +# pipefs-directory=/var/lib/nfs/rpc_pipefs +# #[lockd] # port=0 # udp-port=0 diff --git a/systemd/nfs.conf.man b/systemd/nfs.conf.man index bdc0988..83cf84a 100644 --- a/systemd/nfs.conf.man +++ b/systemd/nfs.conf.man @@ -215,6 +215,15 @@ See for details. .TP +.B idmapd +Recognized values: +.BR pipefs-directory . + +See +.BR rpc.idmapd (8) +for details. + +.TP .B svcgssd Recognized values: .BR principal . diff --git a/utils/idmapd/idmapd.c b/utils/idmapd/idmapd.c index f4e083a..561a4b8 100644 --- a/utils/idmapd/idmapd.c +++ b/utils/idmapd/idmapd.c @@ -214,13 +214,11 @@ main(int argc, char **argv) struct event initialize; struct passwd *pw; struct group *gr; - struct stat sb; char *xpipefsdir = NULL; int serverstart = 1, clientstart = 1; int ret; char *progname; - conf_path = _PATH_IDMAPDCONF; nobodyuser = NFS4NOBODY_USER; nobodygroup = NFS4NOBODY_GROUP; strlcpy(pipefsdir, PIPEFS_DIR, sizeof(pipefsdir)); @@ -231,11 +229,9 @@ main(int argc, char **argv) progname = argv[0]; xlog_open(progname); -#define GETOPTSTR "hvfd:p:U:G:c:CS" +#define GETOPTSTR "hvfd:p:U:G:CS" opterr=0; /* Turn off error messages */ while ((opt = getopt(argc, argv, GETOPTSTR)) != -1) { - if (opt == 'c') - conf_path = optarg; if (opt == '?') { if (strchr(GETOPTSTR, optopt)) warnx("'-%c' option requires an argument.", optopt); @@ -247,20 +243,19 @@ main(int argc, char **argv) } optind = 1; - if (stat(conf_path, &sb) == -1 && (errno == ENOENT || errno == EACCES)) { - warn("Skipping configuration file \"%s\"", conf_path); - conf_path = NULL; - } else { - conf_init(); - verbose = conf_get_num("General", "Verbosity", 0); - cache_entry_expiration = conf_get_num("General", - "Cache-Expiration", DEFAULT_IDMAP_CACHE_EXPIRY); - CONF_SAVE(xpipefsdir, conf_get_str("General", "Pipefs-Directory")); - if (xpipefsdir != NULL) - strlcpy(pipefsdir, xpipefsdir, sizeof(pipefsdir)); - CONF_SAVE(nobodyuser, conf_get_str("Mapping", "Nobody-User")); - CONF_SAVE(nobodygroup, conf_get_str("Mapping", "Nobody-Group")); - } + conf_path = NFS_CONFFILE; + conf_init(); + CONF_SAVE(xpipefsdir, conf_get_str("idmapd", "pipefs-directory")); + if (xpipefsdir != NULL) + strlcpy(pipefsdir, xpipefsdir, sizeof(pipefsdir)); + + conf_path = _PATH_IDMAPDCONF; + conf_init(); + verbose = conf_get_num("General", "Verbosity", 0); + cache_entry_expiration = conf_get_num("General", + "cache-expiration", DEFAULT_IDMAP_CACHE_EXPIRY); + CONF_SAVE(nobodyuser, conf_get_str("Mapping", "Nobody-User")); + CONF_SAVE(nobodygroup, conf_get_str("Mapping", "Nobody-Group")); while ((opt = getopt(argc, argv, GETOPTSTR)) != -1) switch (opt) { @@ -307,8 +302,6 @@ main(int argc, char **argv) #ifdef HAVE_NFS4_SET_DEBUG nfs4_set_debug(verbose, xlog_warn); #endif - if (conf_path == NULL) - conf_path = _PATH_IDMAPDCONF; if (nfs4_init_name_mapping(conf_path)) errx(1, "Unable to create name to user id mappings."); diff --git a/utils/idmapd/idmapd.man b/utils/idmapd/idmapd.man index d4ab894..0fbc24c 100644 --- a/utils/idmapd/idmapd.man +++ b/utils/idmapd/idmapd.man @@ -78,6 +78,21 @@ Client-only: perform no idmapping for any NFS server, even if one is detected. .It Fl S Server-only: perform no idmapping for any NFS client, even if one is detected. .El +.Sh CONFIGURATION FILES +The +.Sy [idmapd] +section of the +.Pa /etc/nfs.conf +configuration file recognizes the following value: +.Bl -tag -width Ds_imagedir +.It Sy pipefs-directory +Equivalent to +.Sy -p . +.El +.Pp +All other settings related to id mapping are found in the +.Pa /etc/idmapd.conf +configuration file. .Sh EXAMPLES .Cm rpc.idmapd -f -vvv .Pp @@ -94,9 +109,11 @@ messages to console, and with a verbosity level of 3. .\" This next request is for sections 1, 6, 7 & 8 only. .\" .Sh ENVIRONMENT .Sh FILES -.Pa /etc/idmapd.conf +.Pa /etc/idmapd.conf , +.Pa /etc/nfs.conf .Sh SEE ALSO .Xr idmapd.conf 5 , +.Xr nfs.conf 5 , .Xr nfsidmap 8 .\".Sh SEE ALSO .\".Xr nylon.conf 4