<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>230249</bug_id>
          
          <creation_ts>2021-09-14 03:16:40 -0700</creation_ts>
          <short_desc>[GTK] TestWebKitAccessibility is always skipped in the bots</short_desc>
          <delta_ts>2021-10-19 21:08:53 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebKitGTK</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>REOPENED</bug_status>
          <resolution></resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>Gtk</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Carlos Garcia Campos">cgarcia</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>bugs-noreply</cc>
    
    <cc>clopez</cc>
    
    <cc>pgriffis</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1793246</commentid>
    <comment_count>0</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2021-09-14 03:16:40 -0700</bug_when>
    <thetext>We fail to start the a11y service under flatpak.

Failed to talk to the accessibility service: Cannot autolaunch D-Bus without X11 $DISPLAY
Could not start accessibility bus, so disabling TestWebKitAccessibility</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1793248</commentid>
    <comment_count>1</comment_count>
      <attachid>438120</attachid>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2021-09-14 03:39:24 -0700</bug_when>
    <thetext>Created attachment 438120
Patch

Let&apos;s try this...</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1793249</commentid>
    <comment_count>2</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2021-09-14 04:02:27 -0700</bug_when>
    <thetext>Still fails :-(

Failed to talk to the accessibility service: Error spawning command line ?dbus-launch --autolaunch=18a76d89e81b4cb5aa297189e9cbb201 --binary-syntax --close-stderr?: Child process exited with code 1</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1797423</commentid>
    <comment_count>3</comment_count>
    <who name="Carlos Alberto Lopez Perez">clopez</who>
    <bug_when>2021-09-27 04:14:35 -0700</bug_when>
    <thetext>This seems an issue related to dbus and the configuration we currently have on the flatpak and the workers.
This is what happens:

	1. When the test runs it starts the dbus accessibility bus
	2. For starting this it looks for an already running dbus daemon
	3. On the workers it doesn&apos;t find the dbus daemon because it is not enabled for the service of the buildbot
	4. Then the test it tries to auto-start dbus by using the dbus magic auto-start feature
	5. On the dbus binary of the flatpak this magic is not enabled so it fails with this error &quot;Autolaunch requested, but X11 support not compiled in. Cannot continue.&quot;


So we need to either:
	- Option A -&gt; Build dbus with the autostart feature inside flatpak
	- Option B -&gt; Start the dbus daemon if needed on the test-runner before starting the tests
	- Option C -&gt; Ensure the workers have a running dbus daemon for the buildbot service

After discussing this with Carlos it seems that Option C seems the best one, as that is closer to the usual developer/user environment where dbus is assumed to be running and working.
I will try to deploy this on the workers.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1805545</commentid>
    <comment_count>4</comment_count>
      <attachid>438120</attachid>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2021-10-18 05:45:23 -0700</bug_when>
    <thetext>Comment on attachment 438120
Patch

This didn&apos;t fix the issue.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1805546</commentid>
    <comment_count>5</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2021-10-18 05:46:43 -0700</bug_when>
    <thetext>In the end, bots have been configured to run dbus service, so tests should be running fine now.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1806486</commentid>
    <comment_count>6</comment_count>
    <who name="Carlos Alberto Lopez Perez">clopez</who>
    <bug_when>2021-10-19 21:08:39 -0700</bug_when>
    <thetext>The fix applied on the bots caused timeouts on the layout tests.

There is something weird interaction happening between flatpak and XDG_RUNTIME_DIR.

This requires more investigation</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1806487</commentid>
    <comment_count>7</comment_count>
    <who name="Carlos Alberto Lopez Perez">clopez</who>
    <bug_when>2021-10-19 21:08:53 -0700</bug_when>
    <thetext>The previous fix was reverted for the moment</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>438120</attachid>
            <date>2021-09-14 03:39:24 -0700</date>
            <delta_ts>2021-10-18 05:45:23 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>wk-a11y-test.diff</filename>
            <type>text/plain</type>
            <size>4696</size>
            <attacher name="Carlos Garcia Campos">cgarcia</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1Rvb2xzL0NoYW5nZUxvZyBiL1Rvb2xzL0NoYW5nZUxvZwppbmRleCA0M2Iy
NTMyYTFiNjguLjNhNGE0OWI3NzcyNSAxMDA2NDQKLS0tIGEvVG9vbHMvQ2hhbmdlTG9nCisrKyBi
L1Rvb2xzL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE4IEBACisyMDIxLTA5LTE0ICBDYXJsb3MgR2Fy
Y2lhIENhbXBvcyAgPGNnYXJjaWFAaWdhbGlhLmNvbT4KKworICAgICAgICBbR1RLXSBUZXN0V2Vi
S2l0QWNjZXNzaWJpbGl0eSBpcyBhbHdheXMgc2tpcHBlZCBpbiB0aGUgYm90cworICAgICAgICBo
dHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MjMwMjQ5CisKKyAgICAgICAg
UmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgTW92ZSB0aGUgYTExeSBEQnVz
IGFjdGl2YXRpb24gZnJvbSB0aGUgc2NyaXB0IHJ1bm5lciB0byB0aGUgdGVzdCwgYmVhdXNlIHRo
ZSBzY3JpcHQgZG9lc24ndCBydW4gdW5kZXIgZmxhdHBhay4KKworICAgICAgICAqIFNjcmlwdHMv
cnVuLWd0ay10ZXN0czoKKyAgICAgICAgKiBUZXN0V2ViS2l0QVBJL1Rlc3RzL1dlYktpdEd0ay9U
ZXN0V2ViS2l0QWNjZXNzaWJpbGl0eS5jcHA6CisgICAgICAgICh0ZXN0QXRzcGlCYXNpY0hpZXJh
cmNoeSk6CisgICAgICAgIChiZWZvcmVBbGwpOgorICAgICAgICAoYWZ0ZXJBbGwpOgorCiAyMDIx
LTA5LTEzICBXZW5zb24gSHNpZWggIDx3ZW5zb25faHNpZWhAYXBwbGUuY29tPgogCiAgICAgICAg
IFttYWNPU10gLXRvZ2dsZUF1dG9tYXRpY1NwZWxsaW5nQ29ycmVjdGlvbjogbWVudSBpdGVtIHZh
bGlkYXRpb24gc2hvdWxkbid0IHJlcXVpcmUgcG9zdC1sYXlvdXQgZWRpdG9yIHN0YXRlCmRpZmYg
LS1naXQgYS9Ub29scy9TY3JpcHRzL3J1bi1ndGstdGVzdHMgYi9Ub29scy9TY3JpcHRzL3J1bi1n
dGstdGVzdHMKaW5kZXggNDY1NTZmYmZlNmY2Li5mZDgyZGNmMDgyNDkgMTAwNzU1Ci0tLSBhL1Rv
b2xzL1NjcmlwdHMvcnVuLWd0ay10ZXN0cworKysgYi9Ub29scy9TY3JpcHRzL3J1bi1ndGstdGVz
dHMKQEAgLTQ1LDM1ICs0NSw5IEBAIGNsYXNzIEd0a1Rlc3RSdW5uZXIoVGVzdFJ1bm5lcik6CiAg
ICAgZGVmIF9faW5pdF9fKHNlbGYsIG9wdGlvbnMsIHRlc3RzPVtdKToKICAgICAgICAgc3VwZXIo
R3RrVGVzdFJ1bm5lciwgc2VsZikuX19pbml0X18oImd0ayIsIG9wdGlvbnMsIHRlc3RzKQogCi0g
ICAgZGVmIF9lbnN1cmVfYWNjZXNzaWJpbGl0eV9zZXJ2aWNlX2lzX3J1bm5pbmcoc2VsZik6Ci0g
ICAgICAgICMgVGhlIGExMXkgc2VydmljZSBpcyBEQnVzIGFjdGl2YXRhYmxlIHNvIHdlIHNpbXBs
eSBuZWVkIHRvIGNhbGwgaXQgdG8gZW5zdXJlIGl0Ci0gICAgICAgICMgaGFzIGJlZW4gc3RhcnRl
ZC4KLSAgICAgICAgdHJ5OgotICAgICAgICAgICAgcHJveHkgPSBHaW8uREJ1c1Byb3h5Lm5ld19m
b3JfYnVzX3N5bmMoR2lvLkJ1c1R5cGUuU0VTU0lPTiwKLSAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgIEdpby5EQnVzUHJveHlGbGFncy5OT05FLAotICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgTm9uZSwKLSAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICdvcmcuYTEx
eS5CdXMnLAotICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgJy9vcmcvYTExeS9idXMnLAotICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgJ29yZy5hMTF5LkJ1cycsCi0gICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICBOb25lKQotCi0gICAgICAgICAgICAjIEFmdGVyIHRo
aXMgcmV0dXJucyB3ZSBrbm93IHRoZSBidXMgaXMgc2V0dXAsIHRoZSByZXR1cm4gdmFsdWUgZG9l
c24ndCBtYXR0ZXIuCi0gICAgICAgICAgICBwcm94eS5HZXRBZGRyZXNzKCkKLSAgICAgICAgZXhj
ZXB0IEdMaWIuRXJyb3IgYXMgZToKLSAgICAgICAgICAgIHN5cy5zdGRlcnIud3JpdGUoIkZhaWxl
ZCB0byB0YWxrIHRvIHRoZSBhY2Nlc3NpYmlsaXR5IHNlcnZpY2U6IHt9XG4iLmZvcm1hdChlLm1l
c3NhZ2UpKQotICAgICAgICAgICAgc3lzLnN0ZGVyci5mbHVzaCgpCi0gICAgICAgICAgICByZXR1
cm4gRmFsc2UKLQotICAgICAgICByZXR1cm4gVHJ1ZQotCiAgICAgZGVmIF9zZXR1cF90ZXN0aW5n
X2Vudmlyb25tZW50KHNlbGYpOgogICAgICAgICBzdXBlcihHdGtUZXN0UnVubmVyLCBzZWxmKS5f
c2V0dXBfdGVzdGluZ19lbnZpcm9ubWVudCgpCiAKLSAgICAgICAgIyBJZiB3ZSBjYW5ub3Qgc3Rh
cnQgdGhlIGFjY2Vzc2liaWxpdHkgZGFlbW9ucywgd2UgY2FuIGp1c3Qgc2tpcCB0aGUgYWNjZXNz
aWJpbGl0eSB0ZXN0cy4KLSAgICAgICAgaWYgbm90IHNlbGYuX2Vuc3VyZV9hY2Nlc3NpYmlsaXR5
X3NlcnZpY2VfaXNfcnVubmluZygpOgotICAgICAgICAgICAgcHJpbnQoIkNvdWxkIG5vdCBzdGFy
dCBhY2Nlc3NpYmlsaXR5IGJ1cywgc28gZGlzYWJsaW5nIFRlc3RXZWJLaXRBY2Nlc3NpYmlsaXR5
IikKLSAgICAgICAgICAgIHNlbGYuX2Rpc2FibGVkX3Rlc3RzLmFwcGVuZCgiV2ViS2l0Mkd0ay9U
ZXN0V2ViS2l0QWNjZXNzaWJpbGl0eSIpCi0KICAgICBkZWYgX3RlYXJfZG93bl90ZXN0aW5nX2Vu
dmlyb25tZW50KHNlbGYpOgogICAgICAgICBzdXBlcihHdGtUZXN0UnVubmVyLCBzZWxmKS5fdGVh
cl9kb3duX3Rlc3RpbmdfZW52aXJvbm1lbnQoKQogCmRpZmYgLS1naXQgYS9Ub29scy9UZXN0V2Vi
S2l0QVBJL1Rlc3RzL1dlYktpdEd0ay9UZXN0V2ViS2l0QWNjZXNzaWJpbGl0eS5jcHAgYi9Ub29s
cy9UZXN0V2ViS2l0QVBJL1Rlc3RzL1dlYktpdEd0ay9UZXN0V2ViS2l0QWNjZXNzaWJpbGl0eS5j
cHAKaW5kZXggMWE0NTZiODBkMzgxLi5jYTMxZGY4MTE4YTkgMTAwNjQ0Ci0tLSBhL1Rvb2xzL1Rl
c3RXZWJLaXRBUEkvVGVzdHMvV2ViS2l0R3RrL1Rlc3RXZWJLaXRBY2Nlc3NpYmlsaXR5LmNwcAor
KysgYi9Ub29scy9UZXN0V2ViS2l0QVBJL1Rlc3RzL1dlYktpdEd0ay9UZXN0V2ViS2l0QWNjZXNz
aWJpbGl0eS5jcHAKQEAgLTI2LDYgKzI2LDggQEAgZXh0ZXJuICJDIiB7CiAjaW5jbHVkZSA8YXRz
cGkvYXRzcGkuaD4KIH0KIAorc3RhdGljIGJvb2wgc19hY2Nlc3NpYmlsaXR5U2VydmljZUlzUnVu
bmluZzsKKwogY2xhc3MgQWNjZXNzaWJpbGl0eVRlc3QgOiBwdWJsaWMgV2ViVmlld1Rlc3Qgewog
cHVibGljOgogICAgIE1BS0VfR0xJQl9URVNUX0ZJWFRVUkUoQWNjZXNzaWJpbGl0eVRlc3QpOwpA
QCAtOTQsNiArOTYsMTAgQEAgcHJpdmF0ZToKIAogc3RhdGljIHZvaWQgdGVzdEF0c3BpQmFzaWNI
aWVyYXJjaHkoQWNjZXNzaWJpbGl0eVRlc3QqIHRlc3QsIGdjb25zdHBvaW50ZXIpCiB7CisgICAg
aWYgKCFzX2FjY2Vzc2liaWxpdHlTZXJ2aWNlSXNSdW5uaW5nKSB7CisgICAgICAgIGdfdGVzdF9z
a2lwKCJBY2Nlc3NpYmlsaXR5IHNlcnZpY2UgaXMgbm90IHJ1bm5pbmciKTsKKyAgICAgICAgcmV0
dXJuOworICAgIH0KICAgICB0ZXN0LT5zaG93SW5XaW5kb3coKTsKICAgICB0ZXN0LT5sb2FkSHRt
bCgKICAgICAgICAgIjxodG1sPiIKQEAgLTE3Nyw5ICsxODMsMjMgQEAgc3RhdGljIHZvaWQgdGVz
dEF0c3BpQmFzaWNIaWVyYXJjaHkoQWNjZXNzaWJpbGl0eVRlc3QqIHRlc3QsIGdjb25zdHBvaW50
ZXIpCiAKIHZvaWQgYmVmb3JlQWxsKCkKIHsKKyAgICAvLyBUaGUgYTExeSBzZXJ2aWNlIGlzIERC
dXMgYWN0aXZhdGFibGUgc28gd2Ugc2ltcGx5IG5lZWQgdG8gY2FsbCBpdCB0byBlbnN1cmUgaXQg
aGFzIGJlZW4gc3RhcnRlZC4KKyAgICBHVW5pcXVlT3V0UHRyPEdFcnJvcj4gZXJyb3I7CisgICAg
R1JlZlB0cjxHREJ1c1Byb3h5PiBwcm94eSA9IGFkb3B0R1JlZihnX2RidXNfcHJveHlfbmV3X2Zv
cl9idXNfc3luYyhHX0JVU19UWVBFX1NFU1NJT04sIEdfREJVU19QUk9YWV9GTEFHU19OT05FLCBu
dWxscHRyLCAib3JnLmExMXkuQnVzIiwgIi9vcmcvYTExeS9idXMiLCAib3JnLmExMXkuQnVzIiwg
bnVsbHB0ciwgJmVycm9yLm91dFB0cigpKSk7CisgICAgaWYgKHByb3h5KSB7CisgICAgICAgIC8v
IEFmdGVyIHRoaXMgcmV0dXJucyB3ZSBrbm93IHRoZSBidXMgaXMgc2V0dXAsIHRoZSByZXR1cm4g
dmFsdWUgZG9lc24ndCBtYXR0ZXIuCisgICAgICAgIEdSZWZQdHI8R1ZhcmlhbnQ+IHZhcmlhbnQg
PSBhZG9wdEdSZWYoZ19kYnVzX3Byb3h5X2NhbGxfc3luYyhwcm94eS5nZXQoKSwgIkdldEFkZHJl
c3MiLCBudWxscHRyLCBHX0RCVVNfQ0FMTF9GTEFHU19OT05FLCAyNTAsIG51bGxwdHIsICZlcnJv
ci5vdXRQdHIoKSkpOworICAgIH0KKworICAgIGlmIChlcnJvcikKKyAgICAgICAgZ19wcmludGVy
cigiRmFpbGVkIHRvIHRhbGsgdG8gdGhlIGFjY2Vzc2liaWxpdHkgc2VydmljZTogJXMiLCBlcnJv
ci0+bWVzc2FnZSk7CisgICAgZWxzZQorICAgICAgICBzX2FjY2Vzc2liaWxpdHlTZXJ2aWNlSXNS
dW5uaW5nID0gdHJ1ZTsKKwogICAgIEFjY2Vzc2liaWxpdHlUZXN0OjphZGQoIldlYktpdEFjY2Vz
c2liaWxpdHkiLCAiYXRzcGktYmFzaWMtaGllcmFyY2h5IiwgdGVzdEF0c3BpQmFzaWNIaWVyYXJj
aHkpOwogfQogCiB2b2lkIGFmdGVyQWxsKCkKIHsKKyAgICBzX2FjY2Vzc2liaWxpdHlTZXJ2aWNl
SXNSdW5uaW5nID0gZmFsc2U7CiB9Cg==
</data>

          </attachment>
      

    </bug>

</bugzilla>