~thumper/juju-core/local-provider-aufs-default-off : changes

~thumper/juju-core/local-provider-aufs-default-off » Changes from revision 2435

From Revision 2435 to 2416

Rev	Summary	Authors	Tags	Date
2435	Default aufs to off, and make the params bools. Default aufs to off, and make the params bools.	Tim Penhey		10 years ago
2434	[r=thumper] Add flags to link libgo statically on power [r=thumper] Add flags to link libgo statically on power Will also be used for arm64 most likely. We want developers to have jujud work properly for the local provider. My suggesting using 'make install' we can ensure that jujud is build with libgo statically linked. https://codereview.appspot.com/77200043/	Tim Penhey		10 years ago
2433	[r=waigani] Update and Validate Environ Config in State [r=waigani] Update and Validate Environ Config in State The main business logic happens in state/state.go. The bulk of the other changes are mechanical i.e. updating method calls. There are a few things going on: -Explicitly add/update or remove attributes with state.UpdateEnvironConf, which replaces state.SetEnvironConf. - Add ConfigValidator to state.Policy interface. - Validate environment configuration within state. - build new config from UpdateEnvironConf args, get old config from state. - pass both configs to st.validate which use the environ policy ConfigValidator to get the correct validation method. - use the validated config to make the actual updates in state. - Update or delete tests that rely on an invalid configuration being set, as this is now not possible.	Jesse Meek		10 years ago
2432	[r=thumper],[bug=1292346] Add local options lxc-clone and lx... [r=thumper],[bug=1292346] Add local options lxc-clone and lxc-clone-aufs These new values default to the existing behaviour. That being that if the user is on trusty or above, they default to true, otherwise false. The reason for these values is two-fold. Firstly, aufs isn't working on power, and we want to be able to test effectively there. Secondly this gives the user on saucy the ability to try it out (not originally planned, but a helpful side effect). https://codereview.appspot.com/74660044/	Tim Penhey		10 years ago
2431	[r=rogpeppe] state: implement WatchAPIAddresses [r=rogpeppe] state: implement WatchAPIAddresses https://codereview.appspot.com/76870044/	Roger Peppe		10 years ago
2430	[r=rogpeppe] state: implement SetAPIAddresses [r=rogpeppe] state: implement SetAPIAddresses We need this so that agents can watch a single document to see when API addresses change. As with the stateServers document, we create it at Open time if it doesn't already exist. Also fix some existing issues with InitializeSuite - it shouldn't really call Open before Initialize, even though that's something that Initialize does internally. Also fix one bug - Initialize was creating the stateServers document with the wrong key (blank). Luckily the impact of the bug is small because Open will have already created the document. https://codereview.appspot.com/76950043/	Roger Peppe		10 years ago
2429	[r=rogpeppe] state: rename State.APIAddresses [r=rogpeppe] state: rename State.APIAddresses To add an API addresses document, we will need to replace the current APIAddresses method with a method that reads the addresses from the document. However until we have a worker that populates the document, we still need access to the old functionality, which returns the API addresses by collating the addresses from the machines in state. This CL renames the old method so that we become free to implement the new method without breaking existing functionality. The old method will eventually be removed. Better suggestions for the name happily received (APIAddressesDeprecated?) https://codereview.appspot.com/76890043/	Roger Peppe		10 years ago
2428	[r=rogpeppe],[bug=1277397] worker/instancepoller: batch inst... [r=rogpeppe],[bug=1277397] worker/instancepoller: batch instancepoller use ratelimit Token Bucket to batch requests in the instance poller. Mitigates launchpad issue 1277397 https://codereview.appspot.com/74900044/	Michael Foord		10 years ago
2427	[r=axwalk],[bug=1279710] Dump cloud-init-output.log on faile... [r=axwalk],[bug=1279710] Dump cloud-init-output.log on failed bootstrap Currently if bootstrap (or manual provisioning) fails, we don't get a useful error message, just "exit status 1" (because the SSH script failed). This change is to dump the contents of the cloud-init-output.log file to stderr when the SSH script fails. Fixes lp:1279710 https://codereview.appspot.com/75990043/	Andrew Wilkins		10 years ago
2426	[r=wallyworld] New architecture utils [r=wallyworld] New architecture utils Introduce some new architecture utility functions. These will be used by future bootstrap fixes. Also, introduce constants for the various supported architecures and homogenise the supportArches implementation in ec2, openstack, azure providers. https://codereview.appspot.com/76710043/	Ian Booth		10 years ago
2425	[r=axwalk],[bug=1293310] worker/uniter: explicit LookPath fo... [r=axwalk],[bug=1293310] worker/uniter: explicit LookPath for hooks Go 1.3/tip changes os/exec.Command such that it conditionally calls LookPath, based on whether or not the specified path in is absolute. Since this differs across versions, I have simply changed our code to explicitly call LookPath all the time. No tests changed; an existing test failed with Go tip, and I verified the change fixes it. Fixes lp:1293310 https://codereview.appspot.com/76780043/	Andrew Wilkins		10 years ago
2424	[r=jameinel] Update juju to 1.17.6 [r=jameinel] Update juju to 1.17.6 Update juju version and win installer to 1.17.6. This version may become 1.18.0. https://codereview.appspot.com/76170043/	Curtis Hovey		10 years ago
2423	[r=rogpeppe] testing: mongo server helper improvements [r=rogpeppe] testing: mongo server helper improvements This makes it possible to see mongo server messages as they happen. They're logged at TRACE level, which we don't log by default, so this also adds a runtime testing flag -juju.log that allows the logging config to be adjusted for tests. We also add a method to allow us to get the info needed for dialling a test mongo server, so code external to testing can twiddle the knobs too. https://codereview.appspot.com/76120043/	Roger Peppe		10 years ago
2422	[r=rogpeppe],[bug=1271144] provider/maas: apt-get install br... [r=rogpeppe],[bug=1271144] provider/maas: apt-get install bridge-utils Previously we added bridge-utils to the required packages in environs/cloudinit, but at bootstrap time, those packages are not installed before the additional commands specified by a provider are run (we want any failures to be visible, so we wait until the synchronous part of the bootstrap process). To fix this, we explicitly add "apt-get install bridge-utils" to provider/maas and remove it from environs/cloudinit. Fixes: https://bugs.launchpad.net/juju-core/+bug/1271144 https://bugs.launchpad.net/juju-core/+bug/1277359 https://codereview.appspot.com/72230045/	Roger Peppe	juju-1.17.5	10 years ago
2421	[r=thumper] Update github.com/juju/testing dependency [r=thumper] Update github.com/juju/testing dependency juju/testing was updated so the tests passed with gccgo https://codereview.appspot.com/75580048/	Tim Penhey		10 years ago
2420	[r=thumper],[bug=1291967] Omit empty proxy settings in confi... [r=thumper],[bug=1291967] Omit empty proxy settings in config. I had changed them to default to "" because there is no way to unset the value once you have set one for an environment. The correct fix is to implement 'juju unset-env' https://codereview.appspot.com/75700043/	Tim Penhey		10 years ago
2419	[r=thumper],[bug=1291207] Fix the permissions on the tools d... [r=thumper],[bug=1291207] Fix the permissions on the tools dir on upgrade The tempdir was created with 0700, and then renamed to be the tools dir. Now we chmod to 0755 prior to rename. https://codereview.appspot.com/75680043/	Tim Penhey		10 years ago
2418	[r=axwalk] Refactor StartInstance to take param struct [r=axwalk] Refactor StartInstance to take param struct This is in preparation for adding an optional argument to StartInstance that names the principal unit initially assigned to the machine. No functional changes here, just refactoring. https://codereview.appspot.com/75130046/	Andrew Wilkins		10 years ago
2417	[r=wallyworld] Upload tools API [r=wallyworld] Upload tools API An API is provided to allow clients to upload tools. This is used by the upgrade command and removes the need for it to have to create an environment and upload directly to storage. It also allows us to switch to non cloud storage. A lot of the changes are due to code refactoring. The implementation uses an authenticated HTTP endpoint which shares code with the charm upload API. So common code was extracted and shared. The sync tools code was refactored to split up the building and uploading of tools. This allows the building part to be stubbed out for testing. A followup branch can rework the bootstrap code to take advantage of this. https://codereview.appspot.com/75330043/	Ian Booth		10 years ago
2416	[r=wwitzel3],[bug=1289316] utils/apt: determine if a package... [r=wwitzel3],[bug=1289316] utils/apt: determine if a package should come from cloud-archive Mitigates issue lp:1289316 https://codereview.appspot.com/72270044/	Wayne Witzel III		10 years ago

Older »