public inbox for git-commits@fedoraproject.org
help / color / mirror / Atom feed
* [rpms/nfs-utils] rawhide: Pass ignore_hosts to export_create() in export_read()
@ 2026-07-15 11:09 Steve Dickson
  0 siblings, 0 replies; only message in thread
From: Steve Dickson @ 2026-07-15 11:09 UTC (permalink / raw)
  To: git-commits

            A new commit has been pushed.

            Repo   : rpms/nfs-utils
            Branch : rawhide
            Commit : 987a3ace7630b39d81a84a2ed538d32583e520f3
            Author : Steve Dickson <steved@redhat.com>
            Date   : 2026-07-15T07:01:41-04:00
            Stats  : +43/-1 in 2 file(s)
            URL    : https://src.fedoraproject.org/rpms/nfs-utils/c/987a3ace7630b39d81a84a2ed538d32583e520f3?branch=rawhide

            Log:
            Pass ignore_hosts to export_create() in export_read()

Signed-off-by: Steve Dickson <steved@redhat.com>

---
diff --git a/nfs-utils-2.9.1-export-ignore-hosts.patch b/nfs-utils-2.9.1-export-ignore-hosts.patch
new file mode 100644
index 0000000..6b77ea2
--- /dev/null
+++ b/nfs-utils-2.9.1-export-ignore-hosts.patch
@@ -0,0 +1,38 @@
+Pass ignore_hosts to export_create() in export_read()
+
+Commit 8f3d12ce ("nfs-server-generator: avoid using external services.")
+added the 'ignore_hosts' flag to export_read() so nfs-server-generator
+can bypass DNS queries when calling it.  If the export doesn't
+already exist, export_read() calls export_create() with the 'canonical'
+argument hard-coded to 0, triggering a DNS query in client_lookup().
+An unresponsive DNS server can cause delays in nfs-server-generator and
+can even lead to 'systemctl daemon-reload' timing out, leading to
+further administrative issues.
+
+nfs-server-generator only cares about *what* is exported, so it can
+create the order-with-mounts.conf config drop-in.  It doesn't need to
+know *who* those filesystems are exported to, so it has no need to
+perform DNS queries.
+
+Pass the 'ignore_hosts' flag from export_read() to export_create() to
+avoid the unnecessary DNS queries.
+
+Fixes: 8f3d12ce ("nfs-server-generator: avoid using external services.")
+Signed-off-by: Scott Mayhew <smayhew@redhat.com>
+---
+ support/export/export.c | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/support/export/export.c b/support/export/export.c
+index 2c8c3335..3caee043 100644
+--- a/support/export/export.c
++++ b/support/export/export.c
+@@ -122,7 +122,7 @@ export_read(char *fname, int ignore_hosts)
+ 	while ((eep = getexportent(0)) != NULL) {
+ 		exp = export_lookup(eep->e_hostname, eep->e_path, ignore_hosts);
+ 		if (!exp) {
+-			if (export_create(eep, 0))
++			if (export_create(eep, ignore_hosts))
+ 				/* possible complaints already logged */
+ 				volumes++;
+ 		}

diff --git a/nfs-utils.spec b/nfs-utils.spec
index d078cfa..4a7b73e 100644
--- a/nfs-utils.spec
+++ b/nfs-utils.spec
@@ -2,7 +2,7 @@ Summary: NFS utilities and supporting clients and daemons for the kernel NFS ser
 Name: nfs-utils
 URL: http://linux-nfs.org/
 Version: 2.9.1
-Release: 4.rc5%{?dist}
+Release: 5.rc5%{?dist}
 Epoch: 1
 
 # group all 32bit related archs
@@ -16,6 +16,7 @@ Source4: 10-nfsv4.conf
 Source5: 10-nfsv3.conf
 
 Patch001: nfs-utils-2.9.2-rc5.patch
+Patch002: nfs-utils-2.9.1-export-ignore-hosts.patch
 
 Patch100: nfs-utils-1.2.1-statdpath-man.patch
 Patch102: nfs-utils-1.2.5-idmap-errmsg.patch
@@ -476,6 +477,9 @@ rm -f %{_sysconfdir}/nfsmount.conf.d/10-nfsv4.conf
 %{_mandir}/*/rpcctl.8.gz
 
 %changelog
+* Wed Jul 15 2026 Steve Dickson <steved@redhat.com> 2.9.1-5-rc5
+- Pass ignore_hosts to export_create() in export_read()
+
 * Sat Jul 11 2026 Steve Dickson <steved@redhat.com> 2.9.1-4-rc5
 - Updated to the latest RC release: nfs-utils-2-9-2-rc5
 

^ permalink raw reply related	[flat|nested] only message in thread

only message in thread, other threads:[~2026-07-15 11:09 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-07-15 11:09 [rpms/nfs-utils] rawhide: Pass ignore_hosts to export_create() in export_read() Steve Dickson

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox