Merge lp://staging/~cjohnston/qa-dashboard/bootspeed-max-length into lp://staging/qa-dashboard

Proposed by Chris Johnston
Status: Merged
Merged at revision: 652
Proposed branch: lp://staging/~cjohnston/qa-dashboard/bootspeed-max-length
Merge into: lp://staging/qa-dashboard
Diff against target: 310 lines (+259/-13)
2 files modified
bootspeed/migrations/0007_increase_max_length.py (+246/-0)
bootspeed/models.py (+13/-13)
To merge this branch: bzr merge lp://staging/~cjohnston/qa-dashboard/bootspeed-max-length
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
Joe Talbott Approve
Chris Johnston Needs Resubmitting
Andy Doan (community) Needs Fixing
Review via email: mp+192152@code.staging.launchpad.net

This proposal supersedes a proposal from 2013-10-21.

Commit message

Increase the max_length of bootspeed model fields

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal

PASSED: Continuous integration, rev:645
http://10.97.0.26:8080/job/dashboard-ci/204/
Executed test runs:

Click here to trigger a rebuild:
http://10.97.0.26:8080/job/dashboard-ci/204/rebuild

review: Approve (continuous-integration)
Revision history for this message
Andy Doan (doanac) wrote :

typo:

292 + display_name = models.CharField(max_length=40965, null=True)

review: Needs Fixing
Revision history for this message
Joe Talbott (joetalbott) wrote :

L292 has a typo I think

review: Needs Fixing
Revision history for this message
Chris Johnston (cjohnston) :
review: Needs Resubmitting
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:645
http://10.97.0.26:8080/job/dashboard-ci/218/
Executed test runs:

Click here to trigger a rebuild:
http://10.97.0.26:8080/job/dashboard-ci/218/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:646
http://10.97.0.26:8080/job/dashboard-ci/220/
Executed test runs:

Click here to trigger a rebuild:
http://10.97.0.26:8080/job/dashboard-ci/220/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:645
http://10.97.0.26:8080/job/dashboard-ci/222/
Executed test runs:

Click here to trigger a rebuild:
http://10.97.0.26:8080/job/dashboard-ci/222/rebuild

review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:646
http://10.97.0.26:8080/job/dashboard-ci/223/
Executed test runs:

Click here to trigger a rebuild:
http://10.97.0.26:8080/job/dashboard-ci/223/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Joe Talbott (joetalbott) :
review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:646
http://10.97.0.26:8080/job/dashboard-ci/225/
Executed test runs:

Click here to trigger a rebuild:
http://10.97.0.26:8080/job/dashboard-ci/225/rebuild

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

The attempt to merge lp:~cjohnston/qa-dashboard/bootspeed-max-length 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
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches