Removable drives are not auto-mounted after screen lock/unlock

Bug #1949352 reported by István Váradi
28
This bug affects 4 people
Affects Status Importance Assigned to Milestone
gnome-shell-extension-dashtodock (Ubuntu)
Won't Fix
Medium
Marco Trevisan (Treviño)
gnome-shell-extension-ubuntu-dock (Ubuntu)
Fix Released
Medium
Marco Trevisan (Treviño)
Impish
Fix Released
Medium
Marco Trevisan (Treviño)

Bug Description

[ Impact ]

If I lock and then unlock the screen, and then connect a removable drive, it is not recognized by the GNOME shell. The kernel logs show the drive, I can mount it manually, but it is not mounted automatically. The logged errors are attached.

[ Test case ]

- Mount a device
- Lock the screen
- Unlock the screen
- The device should be re-mounted (and no errors in journal)

[ Regression Potential ]

Devices could be leaked if gc doesn't work as expected

---

I could fix the issue by modifying the source code as given in the attached patch.
---
ProblemType: Bug
ApportVersion: 2.20.11-0ubuntu71
Architecture: amd64
CasperMD5CheckResult: pass
CurrentDesktop: ubuntu:GNOME
DisplayManager: gdm3DistroRelease: Ubuntu 21.10
InstallationDate: Installed on 2021-11-01 (1 days ago)
InstallationMedia: Ubuntu 21.10 "Impish Indri" - Release amd64 (20211012)
Package: gnome-shell 40.5-1ubuntu2
PackageArchitecture: amd64
ProcVersionSignature: Ubuntu 5.13.0-20.20-generic 5.13.14
RelatedPackageVersions: mutter-common 40.5-1ubuntu2
Tags: wayland-session impish
Uname: Linux 5.13.0-20-generic x86_64
UpgradeStatus: No upgrade log present (probably fresh install)
UserGroups: adm cdrom dip lpadmin lxd plugdev sambashare sudo
_MarkForUpload: True

Revision history for this message
István Váradi (ivaradi) wrote :
Revision history for this message
István Váradi (ivaradi) wrote :
Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "The patch with my changes" seems to be a patch. If it isn't, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are a member of the ~ubuntu-reviewers, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issues please contact him.]

tags: added: patch
description: updated
Revision history for this message
Daniel van Vugt (vanvugt) wrote (last edit ):

Thank you for taking the time to report this bug and helping to make Ubuntu better. Please execute the following command only once, as it will automatically gather debugging information, in a terminal:

  apport-collect 1949352

When reporting bugs in the future please use apport by using 'ubuntu-bug' and the name of the package affected. You can learn more about this functionality at https://wiki.ubuntu.com/ReportingBugs.

Changed in gnome-shell (Ubuntu):
status: New → Incomplete
summary: - Removable drives are not handled after screen lock/unlock
+ Removable drives are not auto-mounted after screen lock/unlock
Revision history for this message
István Váradi (ivaradi) wrote : Dependencies.txt

apport information

tags: added: apport-collected impish wayland-session
description: updated
Revision history for this message
István Váradi (ivaradi) wrote : GsettingsChanges.txt

apport information

Revision history for this message
István Váradi (ivaradi) wrote : ProcCpuinfoMinimal.txt

apport information

Revision history for this message
István Váradi (ivaradi) wrote : ProcEnviron.txt

apport information

Revision history for this message
István Váradi (ivaradi) wrote : ShellJournal.txt

apport information

Revision history for this message
István Váradi (ivaradi) wrote : monitors.xml.txt

apport information

Revision history for this message
Daniel van Vugt (vanvugt) wrote : Re: [vmware] Removable drives are not auto-mounted after screen lock/unlock

If you find the patch in comment #2 helps then please propose it at https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests

Changed in gnome-shell (Ubuntu):
status: Incomplete → New
summary: - Removable drives are not auto-mounted after screen lock/unlock
+ [vmware] Removable drives are not auto-mounted after screen lock/unlock
tags: added: vmware
Revision history for this message
István Váradi (ivaradi) wrote :

Please note, that while I created the apport report in a VM (actually VirtualBox, no VMWare), the problem originally occurred on a native installation. So I am not sure the vmware tag is relevant.

Revision history for this message
István Váradi (ivaradi) wrote :
Revision history for this message
Daniel van Vugt (vanvugt) wrote :
affects: gnome-shell (Ubuntu) → gnome-shell-extension-ubuntu-dock (Ubuntu)
Changed in gnome-shell-extension-dashtodock (Ubuntu):
assignee: nobody → Marco Trevisan (Treviño) (3v1n0)
status: New → In Progress
summary: - [vmware] Removable drives are not auto-mounted after screen lock/unlock
+ Removable drives are not auto-mounted after screen lock/unlock
Changed in gnome-shell-extension-dashtodock (Ubuntu):
importance: Undecided → Medium
Changed in gnome-shell-extension-ubuntu-dock (Ubuntu):
importance: Undecided → Medium
status: New → Triaged
assignee: nobody → Marco Trevisan (Treviño) (3v1n0)
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (5.7 KiB)

This bug was fixed in the package gnome-shell-extension-ubuntu-dock - 72~ubuntu1

