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 a lightdm session for sys-gui #152

Closed
wants to merge 2 commits into from
Closed

Conversation

pwmarcz
Copy link
Contributor

@pwmarcz pwmarcz commented Jul 21, 2020

Starts just sys-gui VM and qvm-start-daemon in a special mode.

See QubesOS/qubes-issues#5662.

This seems to need allow_fullscreen. I'd rather not add it to sys-gui because then it's becomes problematic to test it when running a normal session. Added an override: qvm-start-daemon --allow-fullscreen.

Remaining issues:

  • Selecting "Logout" in sys-gui seems to shut down the Xephyr window but not the VM. Should it work that way? Should we detect that the GUI agent inside has died and then exit dom0 session, or rather make the VM shut down?

@codecov
Copy link

codecov bot commented Jul 21, 2020

Codecov Report

Merging #152 into master will decrease coverage by 0.05%.
The diff coverage is 35.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #152      +/-   ##
==========================================
- Coverage   74.09%   74.03%   -0.06%     
==========================================
  Files          49       49              
  Lines        6327     6359      +32     
==========================================
+ Hits         4688     4708      +20     
- Misses       1639     1651      +12     
Impacted Files Coverage Δ
qubesadmin/tools/qvm_start_daemon.py 53.55% <35.00%> (-1.58%) ⬇️
qubesadmin/tools/qvm_run.py 80.31% <0.00%> (+2.65%) ⬆️
qubesadmin/spinner.py 77.41% <0.00%> (+6.45%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 60d28dc...a0ce82f. Read the comment docs.

Starts just sys-gui VM and qvm-start-daemon in a special mode.

See QubesOS/qubes-issues#5662.
@pwmarcz pwmarcz force-pushed the gui-session branch 3 times, most recently from f26214a to a0ce82f Compare July 23, 2020 14:13
I'm doing it as a special option to qvm-start-daemon instead of
changing the option globally for sys-gui, because I want sys-gui
windows to be also usable in normal XFCE session, e.g. for
development.
@pwmarcz
Copy link
Contributor Author

pwmarcz commented Jul 27, 2020

This seems to need allow_fullscreen. I'd rather not add it to sys-gui because then it's becomes problematic to test it when running a normal session. Added an override: qvm-start-daemon --allow-fullscreen.

After discussing: remove --allow-fullscreen and the permission for sys-gui globally in Salt configuration.

@pwmarcz
Copy link
Contributor Author

pwmarcz commented Jul 27, 2020

Salt should create a xsession file alongside the GuiVM, so that if you create several GuiVMs, it will create several session files.

@pwmarcz
Copy link
Contributor Author

pwmarcz commented Jul 27, 2020

  • qvm-start-daemon should exit when GUI daemon exits (and run it in foreground)
  • GUI daemon should have a --no-restart option

@pwmarcz
Copy link
Contributor Author

pwmarcz commented Jul 27, 2020

It might be better to modify qubes-guid's restart_guid function.

@pwmarcz
Copy link
Contributor Author

pwmarcz commented Jul 28, 2020

That will need a new branch, I'll close this one.

@pwmarcz pwmarcz closed this Jul 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant