<?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>152196</bug_id>
          
          <creation_ts>2015-12-11 15:52:58 -0800</creation_ts>
          <short_desc>Perf dashboard&apos;s buildbot sync config JSON duplicates too much information</short_desc>
          <delta_ts>2016-02-16 13:53:33 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Perf Dashboard</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</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="Ryosuke Niwa">rniwa</reporter>
          <assigned_to name="Ryosuke Niwa">rniwa</assigned_to>
          <cc>cdumez</cc>
    
    <cc>dewei_zhu</cc>
    
    <cc>jond</cc>
    
    <cc>rniwa</cc>
    
    <cc>slewis</cc>
    
    <cc>timothy</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1148581</commentid>
    <comment_count>0</comment_count>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2015-12-11 15:52:58 -0800</bug_when>
    <thetext>Consolidate the duplicated entries.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1148584</commentid>
    <comment_count>1</comment_count>
      <attachid>267196</attachid>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2015-12-11 15:56:31 -0800</bug_when>
    <thetext>Created attachment 267196
Added shared configurations</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1148586</commentid>
    <comment_count>2</comment_count>
      <attachid>267196</attachid>
    <who name="">dewei_zhu</who>
    <bug_when>2015-12-11 15:59:46 -0800</bug_when>
    <thetext>Comment on attachment 267196
Added shared configurations

This will definitely simplify  the process of enabling a new test. Is the data format documented anywhere?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1148590</commentid>
    <comment_count>3</comment_count>
      <attachid>267196</attachid>
    <who name="Stephanie Lewis">slewis</who>
    <bug_when>2015-12-11 16:07:40 -0800</bug_when>
    <thetext>Comment on attachment 267196
Added shared configurations

View in context: https://bugs.webkit.org/attachment.cgi?id=267196&amp;action=review

&gt; Websites/perf.webkit.org/tools/sync-with-buildbot.py:78
&gt; +                config[key].update(value)

Would we ever have a list?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1148596</commentid>
    <comment_count>4</comment_count>
      <attachid>267196</attachid>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2015-12-11 16:11:54 -0800</bug_when>
    <thetext>Comment on attachment 267196
Added shared configurations

View in context: https://bugs.webkit.org/attachment.cgi?id=267196&amp;action=review

&gt;&gt; Websites/perf.webkit.org/tools/sync-with-buildbot.py:78
&gt;&gt; +                config[key].update(value)
&gt; 
&gt; Would we ever have a list?

