Merge lp:~cjohnston/qa-dashboard/1086158 into lp:qa-dashboard

Proposed by Chris Johnston
Status: Merged
Approved by: Joe Talbott
Approved revision: 155
Merged at revision: 155
Proposed branch: lp:~cjohnston/qa-dashboard/1086158
Merge into: lp:qa-dashboard
Diff against target: 25 lines (+2/-2)
2 files modified
dashboard/management/commands/jenkins_pull.py (+1/-1)
dashboard/templates/smoke/result_logs.html (+1/-1)
To merge this branch: bzr merge lp:~cjohnston/qa-dashboard/1086158
Reviewer Review Type Date Requested Status
Joe Talbott Approve
Review via email: mp+137821@code.launchpad.net

Commit message

Fixes broken log url's for smoke results

Description of the change

Fixes broken smoke log URLs

------------- This line and the following will be ignored --------------

Source: lp:~chrisjohnston/qa-dashboard/1086158
Target: lp:qa-dashboard

To post a comment you must log in.
Revision history for this message
Joe Talbott (joetalbott) wrote :

LGTM, Thanks.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'dashboard/management/commands/jenkins_pull.py'
2--- dashboard/management/commands/jenkins_pull.py 2012-11-30 15:16:23 +0000
3+++ dashboard/management/commands/jenkins_pull.py 2012-12-04 12:12:21 +0000
4@@ -312,7 +312,7 @@
5 if regexes['artifact-ignore'].match(path):
6 continue
7
8- remote_url = "%s/artifact/%s" % (build_url, path)
9+ remote_url = "%sartifact/%s" % (build_url, path)
10 art_args = {
11 'display_name': path,
12 'remote_url': remote_url,
13
14=== modified file 'dashboard/templates/smoke/result_logs.html'
15--- dashboard/templates/smoke/result_logs.html 2012-11-20 02:23:46 +0000
16+++ dashboard/templates/smoke/result_logs.html 2012-12-04 12:12:21 +0000
17@@ -61,7 +61,7 @@
18 {% endfor %}
19 <tr>
20 <td>
21- <a href='{{result.jenkins_url}}console'>Console Output</a>
22+ <a href='{{result.jenkins_url}}/console'>Console Output</a>
23 </td>
24 </tr>
25 </tbody>

Subscribers

People subscribed via source and target branches