Merge lp://staging/~cjwatson/launchpad/init-packageset-fixes into lp://staging/launchpad
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | William Grant | ||||
Approved revision: | no longer in the source branch. | ||||
Merged at revision: | 15137 | ||||
Proposed branch: | lp://staging/~cjwatson/launchpad/init-packageset-fixes | ||||
Merge into: | lp://staging/launchpad | ||||
Diff against target: |
99 lines (+53/-9) 2 files modified
lib/lp/soyuz/scripts/initialize_distroseries.py (+3/-5) lib/lp/soyuz/scripts/tests/test_initialize_distroseries.py (+50/-4) |
||||
To merge this branch: | bzr merge lp://staging/~cjwatson/launchpad/init-packageset-fixes | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
William Grant (community) | code | Approve | |
Review via email: mp+103036@code.staging.launchpad.net |
Commit message
Fix InitializeDistr
Description of the change
== Summary ==
Bug 887185 reports that initialize-
== Proposed fix ==
packagesets = self._store.find(
The intention is obvious here, but that code is missing anything relating the Packageset and Distroseries rows, so it ends up just loading all packagesets. Adding the obvious And(Packageset.
== Implementation details ==
+49 LoC, but I'd like to get this landed in time for opening Ubuntu Q and so don't want to spend lots of time refactoring if possible; could I use some credit from r15080 (-232) or r15103 (-93)?
== Tests ==
bin/test -vvct test_initialize
== Demo and Q/A ==
I'm open to suggestions of something that can be done in practice in a reasonable time. My guess would be that we run initialize-
== lint ==
None.
Oh, incidentally, I don't think this entirely closes this bug, as we need to check for existing busted data and do something about it if any, so this is probably [incr]. I'm willing to work on the rest, although probably not immediately.