<?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>107052</bug_id>
          
          <creation_ts>2013-01-16 14:49:16 -0800</creation_ts>
          <short_desc>Buildmaster shouldn&apos;t restart after all webkitpy change</short_desc>
          <delta_ts>2015-09-24 07:32:04 -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>New Bugs</component>
          <version>420+</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>WONTFIX</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Csaba Osztrogonác">ossy</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>lforschler</cc>
    
    <cc>ossy</cc>
    
    <cc>wsiegrist</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>808736</commentid>
    <comment_count>0</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2013-01-16 14:49:16 -0800</bug_when>
    <thetext>It seems build.webkit.org restarts itself after all webkitpy change.
Today it was restarted twice in a short period because of these changes:
- http://build.webkit.org/changes/20528 (http://trac.webkit.org/changeset/139913)
- http://build.webkit.org/changes/20539 (http://trac.webkit.org/changeset/139924)

Do we really need restarting master after all webkitpy change?
I think only master.cfg changes should trigger master restart.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1128486</commentid>
    <comment_count>1</comment_count>
    <who name="Lucas Forschler">lforschler</who>
    <bug_when>2015-09-24 07:29:30 -0700</bug_when>
    <thetext>Buildbot is restarted via post commit hook from the svn sever. The post commit hook reads as follows:

#
# update build.webkit.org
#
UPDATE_BUILDBOT=$(/usr/bin/svnlook changed $REPOS -r $REV | /bin/grep -cE &apos;(webkitpy|BuildSlaveSupport)&apos;)
if [ $UPDATE_BUILDBOT -gt 0 ];
then
	/usr/bin/ssh -t buildbot@build.webkit.org &quot;/usr/bin/sudo -n /usr/local/bin/buildbot-update-restart --force&quot; &gt;&gt; /tmp/post-commit.log 2&gt;&amp;1 &amp;
fi</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1128487</commentid>
    <comment_count>2</comment_count>
    <who name="Lucas Forschler">lforschler</who>
    <bug_when>2015-09-24 07:32:04 -0700</bug_when>
    <thetext>However, looking in the buildbot-update-restart code, it seems we filter out on config.json or master.cfg changes

RESTART=$(/usr/bin/svn log -r $OLDREV:HEAD -vq | grep -cE &apos;(config.json|master.cfg)&apos;);</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>