~rogpeppe/juju-core/438-local-instance-Addresses

From Revision 1940 to 1921

expand all

collapse all

Rev		Summary	Authors	Date	Diff	Files
1940		[r=axwalk],[bug=1234125] provider/null: fix intermittent tes... [r=axwalk],[bug=1234125] provider/null: fix intermittent test failure Also, in environs/sshstorage (related to test failure): capture command output in case of outer ssh/bash command failure. Fixes #1234125 https://codereview.appspot.com/14315044/	Andrew Wilkins	11 years ago
1939		[r=thumper],[bug=1228992] Fail earlier when bootstrapping if... [r=thumper],[bug=1228992] Fail earlier when bootstrapping if up already. I broke apart the VerifyBootstrap method into two:   EnsureNotBootstrapped   VerifyStorage Well I didn't create the second, just didn't call it from the EnsureNotBootstrapped call. The ensure method is now called in the command bootstrap prior to any upload tools checks. However the implication of this was much futher reaching than I expected. Many tests assumed that bootstrap.Bootstrap was enough to bootstrap and environment, and that it would complain if already bootstrapped. Since the complaining was moved into the command function, some changes had to be made. Since some of the tests were refactored to check the storage, this required making the dummy provider actually write the provider-state into storage like a normal provider. I moved the VerifyStorage out of emptystorage.go and into open.go as it seemed better fitting there (and the comment actually makes sense now). Also used the environs constant for the VerificationFilename. https://codereview.appspot.com/14279044/	Tim Penhey	11 years ago
1938		[r=thumper],[bug=1216775] Allow the network-bridge to be con... [r=thumper],[bug=1216775] Allow the network-bridge to be configured. Enable the local provider config to accept network-bridge. This is then passed through to the container creation network config. A slightly more informative message is added when the bridge is not found. https://codereview.appspot.com/14321043/	Tim Penhey	11 years ago
1937		[r=axwalk] environs/sshstorage: quote escapes in storage dir [r=axwalk] environs/sshstorage: quote escapes in storage dir https://codereview.appspot.com/14195043/	Andrew Wilkins	11 years ago
1936		[r=rogpeppe] cmd/jujud: start address updater [r=rogpeppe] cmd/jujud: start address updater https://codereview.appspot.com/14251044/	Roger Peppe	11 years ago
1935		[r=rogpeppe] worker/addressupdater: wire up [r=rogpeppe] worker/addressupdater: wire up It could do with some more comprehensive testing, but it might be OK to land anyway. https://codereview.appspot.com/14306043/	Roger Peppe	11 years ago
1934		[r=fwereade] testing: try harder to reconnect in MgoReset [r=fwereade] testing: try harder to reconnect in MgoReset We were restarting the mongo server on almost every test because the dummy provider sets the admin password. This makes the situation better, although there are probably places that use a different testing admin secret still. https://codereview.appspot.com/13968043/	Roger Peppe	11 years ago
1933		[r=fwereade] ec2,openstack: control-bucket set in Prepare [r=fwereade] ec2,openstack: control-bucket set in Prepare https://codereview.appspot.com/14307043/	William Reade	11 years ago
1932		[r=fwereade] environs: Prepare sets admin-secret [r=fwereade] environs: Prepare sets admin-secret https://codereview.appspot.com/14284044/	William Reade	11 years ago
1931		[r=jameinel],[bug=1231313] environs/cloudinit: Allow for dis... [r=jameinel],[bug=1231313] environs/cloudinit: Allow for disabling hostname Step two towards fixing bug #1202163. This sets up "cloud-init" so that it passes the "--no-check-certificate" flag to wget. I did end up testing this live, and sshing into the machine to delete the ca-certificates and verify that with this flag wget works correctly. (I ran the line in cloud-init and verified that with the flag it downloads, without the flag it fails.) https://codereview.appspot.com/13962043/	John Arbash Meinel	11 years ago
1930		[r=fwereade],[bug=1234035] Fix upgrade 1.14 -> 1.15 [r=fwereade],[bug=1234035] Fix upgrade 1.14 -> 1.15 This adds a Tools() API call to provisioner API, similarly to the upgrader API. Common code factored out into apiserver/common/tools.go. SetAgentTools() renamed to SetAgentVersion() in machine and unit state types. Also renamed a few types in params to be shorter or better reflect what are they about. Added DEPRECATE(v1.18) tags to bits of the API that need cleanup. Live tested on EC2 - after copying the tools from /tools/releases/ to /tools/ the upgrade proceeds and finishes successfully. https://codereview.appspot.com/14231044/ R=fwereade, jameinel	Dimiter Naydenov	11 years ago
1929		[r=rogpeppe] worker/addresspublisher: new package [r=rogpeppe] worker/addresspublisher: new package This package implements a worker that watches machines in state and populates their addresses. This is the start of the implementation of this package, with internal tests for runMachine completed, but only a single test for watchMachinesLoop done. To come: complete tests for watchMachinesLoop; wiring up the logic to real state. https://codereview.appspot.com/14038045/	Roger Peppe	11 years ago
1928		[r=rogpeppe] environs/configstore: fix for Windows [r=rogpeppe] environs/configstore: fix for Windows Under Windows you can't rename a file while it's still open. https://codereview.appspot.com/14285043/	Roger Peppe	11 years ago
1927		[r=jameinel],[bug=1229480] environs/openstack: fix hpcloud d... [r=jameinel],[bug=1229480] environs/openstack: fix hpcloud description Same fix for bug #1229480 that landed on the 1.14 branch, but fixing it in 1.15 requires submitting it for review again. https://codereview.appspot.com/14272043/	John Arbash Meinel	11 years ago
1926		[r=rogpeppe] environs/dummy: add state-id [r=rogpeppe] environs/dummy: add state-id This makes the dummy provider more like the other providers in that the Environ does not refer directly to the bootstrapped state, but is looked up when operations are performed on it. This is also a test case for adding attributes at Prepare time - many tests needed changing to correctly deal with this. https://codereview.appspot.com/14207046/	Roger Peppe	11 years ago
1925		[r=fwereade] provider/common: Destroy, and tests [r=fwereade] provider/common: Destroy, and tests Added a common implementation of Destroy, and tests for Bootstrap which had somehow never been written. Also drive-by removed the machineId arg from consideration when bootstrapping, because, WTF... a non-0 bootstrap machine would break *everything*. https://codereview.appspot.com/14254043/	William Reade	11 years ago
1924		[r=rogpeppe] provider/openstack: remove redundant fields [r=rogpeppe] provider/openstack: remove redundant fields https://codereview.appspot.com/14123043/	Roger Peppe	11 years ago
1923		[r=axwalk],[bug=1233924] provider/azure: remove default dail... [r=axwalk],[bug=1233924] provider/azure: remove default daily stream The simplestreams.GetMetadata function was being called with three datasources: storage, daily and released. In our tests, storage was empty, but daily and released were not; daily did not, however, have any matching image ids. There's an issue with simplestreams.GetMetadata not attempting the next datasource if the metadata index is valid, but does not contain any matching product IDs. Since the behaviour of simplestreams.GetMetadata is potentially useful for private clouds, we will just remove the daily stream from the defaults for now. If people really want it, they can get at it with image-stream and image-metadata-url. Fixes #1233924 https://codereview.appspot.com/14266043/	Andrew Wilkins	11 years ago
1922		[r=axwalk] environs/sshstorage: stream stdin [r=axwalk] environs/sshstorage: stream stdin Change sshstorage's Put to stream to the ssh command's stdin, rather than reading into memory and then writing. Also, drive-by: use `mktemp --tmpdir` rather than exporting $TMPDIR. https://codereview.appspot.com/14197043/	Andrew Wilkins	11 years ago
1921		[r=axwalk] Wire up authenticating httpstorage [r=axwalk] Wire up authenticating httpstorage There are two things happening here:  - httpstorage authentication changes  - wiring up to null provider/localstorage worker. The httpstorage authentication mechanism has changed to use an auth key (randomly generated as part of the boilerplate), rather than client certificates. This gives us more options for using the storage, since now you don't need the client to share the CA key. Also, when serving HTTPS, HTTP is *also* provided, but only for non-modifying operations. This allows wget et al. to fetch files from the storage without bypassing certificate validation. The HTTP interface disallows modifying operations if there is an HTTPS interface backing it. To get an HTTPS URL, a HEAD request may be performed on the HTTP interface. https://codereview.appspot.com/14011044/	Andrew Wilkins	11 years ago

• Older »

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