If we did have a list, then overriding the old list with the new one (not merging is the right approach
so the code below (config[key] = value) will suffice.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1148597</commentid>
    <comment_count>5</comment_count>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2015-12-11 16:12:00 -0800</bug_when>
    <thetext>Committed r193992: &lt;http://trac.webkit.org/changeset/193992&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>267196</attachid>
            <date>2015-12-11 15:56:31 -0800</date>
            <delta_ts>2015-12-11 16:07:40 -0800</delta_ts>
            <desc>Added shared configurations</desc>
            <filename>bug-152196-20151211155559.patch</filename>
            <type>text/plain</type>
            <size>2154</size>
            <attacher name="Ryosuke Niwa">rniwa</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYnNpdGVzL3BlcmYud2Via2l0Lm9yZy9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0g
V2Vic2l0ZXMvcGVyZi53ZWJraXQub3JnL0NoYW5nZUxvZwkocmV2aXNpb24gMTkzOTkwKQorKysg
V2Vic2l0ZXMvcGVyZi53ZWJraXQub3JnL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwz
ICsxLDE2IEBACisyMDE1LTEyLTExICBSeW9zdWtlIE5pd2EgIDxybml3YUB3ZWJraXQub3JnPgor
CisgICAgICAgIFBlcmYgZGFzaGJvYXJkJ3MgYnVpbGRib3Qgc3luYyBjb25maWcgSlNPTiBkdXBs
aWNhdGVzIHRvbyBtdWNoIGluZm9ybWF0aW9uCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQu
b3JnL3Nob3dfYnVnLmNnaT9pZD0xNTIxOTYKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkg
KE9PUFMhKS4KKworICAgICAgICBBZGRlZCBzaGFyZWQsIHBlci1idWlsZGVyLCBhbmQgcGVyLXRl
c3QgKGNhbGxlZCB0eXBlKSBjb25maWd1cmF0aW9ucy4KKworICAgICAgICAqIHRvb2xzL3N5bmMt
d2l0aC1idWlsZGJvdC5weToKKyAgICAgICAgKGxvYWRfY29uZmlnKToKKyAgICAgICAgKGxvYWRf
Y29uZmlnLm1lcmdlKToKKwogMjAxNS0xMi0wMiAgUnlvc3VrZSBOaXdhICA8cm5pd2FAd2Via2l0
Lm9yZz4KIAogICAgICAgICBQZXJmIGRhc2hib2FyZCBzaG91bGQgYXZvaWQgb3ZlcmZsb3cgZHVy
aW5nIGdlb21ldHJpYyBtZWFuIGNvbXB1dGF0aW9uCkluZGV4OiBXZWJzaXRlcy9wZXJmLndlYmtp
dC5vcmcvdG9vbHMvc3luYy13aXRoLWJ1aWxkYm90LnB5Cj09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYnNpdGVz
L3BlcmYud2Via2l0Lm9yZy90b29scy9zeW5jLXdpdGgtYnVpbGRib3QucHkJKHJldmlzaW9uIDE5
Mzk4OSkKKysrIFdlYnNpdGVzL3BlcmYud2Via2l0Lm9yZy90b29scy9zeW5jLXdpdGgtYnVpbGRi
b3QucHkJKHdvcmtpbmcgY29weSkKQEAgLTY1LDkgKzY1LDI3IEBAIGRlZiBtYWluKCk6CiAKIGRl
ZiBsb2FkX2NvbmZpZyhjb25maWdfanNvbl9wYXRoLCBidWlsZGJvdF91cmwpOgogICAgIHdpdGgg
b3Blbihjb25maWdfanNvbl9wYXRoKSBhcyBjb25maWdfanNvbjoKLSAgICAgICAgY29uZmlndXJh
dGlvbnMgPSBqc29uLmxvYWQoY29uZmlnX2pzb24pCisgICAgICAgIG9wdGlvbnMgPSBqc29uLmxv
YWQoY29uZmlnX2pzb24pCiAKKyAgICBzaGFyZWRfY29uZmlnID0gb3B0aW9uc1snc2hhcmVkJ10K
KyAgICB0eXBlX2NvbmZpZyA9IG9wdGlvbnNbJ3R5cGVzJ10KKyAgICBidWlsZGVyX2NvbmZpZyA9
IG9wdGlvbnNbJ2J1aWxkZXJzJ10KKworICAgIGRlZiBtZXJnZShjb25maWcsIGNvbmZpZ190b19t
ZXJnZSk6CisgICAgICAgIGZvciBrZXksIHZhbHVlIGluIGNvbmZpZ190b19tZXJnZS5pdGVyaXRl
bXMoKToKKyAgICAgICAgICAgIGlmIGlzaW5zdGFuY2UodmFsdWUsIGRpY3QpOgorICAgICAgICAg
ICAgICAgIGNvbmZpZy5zZXRkZWZhdWx0KGtleSwge30pCisgICAgICAgICAgICAgICAgY29uZmln
W2tleV0udXBkYXRlKHZhbHVlKQorICAgICAgICAgICAgZWxzZToKKyAgICAgICAgICAgICAgICBj
b25maWdba2V5XSA9IHZhbHVlCisKKyAgICBjb25maWd1cmF0aW9ucyA9IG9wdGlvbnNbJ2NvbmZp
Z3VyYXRpb25zJ10KICAgICBmb3IgY29uZmlnIGluIGNvbmZpZ3VyYXRpb25zOgorCisgICAgICAg
IG1lcmdlKGNvbmZpZywgc2hhcmVkX2NvbmZpZykKKyAgICAgICAgbWVyZ2UoY29uZmlnLCB0eXBl
X2NvbmZpZ1tjb25maWcucG9wKCd0eXBlJyldKQorICAgICAgICBtZXJnZShjb25maWcsIGJ1aWxk
ZXJfY29uZmlnW2NvbmZpZy5wb3AoJ2J1aWxkZXInKV0pCisKICAgICAgICAgZXNjYXBlZF9idWls
ZGVyX25hbWUgPSB1cmxsaWIucXVvdGUoY29uZmlnWydidWlsZGVyJ10pCiAgICAgICAgIGNvbmZp
Z1sndXJsJ10gPSAnJXMvYnVpbGRlcnMvJXMvJyAlIChidWlsZGJvdF91cmwsIGVzY2FwZWRfYnVp
bGRlcl9uYW1lKQogICAgICAgICBjb25maWdbJ2pzb25VUkwnXSA9ICclcy9qc29uL2J1aWxkZXJz
LyVzLycgJSAoYnVpbGRib3RfdXJsLCBlc2NhcGVkX2J1aWxkZXJfbmFtZSkK
</data>
<flag name="review"
          id="292254"
          type_id="1"
          status="+"
          setter="slewis"
    />
          </attachment>
      

    </bug>

</bugzilla>