<?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>64490</bug_id>
          
          <creation_ts>2011-07-13 15:51:58 -0700</creation_ts>
          <short_desc>NRWT spins the HTTP server down and back up an extra time when retrying a failed HTTP test when</short_desc>
          <delta_ts>2012-08-08 16:51:28 -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>528+ (Nightly build)</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>NEW</bug_status>
          <resolution></resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>NRWT</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>64491</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="WebKit Review Bot">webkit.review.bot</reporter>
          <assigned_to name="Eric Seidel (no email)">eric</assigned_to>
          <cc>abarth</cc>
    
    <cc>dpranke</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>436723</commentid>
    <comment_count>0</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2011-07-13 15:51:58 -0700</bug_when>
    <thetext>NRWT spins the HTTP server down and back up an extra time when retrying a failed HTTP test when
Requested by abarth on #webkit.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>436728</commentid>
    <comment_count>1</comment_count>
    <who name="Dirk Pranke">dpranke</who>
    <bug_when>2011-07-13 15:55:06 -0700</bug_when>
    <thetext>I&apos;m not sure if you can do anything about this and still drop the http lock aggressively (in order to enable concurrent NRWT runs to share the http lock).

Two possible options are to not drop the lock if there is a failure (assume that retrying the test takes precedence over sharing the machine), or to provide a command line flag to toggle whether we need locks at all (and just start and stop the server at the beginning/end if you don&apos;t).

But starting and stopping should be pretty fast ... is this causing some other problem, or is it just slow?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>436735</commentid>
    <comment_count>2</comment_count>
    <who name="Adam Barth">abarth</who>
    <bug_when>2011-07-13 16:00:39 -0700</bug_when>
    <thetext>I suspect an easy solution is to just keep holding the lock when all the tests we&apos;ve been asked to run are HTTP tests.  That case will basically only occur when running NRWT interactively and when the user has selected a small subset of tests to run.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>690928</commentid>
    <comment_count>3</comment_count>
    <who name="Dirk Pranke">dpranke</who>
    <bug_when>2012-08-08 16:51:28 -0700</bug_when>
    <thetext>at least now (and I think this was probably the case even when this bug was filed), the code that manages retrying on failures is separate from the code that actually starts and stops the servers (manager.py vs. layout_test_runner.py).

So, we&apos;d have to rework the layering to add a dont_stop_the_server_if_there_are_failures and then a _no_need_to_start_the_server_its_already_running flag on those interfaces. 

This doesn&apos;t really seem worth it to me, but I&apos;m not sure if this is really annoying anyone else?

Thoughts?</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>