---------------
gnome-shell-extension-ubuntu-dock (72~ubuntu1) jammy; urgency=medium

  [ Marco Trevisan (Treviño) ]
  * Revert "extension: Disable starting in the overview"
  * locations: Do not manually dispose objects (LP: #1949352, LP: #1951599)
  * po/es.po: Remove duplicated entry for Quit translation
  * docking: Delay dash animation callback at actual completion
  * utils: Fix variable in error template string
  * appIcons: In case we have no window to activate, let's just present it
  * docking: Do not try to re-add a Meta.Later if one is already queued
  * docking: Optionally disable animation to overview on startup
  * dash: Track hover on separator item so that it can contribute to scrolling
  * dash: Remove the scroll view visibility idle if triggering actor is
    destroyed
  * docking: Handle monitoring of discrete GPU not to leak signals
  * appIcons: Random cleanups to use more relevant variable names
  * appIcons: Compare workspaces instead of their indexes
  * appIcons: Ensure we only consider as urgent the ones respecting our rules
  * appIcons: Use new-window action only when requested, avoiding duplication
  * appIcons: Fixed logic on handling preview click actions (LP: #1947445)
  * stylesheet: Use some better naming for start marging value
  * stylesheet: Use scss generation for overview-icon background
  * stylesheet: Set the dash-separator color in both horizontal and vertical
    mode
  * windowPreview: Try generating previews clones on meta later only if have
    valid size (LP: 1949572)
  * utils: Remove handlers in reversed order
  * Dash: Update appIcon geometry on parent geometry changes
  * docking: Reduce the space used by the window picker / app grid
  * cleanup: Use cached settings values instead of doing C calls
  * locations: Use shell domain to translate reused strings
  * locations: Use native GIcon as location Apps-icons (LP: #1874578)
  * locations: Use native Gio.FileMonitor rate limit instead of a custom idle
  * locations: Fix typo on FileManager app (un)wrapping functions
  * locations: Fix handling of location apps on isolated workspaces mode
  * locations: Also destroy all removable devices on Removables manager
    destruction
  * locations: Only emit changed signal if a monitored device has been removed
  * locations: Only update Trash icon on changes
  * locations: Remove unmounted locations by value
  * locations: Use native AppInfo's and Shell mount operations
  * metadata: Add support for gnome-shell 42
  * fileManager1API: Cleanup the code to get windows from location path
  * fileManager1API: Keep windows paths cached so we can monitor changes
  * utils: Add support for (un)blocking signals handlers
  * locations: Smart managing wrapped windows backed applications signal
    connections
  * locations: Use proxy properties to expose private items to public object
  * locations: Chain up to parent destroy() function if any
  * locations: Manage sources as part of DtdData
  * locations: Manage windows changes in base class
  * cleanup: Move shellAppCompare under Utils as it's a generic tool
  * locations: Keep...

Read more...

Changed in gnome-shell-extension-ubuntu-dock (Ubuntu):
status: Triaged → Fix Released
description: updated
Changed in gnome-shell-extension-dashtodock (Ubuntu Impish):
assignee: nobody → Marco Trevisan (Treviño) (3v1n0)
importance: Undecided → Medium
status: New → In Progress
Changed in gnome-shell-extension-ubuntu-dock (Ubuntu Impish):
assignee: nobody → Marco Trevisan (Treviño) (3v1n0)
importance: Undecided → Medium
no longer affects: gnome-shell-extension-dashtodock (Ubuntu Impish)
Changed in gnome-shell-extension-dashtodock (Ubuntu):
status: In Progress → Won't Fix
Changed in gnome-shell-extension-ubuntu-dock (Ubuntu Impish):
status: New → In Progress
Revision history for this message
Brian Murray (brian-murray) wrote : Please test proposed package

Hello István, or anyone else affected,

Accepted gnome-shell-extension-ubuntu-dock into impish-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/gnome-shell-extension-ubuntu-dock/70~ubuntu3.21.10.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, what testing has been performed on the package and change the tag from verification-needed-impish to verification-done-impish. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-impish. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Changed in gnome-shell-extension-ubuntu-dock (Ubuntu Impish):
status: In Progress → Fix Committed
tags: added: verification-needed verification-needed-impish
Revision history for this message
István Váradi (ivaradi) wrote :

Hello,

the proposed package works for me. I have tested it as follows after installing:

1. Logged out from and then logged in to the GNOME desktop.
2. Locked the screen and then unlocked it.
3. Inserted a pen drive, which got mounted (and the sound effect could be heard too).
4. No error messages related to the extension were present in the journal.

István

tags: added: verification-done-impish
removed: verification-needed-impish
Revision history for this message
Angel D. Segarra (angel-segarra) wrote :

Fixed here as well, tested same steps as #17. Just wanted to add tested version is 70~ubuntu3.21.10.1

Revision history for this message
Chris Halse Rogers (raof) wrote : Update Released

The verification of the Stable Release Update for gnome-shell-extension-ubuntu-dock has completed successfully and the package is now being released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package gnome-shell-extension-ubuntu-dock - 70~ubuntu3.21.10.1

---------------
gnome-shell-extension-ubuntu-dock (70~ubuntu3.21.10.1) impish; urgency=medium

  [ Marco Trevisan (Treviño) ]
  * locations: Do not manually dispose objects (LP: #1949352, #1951599)
  * windowPreview: Try generating previews clones on meta later only if have valid size
    (LP: #1949572)
  * locations: Use native GIcon as location Apps-icons (LP: #1874578)
  * debian: Update branch info for impish

  [ nename0 ]
  * Cleanup nautilus listeners (LP: #1949352)

  [ Pepijn Bogaard ]
  * appIcons: Fixed logic on handling preview click actions (LP: #1947445)

 -- Marco Trevisan (Treviño) <email address hidden> Tue, 25 Jan 2022 01:08:43 +0100

Changed in gnome-shell-extension-ubuntu-dock (Ubuntu Impish):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.