lp:~jameinel/juju-core/reflect-only-facades

Branch merges

Propose for merging

No branches dependent on this one.

On hold for merging into lp:~go-bot/juju-core/trunk

Juju Engineering: Pending requested 2014-05-08

Diff: 652 lines (+316/-90)

6 files modified

rpc/reflect_test.go (+36/-20)
rpc/rpc_test.go (+208/-19)
rpc/rpcreflect/type.go (+13/-6)
rpc/rpcreflect/value.go (+33/-23)
rpc/server.go (+6/-4)
state/apiserver/root_test.go (+20/-18)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Related blueprints

2720. By John A Meinel on 2014-05-12

Merge trunk to get bugfix for bug #1301353

This lets the test suite run on Trusty with juju-mongodb.

2719. By John A Meinel on 2014-05-12

don't refer to the now unused package

2718. By John A Meinel on 2014-05-11

Remove the test that would fail if we just merged reflect-only-facades

2717. By John A Meinel on 2014-05-10

remove the unused attributes of MethodCaller

2716. By John A Meinel on 2014-05-10

We don't need the rootMethod of MethodCaller anymore

2715. By John A Meinel on 2014-05-08

We don't use the Method.ObjType anymore, so we can get rid of it.

2714. By John A Meinel on 2014-05-08

Start cleaning up the test suite.

2713. By John A Meinel on 2014-05-08

tweak the spelling for unknown versions

2712. By John A Meinel on 2014-05-08

MethodCaller.Call doesn't need the Id anymore.

We've already resolved the concrete object, cache it and use it
rather than looking it up a second time.

2711. By John A Meinel on 2014-05-08

Since now methods may exist on different versions of the Interface, we need to put more detail into the error message.

For backwards compatibility if id = "" we omit that portion.