~bac/juju-core/add-config : changes

~bac/juju-core/add-config » Changes from revision 906

From Revision 902 to 883

Rev	Summary	Authors	Date
902	testing/charms: Remove the series parameter testing/charms: Remove the series parameter This change removes some of the symlinks in the working tree (testing/repo/*). Following on to it, there were a bunch of apis in testing.Charms that took a series parameter, but all callers were just passing "series" anyway. All the tests still pass, so I'm pretty happy with the changes. R= CC= https://codereview.appspot.com/7301073	John A Meinel	11 years ago
901	state/api: use reliable test state/api: use reliable test When we Stop the server, the client does not get immediate notification, so we can't guarantee an immediate ErrShutdown response. Also make errors more consistent in rpc. R=dimitern, dfc CC= https://codereview.appspot.com/7310095	Roger Peppe	11 years ago
900	worker/uniter/jujuc: log callbacks worker/uniter/jujuc: log callbacks It's useful to see the actual commands that a hook makes back to the uniter when debugging. R=fwereade CC= https://codereview.appspot.com/7317050	Roger Peppe	11 years ago
899	local: remove files of old approach local: remove files of old approach First of a number of CLs to push the so far developed parts. This one only removes the files of an old approach that aren't needed anymore. R=fwereade, jameinel CC= https://codereview.appspot.com/7326048	Frank Mueller	11 years ago
898	state: trivial Constraints state: trivial Constraints This does not necessarily reflect the current consensus re what we should provide, but at least it's something concrete to argue about. R=dfc, TheMue, rog, jameinel, dimitern CC= https://codereview.appspot.com/7324049	William Reade	11 years ago
897	charm: Fix bug #864164 - non-exec hooks charm: Fix bug #864164 - non-exec hooks Changed ExpandTo() and BundleTo() to set executable permissions while processing charm hook files. In the case of BundleTo(), as mentioned in the bug, a warning is issued to the log about this, while in ExpandTo() it happens silently. Permissions of hooks are set executable by owner. Added tests and a testing charm with all hooks defined. R=fwereade, rog CC= https://codereview.appspot.com/7305096	Dimiter Naydenov	11 years ago
896	Refactor charm/hooks,worker/uniter/hook Refactor charm/hooks,worker/uniter/hook As discussed, this splits worker/uniter/hook module into uniter-related stuff (Info and validation) and generic charm hooks stuff (defined hooks kinds). Adds UnitHooks and RelationHooks func, which return their respective Kinds as slices. Changed uniter code to match the code split. Added a Meta method Hooks() to return map[string]bool of all (possible) hooks according to the metadata. This is needed for a follow-up CL to fix bug #864164. R=fwereade, rog CC= https://codereview.appspot.com/7317045	Dimiter Naydenov	11 years ago
895	store: reinstate store package store: reinstate store package * revert rev 886 * update testing to new Charm api R= CC= https://codereview.appspot.com/7346043	Dave Cheney	11 years ago
894	rpc: add support for error codes rpc: add support for error codes We need some way of transporting known kinds of error across the link. This seems better than regexp matches on error messages. R=dimitern, fwereade CC= https://codereview.appspot.com/7311097	Roger Peppe	11 years ago
893	rpc, state/api: consistent errors. rpc, state/api: consistent errors. It helps to close the right thing, and there's probably no point in sending a message to the server when it's about to be closed. Not easy to test, unfortunately, as triggering the different error message involves catching the input goroutine in just the right place. R= CC= https://codereview.appspot.com/7324054	Roger Peppe	11 years ago
892	state: fix validity checks in AuthEntity state: fix validity checks in AuthEntity R=rog CC= https://codereview.appspot.com/7312099	William Reade	11 years ago
891	Get mongo from provider storage Get mongo from provider storage Instead of hard coding the location of mongo to be an EC2 public bucket, allow mongo to be uploaded to the specific provider's private or public storage. Use the EC2 location only as a fallback. This change means that we can have an Openstack provider for which the tools and mongo can all be downloaded from the same provider container, with no dependency on EC2, instead of the tools being somwhere and mongo somewhere else. R=jameinel, dimitern CC= https://codereview.appspot.com/7307108	Ian Booth	11 years ago
890	openstack: Basic ports support openstack: Basic ports support Implement opening and closing of ports for the openstack provider using security groups. This is pretty much copy-code from the existing ec2 version. R=dimitern, jameinel https://codereview.appspot.com/7314053/	Martin Packman	11 years ago
889	rpc: always use structs rpc: always use structs Go 1.0.2 is more strict about json unmarshalling from nil, so this patch makes the rpc package strict about requiring structs for all parameters and results. R=dimitern CC= https://codereview.appspot.com/7324048	Roger Peppe	11 years ago
888	rpc: don't hang tests on failure rpc: don't hang tests on failure As suggested in previous CL. Also delete Accept method, also suggested earlier and not done. R=dimitern, fwereade CC= https://codereview.appspot.com/7324046	Roger Peppe	11 years ago
887	testing/charms: Remove the series parameter testing/charms: Remove the series parameter This change removes some of the symlinks in the working tree (testing/repo/*). Following on to it, there were a bunch of apis in testing.Charms that took a series parameter, but all callers were just passing "series" anyway. All the tests still pass, so I'm pretty happy with the changes. R=dimitern, gz CC= https://codereview.appspot.com/7301073	John A Meinel	11 years ago
886	store, cmd/charm: Move them into lp:juju-store store, cmd/charm: Move them into lp:juju-store Break out store and cmd/charm* into lp:juju-store Red Squad are going to be working on the charm store, and it was suggested that an early task would be to split the charm store out into a separate project. That work has already been done - see lp:juju-store - and this is the clean-up job. mgz helped me a lot in doing both these tasks. Fwiw, juju-store has not been advertised, so feel free to suggest a different name. R=fwereade CC= https://codereview.appspot.com/7058063/	Gavin Panella	11 years ago
885	rpc: allow concurrent calls rpc: allow concurrent calls The client side comes substantially from the net/rpc package (no point in reinventing the same wheel); the server side is a little simpler, I hope. R=fwereade CC= https://codereview.appspot.com/7307090	Roger Peppe	11 years ago
884	state/api: enable password checking state/api: enable password checking R=TheMue, dimitern, fwereade CC= https://codereview.appspot.com/7299066	Roger Peppe	11 years ago
883	rpc: new package rpc: new package The rpc package is styled after the standard Go rpc package, but is a little more flexible and closer to our requirements. We also change state/api to use this package, so there's an idea of how it will look in practice. R=dimitern, TheMue, fwereade CC= https://codereview.appspot.com/6878052	Roger Peppe	11 years ago

Older »