Need to allow for both Unity 8 desktop "bare metal" and LXC sessions

Bug #1359332 reported by Christopher Townsend
14
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Light Display Manager
Fix Released
Medium
Robert Ancell
1.10
Fix Released
Medium
Robert Ancell
Unity8 Desktop Preview in LXC
Fix Released
Medium
Christopher Townsend
unity8-desktop-session
Fix Released
Medium
Christopher Townsend
Trusty
Fix Released
Medium
Christopher Townsend
lightdm (Ubuntu)
Fix Released
Medium
Unassigned
Trusty
Fix Released
Undecided
Unassigned
Utopic
Fix Released
Medium
Unassigned
unity8-desktop-session (Ubuntu)
Fix Released
Medium
Christopher Townsend
Trusty
Fix Released
Undecided
Unassigned
Utopic
Fix Released
Medium
Christopher Townsend

Bug Description

[Impact]
Users trying to use the Unity8 desktop preview in a Linux Container cannot log in using LightDM. They are presented a black screen with a cursor.

[Test Case]
1. Install the Unity8 desktop preview in LXC package (currently in a PPA here: https://code.launchpad.net/~unity8-desktop-session-team/+archive/ubuntu/unity8-preview-lxc) alongside the unity8-desktop-session-mir package in Universe. Setup the Unity8 desktop preview in LXC as shown in https://wiki.ubuntu.com/Unity8inLXC#Setting_up_the_Unity_8_Desktop_Preview_LXC
2. Log in to the "Unity8 Mir in LXC" session from LightDM.

The Unity8 desktop preview will load.

[Regression potential]
Non observed.

Original Description:

It would be desirable to allow both the Unity 8 desktop preview session from the archive and the Unity 8 desktop preview in the LXC to be installed at the same time.

Currently, the Unity 8 desktop preview in LXC conflicts the "bare metal" Unity 8 desktop preview session.

Related branches

Changed in unity8-desktop-session:
status: New → In Progress
Changed in unity8-desktop-session (Ubuntu):
status: New → In Progress
Changed in unity8-preview-lxc:
status: New → In Progress
importance: Undecided → Medium
Changed in unity8-desktop-session:
importance: Undecided → Medium
Changed in unity8-desktop-session (Ubuntu):
importance: Undecided → Medium
Changed in unity8-preview-lxc:
assignee: nobody → Christopher Townsend (townsend)
Changed in unity8-desktop-session:
assignee: nobody → Christopher Townsend (townsend)
Changed in unity8-desktop-session (Ubuntu):
assignee: nobody → Christopher Townsend (townsend)
no longer affects: unity8-preview-lxc
Changed in unity8-preview-lxc:
status: New → In Progress
importance: Undecided → Medium
assignee: nobody → Christopher Townsend (townsend)
Changed in lightdm (Ubuntu):
status: New → Triaged
Changed in lightdm:
status: New → Triaged
status: Triaged → Fix Committed
importance: Undecided → Medium
Changed in lightdm (Ubuntu):
importance: Undecided → Medium
Changed in lightdm:
milestone: none → 1.11.6
Changed in lightdm:
status: Fix Committed → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package lightdm - 1.11.6-0ubuntu1

---------------
lightdm (1.11.6-0ubuntu1) utopic; urgency=medium

  * New upstream release:
    * Add a seat option 'allow-user-switching' that can disable all user
      switching for that seat
    * Add a new session type 'mir-container' that allows the session to run
      inside a custom system compositor (LP: #1359332)
    * Only seat0 takes VT from Plymouth
    * Removed unused GAsyncResultIface.is_tagged as this attribute was not
      available in earlier versions of Glib I/O.
    * Abort autogen if yelp-tools not installed
    * Return correct errors for D-Bus calls
 -- Robert Ancell <email address hidden> Fri, 22 Aug 2014 09:10:29 +1200

Changed in lightdm (Ubuntu):
status: Triaged → Fix Released
Changed in unity8-preview-lxc:
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package unity8-desktop-session - 1.0.12+14.10.20140829-0ubuntu1

---------------
unity8-desktop-session (1.0.12+14.10.20140829-0ubuntu1) utopic; urgency=low

  [ Christopher Townsend ]
  * Add check to see if the Unity8 LXC session is not being started
    before calling the init script. (LP: #1359332)

  [ Stephen M. Webb ]
  * remove unity8-desktop-session-x11 because it's unsupported (LP:
    #1296733)
 -- Ubuntu daily release <email address hidden> Fri, 29 Aug 2014 18:19:53 +0000

Changed in unity8-desktop-session (Ubuntu):
status: In Progress → Fix Released
Changed in unity8-desktop-session:
status: In Progress → Fix Committed
no longer affects: unity8-desktop-session (Ubuntu Trusty)
Changed in lightdm:
assignee: nobody → Robert Ancell (robert-ancell)
Revision history for this message
Brian Murray (brian-murray) wrote :

Here is the diff for future reviewers:

diff -Nru unity8-desktop-session-1.0.10+14.04.20140417.1/data/55-unity8.conf unity8-desktop-session-1.0.10+14.04.20140915/data/55-unity8.conf
--- unity8-desktop-session-1.0.10+14.04.20140417.1/data/55-unity8.conf 2014-04-17 05:07:47.000000000 -0700
+++ unity8-desktop-session-1.0.10+14.04.20140915/data/55-unity8.conf 2014-09-15 05:32:54.000000000 -0700
@@ -1,2 +1,4 @@
 [SeatDefaults]
+user-session=unity8-mir
+unity-compositor-command=/usr/sbin/unity-system-compositor
 session-wrapper=lightdm-unity8-session
diff -Nru unity8-desktop-session-1.0.10+14.04.20140417.1/data/lightdm-unity8-session unity8-desktop-session-1.0.10+14.04.20140915/data/lightdm-unity8-session
--- unity8-desktop-session-1.0.10+14.04.20140417.1/data/lightdm-unity8-session 2014-04-17 05:07:47.000000000 -0700
+++ unity8-desktop-session-1.0.10+14.04.20140915/data/lightdm-unity8-session 2014-09-15 05:32:54.000000000 -0700
@@ -4,7 +4,7 @@
 # traditional X-based desktop session startup.
 #

-if test -z "$DISPLAY"; then
+if test -z "$DISPLAY" && [ $DESKTOP_SESSION != "unity8-mir-lxc" ]; then
   exec init --user
 else
   exec /usr/sbin/lightdm-session "$@"
diff -Nru unity8-desktop-session-1.0.10+14.04.20140417.1/debian/changelog unity8-desktop-session-1.0.10+14.04.20140915/debian/changelog
--- unity8-desktop-session-1.0.10+14.04.20140417.1/debian/changelog 2014-09-18 13:49:23.000000000 -0700
+++ unity8-desktop-session-1.0.10+14.04.20140915/debian/changelog 2014-09-18 13:49:23.000000000 -0700
@@ -1,3 +1,12 @@
+unity8-desktop-session (1.0.10+14.04.20140915-0ubuntu1) trusty; urgency=low
+
+ [ Chris Townsend ]
+ * Add check to see if the Unity8 LXC session is not being started
+ before calling the init script. Also, add explicit call to
+ /usr/sbin/unity-system-compositor. (LP: #1359332)
+
+ -- Ubuntu daily release <email address hidden> Mon, 15 Sep 2014 12:33:09 +0000
+
 unity8-desktop-session (1.0.10+14.04.20140417.1-0ubuntu1) trusty; urgency=low

   [ Michał Sawicz ]

Revision history for this message
Brian Murray (brian-murray) wrote : Missing SRU information

Thanks for uploading the fix for this bug report to -proposed. However, when reviewing the package in -proposed and the details of this bug report I noticed that the bug description is missing information required for the SRU process. You can find full details at http://wiki.ubuntu.com/StableReleaseUpdates#Procedure but essentially this bug is missing some of the following: a statement of impact, a test case and details regarding the regression potential. Thanks in advance!

description: updated
Revision history for this message
Chris J Arges (arges) wrote : Please test proposed package

Hello Christopher, or anyone else affected,

Accepted unity8-desktop-session into trusty-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/unity8-desktop-session/1.0.10+14.04.20140915-0ubuntu1 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 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, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

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

Changed in unity8-desktop-session (Ubuntu Trusty):
status: New → Fix Committed
tags: added: verification-needed
Revision history for this message
Stephen M. Webb (bregma) wrote :

Verified this seems to work with unity-desktop-session-mir version 1.0.10+14.04.20140915-0ubuntu1 on Trusty.

tags: added: verification-done
removed: verification-needed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package unity8-desktop-session - 1.0.10+14.04.20140915-0ubuntu1

---------------
unity8-desktop-session (1.0.10+14.04.20140915-0ubuntu1) trusty; urgency=low

  [ Chris Townsend ]
  * Add check to see if the Unity8 LXC session is not being started
    before calling the init script. Also, add explicit call to
    /usr/sbin/unity-system-compositor. (LP: #1359332)
 -- Ubuntu daily release <email address hidden> Mon, 15 Sep 2014 12:33:09 +0000

Changed in unity8-desktop-session (Ubuntu Trusty):
status: Fix Committed → Fix Released
Revision history for this message
Chris J Arges (arges) wrote : Update Released

The verification of the Stable Release Update for unity8-desktop-session has completed successfully and the package has now been 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.

Changed in unity8-preview-lxc:
milestone: none → 1.0.0
Changed in unity8-preview-lxc:
status: Fix Committed → Fix Released
Revision history for this message
Mathew Hodson (mhodson) wrote :

https://launchpad.net/ubuntu/+source/lightdm/1.10.3-0ubuntu2 is available in trusty-proposed.

---------------
lightdm (1.10.3-0ubuntu2) trusty; urgency=medium

  * Refresh patches

 -- Robert Ancell <email address hidden> Fri, 10 Oct 2014 21:24:51 +1300

lightdm (1.10.3-0ubuntu1) trusty; urgency=medium

  * New upstream release:
    - Fix crash when having configuration keys defined in multiple places
      (LP: #1377373)
    - Allow user switching in multi-seat until bug stopping greeter showing on
      logout is fixed
    - Don't access .dmrc files until information from these files is required
      (LP: #1370852)
    - Do timed autologin each time you are returned to the greeter
      (LP: #1302491)

 -- Robert Ancell <email address hidden> Thu, 09 Oct 2014 09:01:21 +1300

lightdm (1.10.2-0ubuntu1) trusty; urgency=medium

  * New upstream release:
    - Use logind to provide the list of seats to use. This is disabled unless
      logind-load-seats (in [LightDM] section) is set to true. This is the
      default behaviour in lightdm 1.12. (LP: #1190581)
    - Fix crash if running script hooks with non-X display servers.
      (LP: #1305006)
    - Add a seat option 'allow-user-switching' that can disable all user
      switching for that seat. (LP: #1350357)
    - Make PAM services configurable. (LP: #1348251)
    - Add liblightdm method to get user UID. (LP: #1370327)
    - Add a new session type 'mir-container' that allows the session to run
      inside a custom system compositor. (LP: #1359332)
    - Add --show-config option that shows combined configuration.
    - Use XDG_SESSION_ID from PAM instead of using the logind D-Bus API.
      (LP: #1364725)
    - Make socket writing code used between greeter and daemon more robust to
      errors.
    - Correct section name in default users.conf file.
    - Fix tests failing with Qt 5.3 due to it checking getuid/geteuid which we
      are faking.
    - Fix small memory leaks.
    - Test improvements

 -- Robert Ancell <email address hidden> Wed, 17 Sep 2014 16:27:53 +1200

Changed in lightdm (Ubuntu Trusty):
status: New → Fix Committed
tags: added: verification-needed
removed: verification-done
tags: added: verification-done
removed: verification-needed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package lightdm - 1.10.3-0ubuntu2

---------------
lightdm (1.10.3-0ubuntu2) trusty; urgency=medium

  * Refresh patches
 -- Robert Ancell <email address hidden> Fri, 10 Oct 2014 21:24:51 +1300

Changed in lightdm (Ubuntu Trusty):
status: Fix Committed → Fix Released
Mathew Hodson (mhodson)
Changed in unity8-desktop-session:
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.