lp:~jameinel/juju-core/rpc-versions

Created by John A Meinel on 2014-05-13 and last modified on 2014-05-13
Get this branch:
bzr branch lp:~jameinel/juju-core/rpc-versions
Only John A Meinel can upload to this branch. If you are John A Meinel please log in for upload directions.

Branch merges

Related bugs

Related blueprints

Branch information

Owner:
John A Meinel
Project:
juju-core
Status:
Development

Recent revisions

2727. By John A Meinel on 2014-05-13

finally we got all the tests passing again.

had to play around a bit to get the right versions in the right place.
But mostly had to change the JSON Codec so that it knows about Version, too.
Because apparently it uses its own types for serialization and deserialization,
though none of the actual content on the wire is checked...

2726. By John A Meinel on 2014-05-13

getting closer, but not 100% tests passing yet

2725. By John A Meinel on 2014-05-13

get the test suite passing.

Now we have Version as an idea inside the code, though we should be able
to simplify some of the Id based late reflection (maybe).

2724. By John A Meinel on 2014-05-13

merge reflect-registry, not quite working yet, but steps along the path

2723. By John A Meinel on 2014-05-13

Start trying to implement a custom MethodCaller. But really, we still want 'Id' to be looked up early rather than late. :(

2722. By John A Meinel on 2014-05-13

Allow MethodCaller to take a Version, but the simple reflect.Value based interface doesn't use it.

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

Expose Version as a new attribute of the RPC Request structure.

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

[r=jameinel],[bug=1301353] testing/mgo: use /usr/lib/juju/bin/mongod

As part of fixing bug #130135, the test suite can now search PATH for
'mongod' and if it doesn't find it, also search for
'/usr/lib/juju/bin/mongod' and disable the store's JS tests.

I came across some other parts of the codebase that were panic() ing
when mongod failed to start, so I fixed that as well.

https://codereview.appspot.com/100380044/

2719. By Menno Finlay-Smits on 2014-05-12

[r=menno.smits] juju/cmd: use RunCommand throughout BootstrapSuite

Replaced uses of cmd.Main with with the RunCommand test helper in
the bootstrap command tests to avoids test boilerplate and make
checking of test results more straightforward.

https://codereview.appspot.com/97330044/

R=thumper

2718. By Jesse Meek on 2014-05-12

[r=waigani],[bug=1318429] Correct debug-log help

Change help message for exclude to read:
"do not show log messages for these entities"

https://codereview.appspot.com/91380044/
R=axwalk

Branch metadata

Branch format:
Branch format 7
Repository format:
Bazaar repository format 2a (needs bzr 1.16 or later)
Stacked on:
lp:~go-bot/juju-core/trunk
This branch contains Public information 
Everyone can see this information.

Subscribers