Ubuntu CI Dashboard

Merge lp://staging/~doanac/qa-dashboard/smoke-multi-pull into lp://staging/qa-dashboard

  1. smoke-multi-pull
  2. Merge into dev
Proposed by Andy Doan
Status: Merged
Merged at revision: 697
Proposed branch: lp://staging/~doanac/qa-dashboard/smoke-multi-pull
Merge into: lp://staging/qa-dashboard
Prerequisite: lp://staging/~doanac/qa-dashboard/smoke-pull-test
Diff against target: 102 lines (+23/-11)
2 files modified
smokeng/management/commands/jenkins_pull_smokeng.py (+13/-7)
smokeng/tests.py (+10/-4)
To merge this branch: bzr merge lp://staging/~doanac/qa-dashboard/smoke-multi-pull
Reviewer Review Type Date Requested Status
Joe Talbott Approve
PS Jenkins bot continuous-integration Approve
Review via email: mp+197642@code.staging.launchpad.net

Commit message

add ability to parse multiple tests from a single job

This adds the ability to get test results from a new naming
pattern in jenkins:

  clientlogs/<test name>/utah.yaml

Description of the change

add ability to parse multiple tests from a single job

This adds the ability to get test results from a new naming
pattern in jenkins:

  clientlogs/<test name>/utah.yaml

This change along with the live-status allow us to enable mega-jobs in CI

To post a comment you must log in.
Revision history for this message
Andy Doan (doanac) wrote :

I've done some local testing as best I know how and also did some pulls with "-a" and "-d 4" to try and make sure I'm proving both my new change works and it doesn't break old stuff. Let me know if there's more testing I should do, or try it yourself and let me know.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:698
http://s-jenkins.ubuntu-ci:8080/job/dashboard-ci/272/
Executed test runs:

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/dashboard-ci/272/rebuild

review: Approve (continuous-integration)
Revision history for this message
Joe Talbott (joetalbott) wrote :

I'm getting conflicts when merging this with lp:qa-dashboard. It looks like the prerequisite MP is mixed in even though it was merged already.

review: Needs Fixing
Revision history for this message
Andy Doan (doanac) wrote :

not sure what went wrong, but I fixed the conflicts. should be ready now

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

Tarmac doesn't like this.. unapproving for now to investigate but unbreak tarmac

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

Andy, tarmac complains about a ghost commit with this. Any ideas what you might have done to cause problems?

Revision history for this message
Andy Doan (doanac) wrote :

On 12/09/2013 01:16 PM, Chris Johnston wrote:
> Andy, tarmac complains about a ghost commit with this. Any ideas what you might have done to cause problems?
>
 I have no idea. Should I just try and merge it manually to see what
happens and how to fix?

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