Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add reboot to linux unlock script #23382

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

dantecatalfamo
Copy link
Member

@dantecatalfamo dantecatalfamo commented Oct 30, 2024

#22437

There is a bug in Ubuntu 24.04's distribution of GDM that prevents it from starting correctly and displaying a prompt to the user if /etc/nologin is present. This issue is not present on the current release of Fedora, meaning it is Ubuntu specific.

The way we lock users out is by manually creating the nologin file and then masking the systemd-user-sessions systemd unit, which creates the file on shutdown and deletes it on startup. This will cause a PAM policy to fail and prevents anyone from logging in. When we unlock the system we delete the nologin file, unmask the systemd-user-sessions unit, and manually run the binary that it should start.

This process removes the cause of the GDM bug, but we need to reboot the machine to get GDM working again.

While I have not yet been able to determine the exact cause of the bug, this fix will prevent the user from being stuck with a black screen once the machine is unlocked.

This fix will not remedy GDM showing a black screen upon being locked, it only ensures that the user isn't stuck having to manually reboot the machine once it's unlocked.

We should check back on this soon to see if the bug gets been fixed upstream.

If some of the following don't apply, delete the relevant line.

  • Changes file added for user-visible changes in changes/, orbit/changes/ or ee/fleetd-chrome/changes.
    See Changes files for more information.
  • Manual QA for all new/changed functionality

@dantecatalfamo dantecatalfamo marked this pull request as ready for review October 30, 2024 14:54
@dantecatalfamo dantecatalfamo requested a review from a team as a code owner October 30, 2024 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant