Ubuntu CI Dashboard

Merge lp://qastaging/~joetalbott/qa-dashboard/memory_blank_proc_name into lp://qastaging/qa-dashboard

  1. memory_blank_proc_name
  2. Merge into dev
Proposed by Joe Talbott
Status: Merged
Approved by: Chris Johnston
Approved revision: 738
Merged at revision: 739
Proposed branch: lp://qastaging/~joetalbott/qa-dashboard/memory_blank_proc_name
Merge into: lp://qastaging/qa-dashboard
Diff against target: 26 lines (+4/-1)
2 files modified
common/management/__init__.py (+1/-1)
memory/management/commands/jenkins_pull_memory.py (+3/-0)
To merge this branch: bzr merge lp://qastaging/~joetalbott/qa-dashboard/memory_blank_proc_name
Reviewer Review Type Date Requested Status
Andy Doan (community) Approve
PS Jenkins bot continuous-integration Approve
Review via email: mp+217638@code.qastaging.launchpad.net

Commit message

memory - Don't blow up on empty process names.

Description of the change

memory - Don't blow up on empty process names.

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

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

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

review: Approve (continuous-integration)
lp://qastaging/~joetalbott/qa-dashboard/memory_blank_proc_name updated
738. By Joe Talbot <email address hidden>

memory - Don't even add unnamed processes.

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

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

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

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

Looks fine - is there some bigger issue in the actual test that's causing this to be empty?

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

On Wed, Apr 30, 2014 at 10:39 AM, Andy Doan <email address hidden>wrote:

> Looks fine - is there some bigger issue in the actual test that's causing
> this to be empty?

Yes

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