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

~jameinel/juju-core/rpc-versions » Changes from revision 2727

From Revision 2727 to 2708

Rev	Summary	Authors	Tags	Date
2727	finally we got all the tests passing again. 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...	John Arbash Meinel		9 years ago
2726	getting closer, but not 100% tests passing yet getting closer, but not 100% tests passing yet	John Arbash Meinel		9 years ago
2725	get the test suite passing. 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).	John Arbash Meinel		9 years ago
2724	merge reflect-registry, not quite working yet, but steps alo... merge reflect-registry, not quite working yet, but steps along the path	John Arbash Meinel		9 years ago
2723	Start trying to implement a custom MethodCaller. But really,... Start trying to implement a custom MethodCaller. But really, we still want 'Id' to be looked up early rather than late. :(	John Arbash Meinel		9 years ago
2722	Allow MethodCaller to take a Version, but the simple reflect... Allow MethodCaller to take a Version, but the simple reflect.Value based interface doesn't use it.	John Arbash Meinel		9 years ago
2721	Expose Version as a new attribute of the RPC Request structu... Expose Version as a new attribute of the RPC Request structure.	John Arbash Meinel		9 years ago
2720	[r=jameinel],[bug=1301353] testing/mgo: use /usr/lib/juju/bi... [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/	John Arbash Meinel		9 years ago
2719	[r=menno.smits] juju/cmd: use RunCommand throughout Bootstra... [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	Menno Smits		9 years ago
2718	[r=waigani],[bug=1318429] Correct debug-log help [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	Jesse Meek		9 years ago
2717	[r=waigani],[bug=1299120] Fix 1299120 command aliases [r=waigani],[bug=1299120] Fix 1299120 command aliases Fix bug 1299120 by ensuring that the preferred command is not displayed as an alias of another command in juju help commands. Change file and function names to be consistant with the preferred command. https://codereview.appspot.com/92080046/ R=fwereade	Jesse Meek		9 years ago
2716	[r=sinzui] Increment juju devel to 1.19.3. [r=sinzui] Increment juju devel to 1.19.3. Update juju version and win installer to 1.19.3. https://codereview.appspot.com/94350044/	Curtis Hovey		9 years ago
2715	[r=wallyworld],[bug=1316869] Merge 1.18 into trunk [r=wallyworld],[bug=1316869] Merge 1.18 into trunk This primarily is to pick up the work done in 1.18 to fix lp:1316869. It also keeps trunk in sync with 1.18	Ian Booth	juju-1.19.2	9 years ago
2714	[r=axwalk],[bug=1313785] cmd/juju: check use-proxy before re... [r=axwalk],[bug=1313785] cmd/juju: check use-proxy before resolving address We were translating machin/unit IDs to internal addresses even if use-proxy=false. This manifested most prominently when attempting to juju ssh to a machine or unit in a 1.18 environment. Fixes lp:1313785 https://codereview.appspot.com/97230043/	Andrew Wilkins		9 years ago
2713	[r=wallyworld],[bug=1314686] Split out Ubuntu series lookup [r=wallyworld],[bug=1314686] Split out Ubuntu series lookup Support for querying Ubuntu series info was in the simplestreams package. We needed to add support for "utopic" but it wasn't obvious where to do it. A previous attempt was made but only a test constant was changed. This branch extracts the series stuff into a new juju/ubuntu package and adds proper "utopic" support. We may well need to look at this again when we want to add supported series support for other OS's but for now this should be good enough. https://codereview.appspot.com/92160043/	Ian Booth		9 years ago
2712	[r=axwalk],[bug=1315216] Introduce lxc-use-clone, ContainerM... [r=axwalk],[bug=1315216] Introduce lxc-use-clone, ContainerManagerConfig A new lxc-use-clone config attribute and a new ContainerManagerConfig provisioner API are introduced. If lxc-use-clone is set to true, then the LXC provisioner will be configured to use cloning regardless of provider type. This setting is currently immutable; additional work will be required to support reacting to changes and reconfiguring the provisioner. Fixes lp:1315216 https://codereview.appspot.com/96140044/	Andrew Wilkins		9 years ago
2711	[r=thumper] Add some developer docs for writing tests. [r=thumper] Add some developer docs for writing tests. Obviously there is a lot that could be written, but this is a start. https://codereview.appspot.com/96110043/	Tim Penhey		9 years ago
2710	[r=jameinel] bzr: fix a test about paths [r=jameinel] bzr: fix a test about paths When bzr looks up the root of a branch, it expands symlinks to get the 'canonical' path. On Mac OS X, /tmp and /var are actually symlinks to /private/tmp and /private/var which broke the tests. So just EvalSymlinks on the path and it works again. https://codereview.appspot.com/98040043/	John Arbash Meinel		9 years ago
2709	[r=jameinel] version/version.go: refactor to allow series on... [r=jameinel] version/version.go: refactor to allow series on Mac This pulls out a bunch of code for determining the series of the current host and makes it possible to implement platform specific calls. It then exposes Mac platform as "darwin13" which matches the $OSTYPE environment variable. Most of the code is written so that it can be tested on all platforms, and only the last little connection is done as platform specific code. I don't know if we plan on doing OS Version specific binaries for Mac/Win like we do for Ubuntu (trusty vs precise), but I figured I could start with the specific version, and we can decide if we want to do version agnostic later. ('darwin' vs 'darwin13') I know we intend that the 'juju' client is supported on Mac, and this is a step towards getting the test suite to pass there. Right now the first test that wants 'tools' fails because the tools version is "unknown", even though all of the code builds successfully.	John Arbash Meinel		9 years ago
2708	[r=axwalk] environs/manual: use old AddMAchines API [r=axwalk] environs/manual: use old AddMAchines API Manual provisioning was implicitly changed to use the new AddMachinesV2 client API, with no backwards compatibility failover. I've changed it to explicitly use the old API, since we do not require anything of the new one. Fixes lp:1317267 https://codereview.appspot.com/95150043/	Andrew Wilkins		9 years ago

Older »