Merge ~binli/plainbox-provider-checkbox/+git/plainbox-provider-checkbox:skip-xorg-in-wayland into plainbox-provider-checkbox:master

Proposed by Bin Li
Status: Rejected
Rejected by: Sylvain Pineau
Proposed branch: ~binli/plainbox-provider-checkbox/+git/plainbox-provider-checkbox:skip-xorg-in-wayland
Merge into: plainbox-provider-checkbox:master
Diff against target: 46 lines (+18/-3)
1 file modified
units/graphics/jobs.pxu (+18/-3)
Reviewer Review Type Date Requested Status
Bin Li Disapprove
Sylvain Pineau Pending
Pierre Equoy Pending
Review via email: mp+425201@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Bin Li (binli) wrote :

When in xwayland.

-------------[ Running job 29 / 30. Estimated time left: unknown ]--------------
--------------------[ Test that the X process is running. ]---------------------
ID: com.canonical.certification::graphics/xorg-process
Category: com.canonical.plainbox::graphics
... 8< -------------------------------------------------------------------------
------------------------------------------------------------------------- >8 ---
Outcome: job passed
-------------[ Running job 30 / 30. Estimated time left: unknown ]--------------
-----------------------------[ Test Xorg version ]------------------------------
ID: com.canonical.certification::graphics/xorg-version
Category: com.canonical.plainbox::graphics
... 8< -------------------------------------------------------------------------
------------------------------------------------------------------------- >8 ---
Outcome: job passed

Revision history for this message
Bin Li (binli) wrote :

After switch to xorg.

-------------[ Running job 29 / 30. Estimated time left: unknown ]--------------
--------------------[ Test that the X process is running. ]---------------------
ID: com.canonical.certification::graphics/xorg-process
Category: com.canonical.plainbox::graphics
... 8< -------------------------------------------------------------------------
8039 /usr/lib/xorg/Xorg vt2 -displayfd 3 -auth /run/user/1001/gdm/Xauthority -nolisten tcp -background none -noreset -keeptty -novtswitch -verbose 3
12082 bash -c # shellcheck disable=SC1091 if [[ $XDG_SESSION_TYPE == "x11" ]] then pgrep -a -f '/usr/lib/xorg/Xorg' fi
------------------------------------------------------------------------- >8 ---
Outcome: job passed
-------------[ Running job 30 / 30. Estimated time left: unknown ]--------------
-----------------------------[ Test Xorg version ]------------------------------
ID: com.canonical.certification::graphics/xorg-version
Category: com.canonical.plainbox::graphics
... 8< -------------------------------------------------------------------------
1.21.1.3
------------------------------------------------------------------------- >8 ---
Outcome: job passed

Revision history for this message
Bin Li (binli) wrote :

Pieq helped find the root cause of the dependency of sutton-cli, and removed it, so I will drop my MR.

https://code.launchpad.net/~pieq/plainbox-provider-checkbox/+git/plainbox-provider-checkbox/+merge/425267

review: Disapprove

Unmerged commits

d105c31... by Bin Li

Check the XDG_SESSION_TYPE before running the jobs. (LP: #1968842)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/units/graphics/jobs.pxu b/units/graphics/jobs.pxu
2index 8206f1a..97b0e59 100644
3--- a/units/graphics/jobs.pxu
4+++ b/units/graphics/jobs.pxu
5@@ -23,7 +23,12 @@ id: graphics/xorg-process
6 requires:
7 package.name == 'xorg'
8 package.name == 'procps'
9-command: pgrep -a -f '/usr/lib/xorg/Xorg'
10+command:
11+ # shellcheck disable=SC1091
12+ if [[ $XDG_SESSION_TYPE == "x11" ]]
13+ then
14+ pgrep -a -f '/usr/lib/xorg/Xorg'
15+ fi
16 estimated_duration: 0.100
17 _description: Test that the X process is running.
18 _summary: Test that the X process is running.
19@@ -33,7 +38,12 @@ category_id: com.canonical.plainbox::graphics
20 id: graphics/xorg-version
21 requires: package.name == "x11-utils"
22 depends: graphics/xorg-process
23-command: set -o pipefail; xdpyinfo | grep "^X.Org version" | cut -d ':' -f 2 | tr -d ' '
24+command:
25+ # shellcheck disable=SC1091
26+ if [[ $XDG_SESSION_TYPE == "x11" ]]
27+ then
28+ set -o pipefail; xdpyinfo | grep "^X.Org version" | cut -d ':' -f 2 | tr -d ' '
29+ fi
30 estimated_duration: 0.018
31 _description: Test to output the Xorg version
32 _summary: Test Xorg version
33@@ -62,7 +72,12 @@ plugin: shell
34 category_id: com.canonical.plainbox::graphics
35 id: graphics/xorg-failsafe
36 requires: package.name == 'xorg'
37-command: ! test -e /var/log/Xorg.failsafe.log
38+command:
39+ # shellcheck disable=SC1091
40+ if [[ $XDG_SESSION_TYPE == "x11" ]]
41+ then
42+ ! test -e /var/log/Xorg.failsafe.log
43+ fi
44 estimated_duration: 0.030
45 _description: Test that the X is not running in failsafe mode.
46 _summary: Test that X is not in failsafe mode.

Subscribers

People subscribed via source and target branches