[frameworks-solid] [Bug 406242] fstab-mounted NFS drive gets two entries in Places panel

2019-12-14 Thread Méven Car
https://bugs.kde.org/show_bug.cgi?id=406242

Méven Car  changed:

   What|Removed |Added

 Status|CONFIRMED   |RESOLVED
   Version Fixed In||5.66
 Resolution|--- |FIXED
  Latest Commit||https://commits.kde.org/sol
   ||id/c97f0b2a3076731b35435f20
   ||0bd09a22859f3e03

--- Comment #8 from Méven Car  ---
Git commit c97f0b2a3076731b35435f200bd09a22859f3e03 by Méven Car.
Committed on 15/12/2019 at 07:27.
Pushed by meven into branch 'master'.

Ensure mounted nfs filesystems matches their fstab declared counterpart

Summary:
When a nfs fs is declared in /etc/fstab and the filesystem path ends with a /
(like 192.168.1.16:/home/meven/ ), once mounted the detected filesystem path
does not have the slash.
This causes the mounted drive not to match the umounted filesystem, causing the
later bug.

I.e :

$ solid-hardware list
[...]
udi = '/org/kde/fstab/192.168.1.16:/home/meven/'  (fstab declared mount)
udi = '/org/kde/fstab/192.168.1.16:/home/meven'(mounted drive as returned
by getmntent)

The patch makes the logic in Solid::Backends::Fstab::FstabHandling::deviceList
matching mounted and unmounted filesytems unsensitive to their path ending with
/
Related: bug 390691
FIXED-IN: 5.66

Test Plan:
Locally tested with Linux with a nfs drive set up with a path ending with a
slash like
192.168.1.16:/home/meven/ /media/NFS nfs defaults,user,auto,noatime,bg 0 0

After patch
$ solid-hardware list
[...]
udi = '/org/kde/fstab/192.168.1.16:/home/meven'(mounted drive as returned
by getmntent)

Only one icon appears for the drive in places panel in Dolphin.

Not tested with getmntinfo / BSD code path, but the patch does not alter the
code behavior much and don't foresee any risk there.

Reviewers: bruns, #frameworks, ngraham

Reviewed By: bruns, ngraham

Subscribers: broulik, dhaumann, anthonyfieroni, ngraham, bruns, apol,
kde-frameworks-devel

Tags: #frameworks

Differential Revision: https://phabricator.kde.org/D21204

M  +26   -3src/solid/devices/backends/fstab/fstabhandling.cpp

https://commits.kde.org/solid/c97f0b2a3076731b35435f200bd09a22859f3e03

-- 
You are receiving this mail because:
You are watching all bug changes.

[frameworks-solid] [Bug 406242] fstab-mounted NFS drive gets two entries in Places panel

2019-05-25 Thread Méven Car
https://bugs.kde.org/show_bug.cgi?id=406242

--- Comment #7 from Méven Car  ---
It seems to be due to specific configuration:

In /etc/fstab
Replacing:
192.168.1.16:/home/meven/ /media/NFS nfs defaults,user,auto,noatime,bg 0 0
With
192.168.1.16:/home/meven /media/NFS nfs defaults,user,auto,noatime,bg 0 0

For instance resolves the issue.

-- 
You are receiving this mail because:
You are watching all bug changes.

[frameworks-solid] [Bug 406242] fstab-mounted NFS drive gets two entries in Places panel

2019-05-11 Thread Méven Car
https://bugs.kde.org/show_bug.cgi?id=406242

Méven Car  changed:

   What|Removed |Added

 Resolution|WAITINGFORINFO  |---
 Status|NEEDSINFO   |CONFIRMED

--- Comment #6 from Méven Car  ---
$ cat /etc/fstab
192.168.1.16:/home/meven/ /media/NFS nfs defaults,user,auto,noatime,bg 0 0

$ cat /proc/mounts
192.168.1.16:/home/meven /media/NFS nfs4
rw,nosuid,nodev,noexec,noatime,vers=4.2,rsize=131072,wsize=131072,namlen=255,hard,proto=tcp,timeo=600,retrans=2,sec=sys,clientaddr=192.168.1.17,local_lock=none,addr=192.168.1.16
0 0

And to reiterate :
$ solid-hardware list
[...]
udi = '/org/kde/fstab/192.168.1.16:/home/meven/'
udi = '/org/kde/fstab/192.168.1.16:/home/meven'

