buildbot UI: add linkification of Rietveld issue URL to build page.

masters/master.tryserver.chromium.linux/templates/build.html

 {% extends "layout.html" %}
 {% import 'forms.html' as forms %}
 {% from "change_macros.html" import change with context %}
 
 {% block content %}
 
 <h1>
 Builder <a href="{{ path_to_builder }}">{{ b.getBuilder().getName() }}</a>
 Build #{{ b.getNumber() }}
 </h1>
 <a href="https://chrome-logs-storage.appspot.com/browser/logs/{{
          b.getProperty('mastername') }}/{{ b.getBuilder().getName() }}/{{ '%.7d'
          % b.getNumber() }}/" title="List of log files cached on Google Cloud
          Storage" class="BuildHeader">Cached logs</a>
 
 <div class="column">
 
 {% if not b.isFinished() %}
   <h2>Build In Progress:</h2>
 
   {% if when_time %}
     <p>ETA: {{ when_time }} [{{ when }}]</p>
   {% endif %}
 
   {{ current_step }}
-  
+
   {% if authz.advertiseAction('stopBuild') %}
     <h2>Stop Build</h2>
     {{ forms.stop_build(build_url+"/stop", authz, on_all=False, short=False, label='This Build') }}
   {% endif %}
 {% else %}
   <h2>Results:</h2>
 
-  <p class="{{ result_css }} result">   
+  <p class="{{ result_css }} result">
     {{ b.getText()|join(' ')|capitalize }}
   </p>
-   
+
   {% if b.getTestResults() %}
     <h3><a href="{{ tests_link }}"/></h3>
   {% endif %}
 {% endif %}
 
 <h2>SourceStamp:</h2>
 
 <table class="info" width="100%">
 {% set ss_class = cycler('alt','') %}
 
@@ skipping 28 matching lines @@
 {% if most_recent_rev_build %}
   <tr class="{{ ss_class.next() }}"><td class="left" colspan="2">Build of most recent revision</td></tr>
 {% endif %}
 
 </table>
 
 {#
  # TODO: turn this into a table, or some other sort of definition-list
  # that doesn't take up quite so much vertical space
  #}
-   
+
 <h2>BuildSlave:</h2>
-  
-{% if slave_url %}  
+
+{% if slave_url %}
   <a href="{{ slave_url|e }}">{{ b.getSlavename()|e }}</a>
 {% else %}
-  {{ b.getSlavename()|e }} 
+  {{ b.getSlavename()|e }}
 {% endif %}
 
 <h2>Reason:</h2>
 <p>
 {{ b.getReason()|e }}
 </p>
 
 <h2>Steps and Logfiles:</h2>
 
 {#
@@ skipping 27 matching lines @@
     </li><li>
   {% elif step_nest < nest_level %}
     {% for _ in range(step_nest, nest_level) %}
     </li></ol></details>
     {% endfor %}
     <li>
   {% endif %}
   {% set nest_level = step_nest %}
 
     <div class="{{ s.css_class }} result">
-      <a href="{{ s.link }}">{{ s.name }}</a> 
+      <a href="{{ s.link }}">{{ s.name }}</a>
       {% set previous_name = s.name %}
       {{ s.text }}&nbsp;<span style="float:right">{{ '( ' + s.time_to_run + ' )' if s.time_to_run else '' }}</span>
     </div>
 
     <ul>
       {% set item_class = cycler('alt', '') %}
       {% for l in s.logs %}
         <li class="{{ item_class.next() }}"><a href="{{ l.link }}">{{ l.name }}</a>
           <a href="https://storage.cloud.google.com/chrome-build-logs-private/logs/{{
                    b.getProperty('mastername') }}/{{ b.getBuilder().getName()
                    }}/{{ '%.7d' % b.getNumber() }}/{{ s.name|replace('/', '_') }}.{{
                    l.name|replace('/', '_') }}"
              title="Raw log stored on Google Cloud Storage (available after up to 5 min after end of build)">cache</a>
 </li>
       {% else %}
         <li class="{{ item_class.next() }}">- no logs -</li>
       {% endfor %}
-    
+
       {% for u in s.urls %}
         <li class="{{ item_class.next() }}"><a href="{{ u.url }}">{{ u.logname }}</a></li>
       {% endfor %}
-    </ul>  
+    </ul>
   </li>
 {% endfor %}
 
 {# Close out all open indentation sections. #}
 {% for _ in range(-1, nest_level) %}
  </li></ol>
 {% endfor %}
 
 </div>
 <div class="column">
 
 <h2>Build Properties:</h2>
 
 <table class="info BuildProperties" width="100%">
 <tr><th>Name</th><th>Value</th><th>Source</th></tr>
 
+{% set rietveld_url = [] %}
+{% for p in properties %}
+  {% if p.name == 'rietveld' and rietveld_url.append(p.value) %}
+  {% endif %}
+{% endfor %}
+
 {% for p in properties %}
   <tr class="{{ loop.cycle('alt', '') }}">
     <td class="left">{{ p.name|e }}</td>
     <td class="middle"><abbr title="
       {% if p.short_value %}
         {{ p.short_value|e }} .. [property value too long]
       {% else %}
         {{ p.value|e }}
       {% endif %}">
     {% if p.short_value %}
       {{ p.short_value|e }} .. [property value too long]
+    {% elif p.name == 'issue' and rietveld_url %}
+      <a style="color: red" href="{{ (rietveld_url[0] + '/' + p.value)|e }} ">{{ p.value|e }}</a>
     {% else %}
       {{ p.value|e }}
     {% endif %}
     </abbr></td>
     <td>{{ p.source|e }}</td>
   </tr>
 {% endfor %}
 
 </table>
 
@@ skipping 20 matching lines @@
 </table>
 
   {% if authz.advertiseAction('forceBuild') %}
     <h3>Resubmit Build:</h3>
     {{ forms.rebuild_build(build_url+"/rebuild", authz, exactly, ss) }}
   {% endif %}
 
 </div>
 
 <br style="clear:both"/>
-  
+
 {% if ss.changes %}
 <div class="column">
   <h2>All Changes:</h2>
   <ol>
   {% for c in ss.changes %}
     <li><h3>Change #{{ c.number }}</h3>
       {{ change(c.asDict()) }}
     </li>
   {% else %}
     <li>no changes</li>
-  {% endfor %}  
+  {% endfor %}
   </ol>
-</div> 
+</div>
 {% endif %}
 
 {% endblock %}