ubuntu / authd

Authentication daemon for external Brokers
GNU Lesser General Public License v3.0
110 stars 9 forks source link

issue-template: Add command to get shell and gdm logs #484

Open 3v1n0 opened 2 months ago

codecov-commenter commented 2 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 84.90%. Comparing base (a99a014) to head (74626a9).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #484 +/- ## ========================================== + Coverage 84.83% 84.90% +0.07% ========================================== Files 79 79 Lines 6922 6922 Branches 75 75 ========================================== + Hits 5872 5877 +5 + Misses 731 730 -1 + Partials 319 315 -4 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

adombeck commented 2 months ago

The command I added in https://github.com/ubuntu/authd/pull/485 includes logs from gdm-authd. What is logged by gnome-shell that's relevant for us?

3v1n0 commented 2 months ago

What is logged by gnome-shell that's relevant for us?

GNOME Shell is the GDM (that is just the backend) user interface, so if there are any failures or relevant logs, we may need them. Ideally I should enable a way to get verbose logging from the shell too, but that's another item.

adombeck commented 2 months ago

GNOME Shell is the GDM (that is just the backend) user interface, so if there are any failures or relevant logs, we may need them.

gnome-shell logs can be pretty big, so I'm a bit hesitant to just include all of them by default in the GitHub issue, also because it increases the effort for the user to redact the logs. Can we somehow restrict which gnome-shell logs we add?

3v1n0 commented 2 months ago

Well, we can tail the last lines (e.g. -n 200) or just use journalctl to limit by time.

3v1n0 commented 2 months ago

Let me know if you prefer to merge this first or close it instead so that can be handled all at once by @adombeck in #485