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

From Revision 1932 to 1913

expand all

collapse all

Rev		Summary	Authors	Date	Diff	Files
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
1920		[r=axwalk] provider/null: define simplestreams tools source [r=axwalk] provider/null: define simplestreams tools source This is somewhat involved, as there's some fixes required (and one drive-by in sshstorage). The change to use GetToolsSources is simple enough, and matches what the local provider does. The Environ override struct in cmd/juju/bootstrap.go had to so that the optional Environ interfaces were still implemented. Overriding like that loses the additional interfaces. To do the above, environs.BootstrapStorager has been changed to have a method that tells the environ to use bootstrap storage, rather than returning a bootstrap storage. The null provider has changed to match, of course, as well as cmd/juju. https://codereview.appspot.com/14011046/	Andrew Wilkins	11 years ago
1919		[r=thumper],[bug=1231724] Make local provider work. [r=thumper],[bug=1231724] Make local provider work. lp:1231724 shows that if a local provider is bootstrapped, the created directory $(JUJU_HOME)/environments is not readable by the user that started the provider as the bootstrap is done by root, and the files and directories created are owned by root with 0600 permission, which means you can't even run juju status on a bootstrapped local provider. https://codereview.appspot.com/14258043/	Tim Penhey	11 years ago
1918		[r=thumper],[bug=1219887] Use a retry strategy to start new ... [r=thumper],[bug=1219887] Use a retry strategy to start new services. On slower disks, upstart doesn't necessarily notice the file straight away. Use a retry strategy to start the newly written upstart file. The first attempt happens straight away, so no slow down on faster systems. https://codereview.appspot.com/14260043/	Tim Penhey	11 years ago
1917		[r=thumper] Really fix the local provider startup. [r=thumper] Really fix the local provider startup. sudo -E causes upstart on saucy to default to use the user bus. --system was added to start and stop, but status was missed. Race conditions during startup masked this before. I've added another check in start so if the actual start fails, check to see if it is started. When the file is written to disk, upstart may start it before we do, particularly, upstart may start it between when we checked it wasn't running, and when we try to start it. A drive by fix to the livetests was added as version.Current was being used to source information from test metadata, and there weren't saucy images defined. The tests pass wtih the version specified with no other alterations. https://codereview.appspot.com/14243043/	Tim Penhey	11 years ago
1916		[r=thumper] Manually clean up lxc autostart symlink [r=thumper] Manually clean up lxc autostart symlink lxc-destroy used to clean this up, now it doensn't. https://codereview.appspot.com/14182044/	Tim Penhey	11 years ago
1915		[r=rogpeppe] environs: add store arg to NewFromName etc [r=rogpeppe] environs: add store arg to NewFromName etc This enables us to delete the EnsureCACert magic. Old certificates should still be used, as the old config magic will still read them. https://codereview.appspot.com/14136043/	Roger Peppe	11 years ago
1914		[r=dimitern] cmd/jujud; others: Disable setting mongo passwd... [r=dimitern] cmd/jujud; others: Disable setting mongo passwds This disables settings mongo passwords for agents. The unit agent is totally cut off from state now, not only it doesn't connect, but it doesn't have a password do to so. The machine agent is a bit more complicated: only machines with JobHostUnits won't have a mongo password or connection to state. Due to firewaller not using the API yet, JobManageEnviron also has to have state connection (and mongo password) for now. We don't do anything yet to reset existing passwords in mongo, but this will come in the following releases. https://codereview.appspot.com/14036045/ R=fwereade, jameinel	Dimiter Naydenov	11 years ago
1913		[r=rogpeppe] cmd/juju: fix test isolation issue [r=rogpeppe] cmd/juju: fix test isolation issue https://codereview.appspot.com/14200044/	Roger Peppe	11 years ago

• Older »

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