<?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>253913</bug_id>
          
          <creation_ts>2023-03-14 15:06:30 -0700</creation_ts>
          <short_desc>[GTK][WPE] run-webkit-tests: Add an option to enable coredumps and selectively enable it on the bots.</short_desc>
          <delta_ts>2023-03-15 17:53:11 -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>Tools / Tests</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          <see_also>https://bugs.webkit.org/show_bug.cgi?id=252762</see_also>
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Carlos Alberto Lopez Perez">clopez</reporter>
          <assigned_to name="Carlos Alberto Lopez Perez">clopez</assigned_to>
          <cc>bugs-noreply</cc>
    
    <cc>fujii</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1941354</commentid>
    <comment_count>0</comment_count>
    <who name="Carlos Alberto Lopez Perez">clopez</who>
    <bug_when>2023-03-14 15:06:30 -0700</bug_when>
    <thetext>On the GTK and WPE test bots we run with coredumps enabled so when layout tests crash they generate a coredump and we get a backtrace.

The problem is on the EWS bots that we repeat each failure up to 11 times each failure 10 times then a lot of coredumps can be generated in a sort space of time. This may end causing the bot to run out of space. In bug 252762 I implemented a way of cleaning the coredump after processing it but it seems this is not enough as sometimes the bots are still filing the disk.

See: https://ews-build.webkit.org/#/builders/86/builds/5299

The cause is that sometimes crashes happen that are not intercepted by the tooling so those coredump don&apos;t get deleted after generating them.

It would be good to investigate why those crashes are not intercepted by the tooling, and is on my todo.

However, at any rate, it makes little sense that we run with coredumps enabled the repeat steps for the EWS. If a test is crashing 11 times we don&apos;t need to generate 11 coredumps. It is enough with just picking one.

So I think we should disable coredumps when doing the repeat steps. For doing this we can implement a new flag that can be passed to run-webkit-tests to enable coredumps and we pass this flag on the bots for the steps we want coredumps to be generated.

This can be useful also for developers as they can just enable coredumps when running tests by passing a flag to run-webkit-tests instead of enabling those on the parent sell.

This also has the advantage of having faster runs when doing the retry step with lot of crashes. Having to generate 11 coredumps per crashing test and running GDB on all of them to generate the backtrace is really slow and can make the retry test step last 10x or 20x more time. Generating each backtrace can take easily 1 or 2 minutes so we are spending 10 or 20 minutes per each test that crashes. Not good.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1941422</commentid>
    <comment_count>1</comment_count>
    <who name="Carlos Alberto Lopez Perez">clopez</who>
    <bug_when>2023-03-14 18:55:41 -0700</bug_when>
    <thetext>Pull request: https://github.com/WebKit/WebKit/pull/11533</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1941641</commentid>
    <comment_count>2</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2023-03-15 13:15:03 -0700</bug_when>
    <thetext>Committed 261707@main (4bf80e3962bb): &lt;https://commits.webkit.org/261707@main&gt;

Reviewed commits have been landed. Closing PR #11533 and removing active labels.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1941642</commentid>
    <comment_count>3</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2023-03-15 13:16:17 -0700</bug_when>
    <thetext>&lt;rdar://problem/106769832&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1941722</commentid>
    <comment_count>4</comment_count>
    <who name="Fujii Hironori">fujii</who>
    <bug_when>2023-03-15 17:53:11 -0700</bug_when>
    <thetext>Regressed: Bug 253994 – REGRESSION(261707@main) run-webkit-tests: ModuleNotFoundError: No module named &apos;resource&apos;</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>