Request 670012 (superseded)

This request is superseded by request 719734 (Show diff)

Overview

Request 670012 superseded

- remove libnfsidmap1 dependency for nfs-client.
It isn't needed.

- Remove service aliases - these files are deleted.
nfs.service
nfsserver.service
nfs-server.nfsserver.conf
nfs-client.nfs.conf
Now the upstream standard service names "nfs-client" and "nfs-server"
must be used.

- 0004-nfsidmap-honour-with-pluginpath-for-instalation.patch
Allow plugins to be installed properly
- nfs-utils.spec
Package shared library correctly

- 0001-nfs.conf-allow-empty-assignments.patch
Fix regression due to unnecessary "error" messages from nfs.conf
- 0002-Let-systemd-know-when-rpc.statd-is-needed.patch
0003-systemd-run-statd-notify-even-when-nfs-client-isn-t-.patch
Fixes for systemd integration
(bsc#1116221)

- New version: nfs-utils-2.3.3
The nfsidmap library source has been merged into
nfs-utils, so this source package now makes
nfsidmap and nfsidmap-devel packages.
New program "nfsconf" improves access to nfs config files.
Delete patches that have been included upstream:
0001-conffile-ignore-empty-environment-variables.patch

Created by neilbrown about 5 years ago
In state superseded
Supersedes 670010
Superseded by 719734
Open review for openSUSE:Factory:Staging:M

Submit nfs-utils

Comments for request 670012 15
Comments for request 670010 1

Stephan Kulow (coolo) - about 5 years ago

found conflict of nfs-client-2.3.3-245.1.x86_64 with nfsidmap-0.27-4.3.x86_64:
  - /usr/share/man/man5/idmapd.conf.5.gz
found conflict of nfsidmap-0.27-4.3.x86_64 with nfsidmap-devel-1.0-245.1.x86_64:
  - /usr/share/man/man3/nfs4_uid_to_name.3.gz

Dominique Leuenberger (dimstar) - about 5 years ago

reviewer

I see what's going on here:

nfsidmap (the one in conflict) is being obsoleted by libnfsidmap1 - and the man pages are moved to nfs-client and nfsidmap-devel.

For the devel, we can argue that the obsoletes in the library would be sufficient (you can't install the devel without upgrading the library package), for the -client it is similar, as we have a dependency on libnfsidmap.so.1, which is provided by libnfsidmap1 (and thus obsoleting nfsidmap)

In any case, asking for a nfsidmap change to accommodate this seems the wrong way.

@msmeissn: this calls for a removal of nfsidmap from openSUSE:Factory. Is this in line with your expectations?

Stephan Kulow (coolo) - about 5 years ago

needs nfsidmap which is aligned

Stephan Kulow (coolo) - about 5 years ago

any news here?

Dominique Leuenberger (dimstar_suse) - about 5 years ago

reviewer target maintainer

Unignored: returned to active backlog.

Dominique Leuenberger (dimstar) - about 5 years ago

reviewer

https://openqa.opensuse.org/tests/856655#step/yast2_nfs_server/18

Stephan Kulow (coolo) - about 5 years ago

the test is gone - make sure the openqa job is labeled as linked

Yuchen Lin (maxlin_factory) - about 5 years ago

reviewer

not ready

Stephan Kulow (coolo) - about 5 years ago

but what is missing?

Dominique Leuenberger (dimstar) - almost 5 years ago

reviewer

Finally we got a new test run - and the issue is thus visible again:

https://openqa.opensuse.org/tests/949489#step/yast2_nfs_server/6

Dominique Leuenberger (dimstar) - almost 5 years ago

reviewer

soo, in fact, this is just because we lost the nfsserver -> nfs-server alias in the systemd services, which is intentional

+- Remove service aliases - these files are deleted.

The only worry I have for this is that if a user had 'nfsserver' enabled (but not nfs-server, which worked, as nfs-server was implicitly started by nfsserver, even when disabled) - after the upgrade to this package, the nfs-server will stay disabled, thus transparently 'disabling' the service (which could cause system disruptions)

Dominique Leuenberger (dimstar) - almost 5 years ago

reviewer

https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/7615

Dominique Leuenberger (dimstar) - almost 5 years ago

reviewer

Now openQA gets beyond starting the daemon - which is good. But with this vresion of nfs-utils we fail to actually mount the share:

https://openqa.opensuse.org/tests/950074#step/yast2_nfs_server/59

the mount command is timing out

Neil Brown (neilbrown) - almost 5 years ago

author source maintainer

Can you please change the test to remove the "fsid=0" from /etc/exports? That is very old syntax that isn't needed any more and might be causing the problem.

Dominique Leuenberger (dimstar) - almost 5 years ago

reviewer

That did not make a difference:

https://openqa.opensuse.org/tests/985504#step/yast2_nfs_server/59