Most PackageBuild._handleStatus* methods attempt to write in a read-only transaction
Bug #905853 reported by
William Grant
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Invalid
|
Critical
|
Unassigned | ||
Bug Description
Spot the difference between the starts of these two methods:
def _handleStatus_
with DatabaseTransac
def _handleStatus_
self.status = BuildStatus.
Most of the failure states don't force a writable transaction, so they crash like <http://
Related branches
lp:~allenap/launchpad/handle-status-ro-crash-bug-905853
- Graham Binns (community): Approve (code)
- Julian Edwards (community): Approve
-
Diff: 1657 lines (+555/-227)16 files modifiedbuildout-templates/bin/retest.in (+11/-2)
lib/lp/archiveuploader/tests/test_uploadprocessor.py (+4/-2)
lib/lp/buildmaster/interfaces/builder.py (+3/-1)
lib/lp/buildmaster/manager.py (+74/-46)
lib/lp/buildmaster/model/builder.py (+28/-15)
lib/lp/buildmaster/model/buildfarmjobbehavior.py (+63/-32)
lib/lp/buildmaster/model/packagebuild.py (+94/-54)
lib/lp/buildmaster/testing.py (+59/-0)
lib/lp/buildmaster/tests/test_builder.py (+25/-2)
lib/lp/buildmaster/tests/test_manager.py (+120/-29)
lib/lp/buildmaster/tests/test_packagebuild.py (+18/-16)
lib/lp/code/model/tests/test_sourcepackagerecipebuild.py (+6/-8)
lib/lp/services/database/tests/test_transaction_policy.py (+14/-4)
lib/lp/services/database/transaction_policy.py (+16/-4)
lib/lp/soyuz/tests/test_binarypackagebuild.py (+7/-4)
lib/lp/translations/model/translationtemplatesbuildbehavior.py (+13/-8)
lp:~allenap/launchpad/undo-read-only-transactions-in-buildmaster
- Gavin Panella (community): Approve
-
Diff: 1620 lines (+251/-470)13 files modifiedlib/lp/archiveuploader/tests/test_uploadprocessor.py (+2/-4)
lib/lp/buildmaster/interfaces/builder.py (+4/-6)
lib/lp/buildmaster/manager.py (+50/-78)
lib/lp/buildmaster/model/builder.py (+15/-28)
lib/lp/buildmaster/model/buildfarmjobbehavior.py (+33/-63)
lib/lp/buildmaster/model/packagebuild.py (+57/-91)
lib/lp/buildmaster/tests/test_builder.py (+7/-27)
lib/lp/buildmaster/tests/test_manager.py (+40/-127)
lib/lp/buildmaster/tests/test_packagebuild.py (+20/-10)
lib/lp/code/model/tests/test_sourcepackagerecipebuild.py (+8/-6)
lib/lp/services/database/transaction_policy.py (+2/-5)
lib/lp/soyuz/tests/test_binarypackagebuild.py (+4/-11)
lib/lp/translations/model/translationtemplatesbuildbehavior.py (+9/-14)
| tags: | added: buildfarm |
| Changed in launchpad: | |
| assignee: | nobody → Gavin Panella (allenap) |
| Changed in launchpad: | |
| status: | Triaged → In Progress |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #1 |
| tags: | added: qa-needstesting |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
| tags: |
added: qa-untestable removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → In Progress |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #2 |
| tags: |
added: qa-needstesting removed: qa-untestable |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
| tags: |
added: qa-bad removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → In Progress |
| Changed in launchpad: | |
| assignee: | Gavin Panella (allenap) → nobody |
| status: | In Progress → Triaged |
Revision history for this message
| William Grant (wgrant) wrote | #3 |
| Changed in launchpad: | |
| status: | Triaged → Invalid |
To post a comment you must log in.
Fixed in stable r14552 <http://