Daisy

Merge lp://staging/~brian-murray/daisy/changed-bucketsystems into lp://staging/daisy

changed-bucketsystems
Merge into trunk

Proposed by Brian Murray on 2013-06-07

Status:	Merged
Merged at revision:	367
Proposed branch:	lp://staging/~brian-murray/daisy/changed-bucketsystems
Merge into:	lp://staging/daisy
Diff against target:	95 lines (+23/-20) 2 files modified daisy/utils.py (+4/-12) tools/build_bucketsystems.py (+19/-8)
To merge this branch:	bzr merge lp://staging/~brian-murray/daisy/changed-bucketsystems
Related bugs:	Link a bug report

Reviewer	Review Type	Date Requested	Status
Daisy Pluckers		2013-06-07	Pending
Review via email: mp+168212@code.staging.launchpad.net

Description of the change

See https://code.launchpad.net/~brian-murray/oops-repository/updated-bucket-systems/+merge/168211 for the reasoning behind this change.

This also contains an update to the backfill of the bucketsystems column family.

Revision history for this message

Evan (ev) wrote on 2013-06-10:

On 7 June 2013 23:30, Brian Murray <email address hidden> wrote:
> + if version and version != '':
> + oopses.update_bucket_systems(oops_config, crash_signature, version,
> + system_uuid)

"if version" is sufficient here. An empty string in Python evaluates
to boolean false.

The rest looks good.

lp://staging/~brian-murray/daisy/changed-bucketsystems updated on 2013-06-10