Ticket #246 (closed defect: fixed)

Opened 4 years ago

Last modified 3 years ago

Shutdown/Reboot fails when multiple users are logged in

Reported by: tomeu Owned by: alsroot
Priority: Normal Milestone: 0.86
Component: sugar Version: 0.84.x
Severity: Critical Keywords: r+
Cc: alsroot Distribution/OS: Unspecified
Bug Status: Assigned

Description

If you try to shutdown but it doesn't because concole sessions were open, trying to shutdown after closing them won't work because something breaks in our session management

Attachments

sugar-246.patch Download (2.0 KB) - added by alsroot 4 years ago.
sugar-246-sugar.patch Download (1.8 KB) - added by alsroot 4 years ago.

Change History

Changed 4 years ago by marcopg

  • priority changed from normal to Normal
  • distribution set to Unspecified
  • severity set to Minor
  • status_field set to New

Changed 4 years ago by erikos

Maybe the same then #427?

Changed 4 years ago by tomeu

  • severity changed from Minor to Blocker

Changed 4 years ago by erikos

See as well #427

Changed 4 years ago by erikos

  • owner changed from marcopg to erikos
  • status changed from new to assigned
  • summary changed from shutdown operation fails after first fail to Shutdown fails when multiple users are logged in
  • status_field changed from New to Assigned

We hit the policy for hal.power.management defined in /usr/share/PolicyKit/policy/org.freedesktop.hal.power-management.policy

 <action id="org.freedesktop.hal.power-management.shutdown-multiple-sessions">
    <description>Shut down the system when multiple users are logged in</description>
    <message>System policy prevents shutting down the system while multiple users are logged in</message>
    <defaults>
      <allow_inactive>no</allow_inactive>
      <allow_active>auth_admin</allow_active>
    </defaults>
  </action>

Changed 4 years ago by erikos

  • summary changed from Shutdown fails when multiple users are logged in to Shutdown/Reboot fails when multiple users are logged in

Changed 4 years ago by alsroot

  • cc alsroot added

Changed 4 years ago by erikos

  • milestone changed from 0.84 to 0.86

Still present - let's move it.

Changed 4 years ago by erikos

  • keywords 0.84 added

Changed 4 years ago by erikos

  • keywords 0.84 removed
  • version changed from Git as of bugdate to 0.84.x
  • severity changed from Blocker to Critical

Changed 4 years ago by erikos

Serious, but could go into 0.86.1, too.

Changed 4 years ago by alsroot

Changed 4 years ago by alsroot

Changed 4 years ago by alsroot

  • keywords r? added

Attached patches just handle errors while rebooting/shutdowning gracefully.

We also can notify user in UI about error but dunno how it could look(since we have only desktop window).

Changed 4 years ago by tomeu

  • keywords r+ added; r? removed
  • owner changed from erikos to alsroot

Changed 4 years ago by alsroot

  • status changed from assigned to closed
  • resolution set to fixed

 http://git.sugarlabs.org/projects/sugar/repos/mainline/commits/e96ab69ad05e60323b0d71cfb9975982afc7135d
 http://git.sugarlabs.org/projects/sugar-toolkit/repos/mainline/commits/3d46fd8c99aeea34fce971f14129432ea52e8e12

Note: See TracTickets for help on using tickets.