~axwalk/juju-core/lp1260171-stopinstances-ids

From Revision 2724 to 2705

expand all

collapse all

Rev		Summary	Authors	Tags	Date	Diff	Files
2724		Make StopInstances variadic Make StopInstances variadic	Andrew Wilkins		9 years ago
2723		Change StopInstances to take []instance.Id Change StopInstances to take []instance.Id StopInstances previously took []instance.Instance, which is unnecessary for most providers, and imposes an unnecessary cost for callers that only have IDs initially. We change StopInstances to  The openstack provider's StopInstances is now slightly more expensive than before, requiring an additional call to Instances. This cost will disappear when we fix the security groups/open-port implementation. Drive-by: the maas provider now uses a bulk "release" in StopInstances. Live tested on ec2, local, virtual MAAS, canonistack, azure. I lack a Joyent account, but I think it should be fine. Fixes lp:1260171 Fixes lp:1316272	Andrew Wilkins		9 years ago
2722		[r=axwalk],[bug=1317197] Take hook exec lock in container ho... [r=axwalk],[bug=1317197] Take hook exec lock in container host init When initialising the host for a container type, we will install packages (lxc, kvm, etc.); this can conflict with hook execution. This CL modifies the container initialisation code to acquire the hook execution lock when initialisting the host to avoid the conflict. Fixes lp:1317197 https://codereview.appspot.com/91350044/	Andrew Wilkins		9 years ago
2721		[r=axwalk] Invert envcmd relationship [r=axwalk] Invert envcmd relationship Previously we embedded EnvCommandBase in all commands requring an environment, and EnvCommandBase included everything (SetFlags, and Init). The problem with this is that it was easy to miss initialisation of the EnvCommandBase type (this happened a few times), which leads to bad things happening like sync-tools destroying environments. I have inverted the relationship so that we now have envcmd.EnvironCommand, an interface that extends Command with a SetEnvName method, and EnvCommandBase, which implements EnvironCommand. A new method, envcmd.Wrap takes an EnvironCommand and creates a Command that calls SetEnvName prior to the wrapped method's Init method. If the environment name cannot be determined, the wrapping command will error out early. https://codereview.appspot.com/94350045/	Andrew Wilkins		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
2707		[r=jameinel] Merge 1.18 into trunk. [r=jameinel] Merge 1.18 into trunk. There were a bunch of conflicts from things that had to be backported but in a slightly different way. I verified that all the conflicts could be resolved in favor of current code. There was one patch that wasn't in trunk at all, but according to the bug "didn't apply to trunk". However, I think at least one of the assertions (bootstrap machine has its addresses set) does still apply, so I kept that bit.	Ian Booth		9 years ago
2706		[r=menno.smits],[bug=1302005] cmd/juju: deprecate --series, ... [r=menno.smits],[bug=1302005] cmd/juju: deprecate --series, added --upload-series Emit a deprecation warning to stderr when --series is used    To make testing easier an interface is now used to allow the calls made by BootstrapCommand.Run to environs/bootstrap to be mocked out. https://codereview.appspot.com/94060044/ R=thumper	Menno Smits		9 years ago
2705		[r=dimitern],[bug=1316762] maas: Fix bug #1316762 - modprobe... [r=dimitern],[bug=1316762] maas: Fix bug #1316762 - modprobe 8021q if needed Conditionally modprobe 8021q driver used for VLAN support, only if it's not already loaded. This is needed to enable the cloud-installer to bootstrap inside an lxc container, or deploy an lxc container at all for that matter (it was broken). Live tested on MAAS with precise and trusty. https://codereview.appspot.com/93130043/ R=natefinch	Dimiter Naydenov		9 years ago

• Older »

Loggerhead is a web-based interface for Breezy
Version: 2.0.1