Based on reading the code I deduced that
Solid::Backends::Fstab::FstabHandling::_k_updateMtabMountPointsCache does not
match the device because they end up with different udi.

Reviewing my settings I noticed that my remote export folder ends with a slash.
And indeed once I removed the slash in my config the issue does not occur
anymore.
So my previous patch seems irrelevant and what we need would be to strip last
slashes out of path read from /etc/fstab.
Does it make sense ?

-- 
You are receiving this mail because:
You are watching all bug changes.

[frameworks-solid] [Bug 406242] fstab-mounted NFS drive gets two entries in Places panel

2019-05-10 Thread Bug Janitor Service
https://bugs.kde.org/show_bug.cgi?id=406242

--- Comment #5 from Bug Janitor Service  ---
Dear Bug Submitter,

This bug has been in NEEDSINFO status with no change for at least
15 days. Please provide the requested information as soon as
possible and set the bug status as REPORTED. Due to regular bug
tracker maintenance, if the bug is still in NEEDSINFO status with
no change in 30 days the bug will be closed as RESOLVED > WORKSFORME
due to lack of needed information.

For more information about our bug triaging procedures please read the
wiki located here:
https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging

If you have already provided the requested information, please
mark the bug as REPORTED so that the KDE team knows that the bug is
ready to be confirmed.

Thank you for helping us make KDE software even better for everyone!

-- 
You are receiving this mail because:
You are watching all bug changes.

[frameworks-solid] [Bug 406242] fstab-mounted NFS drive gets two entries in Places panel

2019-04-26 Thread Stefan Brüns
https://bugs.kde.org/show_bug.cgi?id=406242

Stefan Brüns  changed:

   What|Removed |Added

 Status|CONFIRMED   |NEEDSINFO
 CC||stefan.bruens@rwth-aachen.d
   ||e
 Resolution|--- |WAITINGFORINFO

--- Comment #4 from Stefan Brüns  ---
@meven: can you provide the relevant lines from both /proc/mounts and
/etc/fstab?

-- 
You are receiving this mail because:
You are watching all bug changes.

[frameworks-solid] [Bug 406242] fstab-mounted NFS drive gets two entries in Places panel

2019-04-07 Thread Méven Car
https://bugs.kde.org/show_bug.cgi?id=406242

--- Comment #3 from Méven Car  ---
Already on it : https://phabricator.kde.org/D20301

-- 
You are receiving this mail because:
You are watching all bug changes.

[frameworks-solid] [Bug 406242] fstab-mounted NFS drive gets two entries in Places panel

2019-04-06 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=406242

Nate Graham  changed:

   What|Removed |Added

 Ever confirmed|0   |1
 Status|REPORTED|CONFIRMED

--- Comment #2 from Nate Graham  ---
Looks like it. Wanna submit a patch? :)

-- 
You are receiving this mail because:
You are watching all bug changes.

[frameworks-solid] [Bug 406242] fstab-mounted NFS drive gets two entries in Places panel

2019-04-06 Thread Méven Car
https://bugs.kde.org/show_bug.cgi?id=406242

--- Comment #1 from Méven Car  ---
Before mounting:

$ solid-hardware list
[...]
udi = '/org/kde/fstab/192.168.1.16:/home/meven/'

Mount:

$ solid-hardware mount '/org/kde/fstab/192.168.1.16:/home/meven/'

List again:

$ solid-hardware list
[...]
udi = '/org/kde/fstab/192.168.1.16:/home/meven/'
udi = '/org/kde/fstab/192.168.1.16:/home/meven'

It might illustrate the bug : the mounted device has a different udi with no
'/' at the end preventing solid to deduplicate them.

-- 
You are receiving this mail because:
You are watching all bug changes.

[frameworks-solid] [Bug 406242] fstab-mounted NFS drive gets two entries in Places panel

2019-04-05 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=406242

Nate Graham  changed:

   What|Removed |Added

  Component|panels: places  |general
Summary|Two nfs icon in network |fstab-mounted NFS drive
   |section |gets two entries in Places
   ||panel
Product|dolphin |frameworks-solid
 CC||kdelibs-b...@kde.org,
   ||n...@kde.org
   Assignee|dolphin-bugs-n...@kde.org   |lu...@kde.org
Version|18.04.3 |5.56.0

-- 
You are receiving this mail because:
You are watching all bug changes.