Merge lp:~cjwatson/launchpad/process-job-source-groups-locking into lp:launchpad
Status: | Merged |
---|---|
Approved by: | Colin Watson |
Approved revision: | no longer in the source branch. |
Merged at revision: | 15713 |
Proposed branch: | lp:~cjwatson/launchpad/process-job-source-groups-locking |
Merge into: | lp:launchpad |
Diff against target: |
53 lines (+9/-9) 2 files modified
cronscripts/process-job-source-groups.py (+3/-1) lib/lp/registry/tests/test_process_job_sources_cronjob.py (+6/-8) |
To merge this branch: | bzr merge lp:~cjwatson/launchpad/process-job-source-groups-locking |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Benji York (community) | code | Approve | |
Review via email:
|
Commit message
Stop process-
Description of the change
== Summary ==
Bug 839659: we can't run process-
== Proposed fix ==
It really doesn't need its own lock, as it just runs process-job-source which locks anyway, so drop it.
== Tests ==
bin/test -vvct test_process_
== Demo and Q/A ==
Run 'cronscripts/
== Lint ==
Just the usual false positive for scripts.
./cronscripts/
10: '_pythonpath' imported but unused
Looks good.