Add webhook support for snaps
Bug #1535826 reported by
Colin Watson
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
High
|
Colin Watson | ||
Bug Description
There are various things that could make use of webhooks for snaps: for example, it seems like the most plausible near-term way to notify the snap store that a build has happened. The webhook infrastructure is mature enough at this point that such a thing is quite easy to add.
Related branches
lp:~cjwatson/launchpad/db-snap-webhooks
- William Grant (community): Approve (code)
- Stuart Bishop: Pending (db) requested
- Launchpad code reviewers: Pending (db) requested
-
Diff: 18 lines (+14/-0)1 file modifieddatabase/schema/patch-2209-69-2.sql (+14/-0)
lp:~cjwatson/launchpad/snap-webhooks
- Celso Providelo (community): Approve
- Launchpad code reviewers: Pending requested
-
Diff: 810 lines (+269/-62)16 files modifieddatabase/schema/security.cfg (+5/-1)
lib/lp/services/webhooks/interfaces.py (+2/-1)
lib/lp/services/webhooks/model.py (+12/-1)
lib/lp/services/webhooks/templates/webhooktarget-webhooks.pt (+2/-2)
lib/lp/services/webhooks/tests/test_browser.py (+55/-5)
lib/lp/services/webhooks/tests/test_model.py (+57/-39)
lib/lp/services/webhooks/tests/test_webservice.py (+12/-1)
lib/lp/snappy/browser/snap.py (+10/-3)
lib/lp/snappy/configure.zcml (+5/-1)
lib/lp/snappy/interfaces/snap.py (+7/-3)
lib/lp/snappy/interfaces/snapbuild.py (+7/-1)
lib/lp/snappy/model/snap.py (+9/-2)
lib/lp/snappy/model/snapbuild.py (+19/-1)
lib/lp/snappy/subscribers/snapbuild.py (+30/-0)
lib/lp/snappy/tests/test_snap.py (+12/-1)
lib/lp/snappy/tests/test_snapbuild.py (+25/-0)
lp:~cjwatson/launchpad/snap-webhooks-permissions
- William Grant (community): Approve (code)
-
Diff: 172 lines (+38/-7)5 files modifieddatabase/schema/security.cfg (+1/-0)
lib/lp/code/model/tests/test_branchmergeproposal.py (+12/-5)
lib/lp/code/model/tests/test_gitjob.py (+11/-0)
lib/lp/codehosting/scanner/tests/test_bzrsync.py (+6/-1)
lib/lp/snappy/tests/test_snapbuild.py (+8/-1)
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #1 |
| tags: | added: qa-needstesting |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
Revision history for this message
| Colin Watson (cjwatson) wrote | #2 |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → In Progress |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #3 |
| tags: |
added: qa-needstesting removed: qa-ok |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
| Changed in launchpad: | |
| status: | Fix Committed → In Progress |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #4 |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Fixed in db-stable r13332 <http://