Method calls over AMP should transparently handle deferred results
Bug #499225 reported by
Free Ekanayaka
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Landscape Client |
Fix Released
|
Low
|
Free Ekanayaka | ||
Bug Description
The MethodCallProtocol introduced in Bug #499018 can't handle remote
methods returing Deferred objects.
The attached branch makes the protocol return a MethodCall uuid to the
peer, in these cases. A further command referencing such uuid will be
sent to the caller as soon as the Deferred returned by the object's
method callbacks, along with the result of the Deferred itself.
Related branches
lp:~free.ekanayaka/landscape-client/method-call-defer
- Kapil Thangavelu (community): Approve
- Duncan McGreggor (community): Approve
-
Diff: 412 lines (+227/-30)2 files modifiedlandscape/lib/amp.py (+123/-13)
landscape/lib/tests/test_amp.py (+104/-17)
| description: | updated |
| summary: |
- Add a decorator to easily create methods that send AMP commands + Method calls over AMP should transparently handle deferred results |
| Changed in landscape-client: | |
| milestone: | 1.4.3 → 1.4.4 |
| Changed in landscape-client: | |
| status: | In Progress → Fix Committed |
| tags: | added: needs-testing |
| Changed in landscape-client: | |
| status: | Fix Committed → Fix Released |
| Changed in landscape-client: | |
| status: | Fix Released → Fix Committed |
| Changed in landscape-client: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
This is not easy to test in staging, removing the needs-testing tag.