Re: pkg/55544: sysutils/ansible: use /proc/mounts

Feel free to provide a better solution.

I have a) no skills in programming Python and b) no knowledge of the inner workings of ansible (or the desire to learn them). The current code is 3 years old, and already uses /proc/cpuinfo and /proc/meminfo, so this is just a normalization of what is already there.

On Wed, Aug 5, 2020 at 10:45 PM Joerg Sonnenberger <joerg%bec.de@localhost> wrote:

The following reply was made to PR pkg/55544; it has been noted by GNATS.

From: Joerg Sonnenberger <joerg%bec.de@localhost>
To: gnats-bugs%netbsd.org@localhost
Cc:
Subject: Re: pkg/55544: sysutils/ansible: use /proc/mounts
Date: Wed, 5 Aug 2020 22:40:01 +0200

On Wed, Aug 05, 2020 at 07:35:00PM +0000, joernc%gmail.com@localhost wrote:
> >Number: 55544
> >Category: pkg
> >Synopsis: sysutils/ansible: use /proc/mounts
> >Confidential: no
> >Severity: serious
> >Priority: medium
> >Responsible: pkg-manager
> >State: open
> >Class: sw-bug
> >Submitter-Id: net
> >Arrival-Date: Wed Aug 05 19:35:00 +0000 2020
> >Originator: Joern Clausen
> >Release:
> >Organization:
> >Environment:
> >Description:
> The fact gathering code for NetBSD already uses /proc in some cases, but reads /etc/fstab to find mounted filesystems. This misses usually (well, at least in my case) ZFS mounts, which are no longer documented there.
>
> >How-To-Repeat:
> $ ansible foo -m setup
>
> will not report mounts on foo except those documented in /etc/fstab
> >Fix:
> https://github.com/ansible/ansible/pull/71113 is a pull request to read /proc/mounts instead. Until (if ever) it makes it to a release, a local patch in pkgsrc would be nice.

What about using mount(8) instead? Not every system has /proc mounted...

Joerg