Merge lp:~joetalbott/qa-dashboard/fix_for_15.04 into lp:qa-dashboard

Proposed by Joe Talbott
Status: Merged
Approved by: Joe Talbott
Approved revision: 769
Merged at revision: 769
Proposed branch: lp:~joetalbott/qa-dashboard/fix_for_15.04
Merge into: lp:qa-dashboard
Diff against target: 58 lines (+6/-6)
2 files modified
qa_dashboard/settings_base.py (+2/-2)
smokeng/urls.py (+4/-4)
To merge this branch: bzr merge lp:~joetalbott/qa-dashboard/fix_for_15.04
Reviewer Review Type Date Requested Status
Para Siva (community) Approve
Paul Larson Approve
Review via email: mp+261250@code.launchpad.net

Commit message

  Fix for snappy stable results (i.e. core_15.04_edge)

  The dot was not being accepted by the url regex.
  Also fix jenkins url.

Description of the change

  Fix for snappy stable results (i.e. core_15.04_edge)

  The dot was not being accepted by the url regex.
  Also fix jenkins url.

To post a comment you must log in.
Revision history for this message
Paul Larson (pwlars) wrote :

Interesting, I didn't know the statsd server had changed. I trust the jenkins url change isn't going to affect the stuff we have running on dev-jenkins becuase that's a different deployment? +1 otherwise

review: Approve
Revision history for this message
Para Siva (psivaa) wrote :

+1, Thanks for the fix

review: Approve
Revision history for this message
Chris Johnston (cjohnston) wrote :

The attempt to merge lp:~joetalbott/qa-dashboard/fix_for_15.04 into lp:qa-dashboard failed. Below is the output from the failed tests.

Creating test database for alias 'default'...
Type 'yes' if you would like to try deleting the test database 'test_qa_dashboard', or 'no' to cancel:
Got an error creating the test database: database "test_qa_dashboard" already exists

EOFError: EOF when reading a line

Revision history for this message
Chris Johnston (cjohnston) wrote :

The attempt to merge lp:~joetalbott/qa-dashboard/fix_for_15.04 into lp:qa-dashboard failed. Below is the output from the failed tests.

Creating test database for alias 'default'...
Type 'yes' if you would like to try deleting the test database 'test_qa_dashboard', or 'no' to cancel:
Got an error creating the test database: database "test_qa_dashboard" already exists

EOFError: EOF when reading a line

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'qa_dashboard/settings_base.py'
2--- qa_dashboard/settings_base.py 2015-03-27 19:39:12 +0000
3+++ qa_dashboard/settings_base.py 2015-06-05 16:42:40 +0000
4@@ -39,7 +39,7 @@
5 PROJECT_PATH = os.path.dirname(os.path.abspath(__file__))
6 LAUNCHPADLIB_CACHE = ".cache/launchpadlib"
7
8-JENKINS_URL = "http://dev-jenkins.ubuntu-ci:8080/"
9+JENKINS_URL = "http://jenkins.qa.ubuntu.com"
10
11 MANAGERS = ADMINS
12
13@@ -323,7 +323,7 @@
14 # txstatsd config
15
16 TXSTATSD_ENABLED = False
17-TXSTATSD_HOST = "snakefruit.canonical.com"
18+TXSTATSD_HOST = "asrai.canonical.com"
19 TXSTATSD_PORT = 10041
20
21 # Use local settings if available
22
23=== modified file 'smokeng/urls.py'
24--- smokeng/urls.py 2013-11-21 17:46:02 +0000
25+++ smokeng/urls.py 2015-06-05 16:42:40 +0000
26@@ -22,12 +22,12 @@
27 'smokeng.views',
28 url(r'^api/', include(v1_api.urls)),
29 url(
30- r'^(?:(?P<release>\w+)/)?(?:(?P<variant>\w+)/)?$',
31+ r'^(?:(?P<release>\w+)/)?(?:(?P<variant>(\w|\.)+)/)?$',
32 'overview',
33 name='smokeng_overview',
34 ),
35 url(
36- r'^(?P<release>\w+)/(?P<variant>\w+)/(?P<arch>\w+)'
37+ r'^(?P<release>\w+)/(?P<variant>(\w|\.)+)/(?P<arch>\w+)'
38 '/(?P<build_number>'
39 '(\d+:\d+(\.\d+|):\d+(\.\d+|)|\d+(\.\d+|( |%20)\?|)))'
40 '/(?P<image_id>\d+)/$',
41@@ -35,7 +35,7 @@
42 name='smokeng_image_overview',
43 ),
44 url(
45- r'^(?P<release>\w+)/(?P<variant>\w+)/(?P<arch>\w+)'
46+ r'^(?P<release>\w+)/(?P<variant>(\w|\.)+)/(?P<arch>\w+)'
47 '/(?P<build_number>'
48 '(\d+:\d+(\.\d+|):\d+(\.\d+|)|\d+(\.\d+|( |%20)\?|)))'
49 '/(?P<image_id>\d+)/(?P<name>.*)/(?P<id>\d+)/$',
50@@ -43,7 +43,7 @@
51 name='smokeng_test_result_detail',
52 ),
53 url(
54- r'^(?P<release>\w+)/(?P<variant>\w+)/(?P<arch>\w+)'
55+ r'^(?P<release>\w+)/(?P<variant>(\w|\.)+)/(?P<arch>\w+)'
56 '/(?P<build_number>'
57 '(\d+:\d+(\.\d+|):\d+(\.\d+|)|\d+(\.\d+|( |%20)\?|)))'
58 '/(?P<image_id>\d+)/(?P<name>.*)/$',

Subscribers

People subscribed via source and target branches