~jameinel/juju-core/scale-testing : changes

~jameinel/juju-core/scale-testing » Changes from revision 2316

From Revision 2296 to 2277

Rev	Summary	Authors	Tags	Date
2296	if we know the tag of the connection, use it when logging a ... if we know the tag of the connection, use it when logging a disconnect	John Arbash Meinel		10 years ago
2295	cherrpick the ec2 instance type changes cherrpick the ec2 instance type changes	John Arbash Meinel		10 years ago
2294	Refresh the scale testing branch for juju-core trunk. Refresh the scale testing branch for juju-core trunk.	John Arbash Meinel		10 years ago
2293	[r=axwalk],[bug=1275657] utils/ssh: explicitly specify defau... [r=axwalk],[bug=1275657] utils/ssh: explicitly specify default identities There is a bug in ssh_openssh.go: if any identities are specified, then default identities are not attempted unless they are loaded into the ssh-agent. This is only a problem if the user relies on the default identity, AND explicitly specifies authorized-keys in their environment config. This problem is fixed by explicitly specifying the default identities if any identities are specified, which is almost always (~/.juju/ssh). Fixes lp:1275657 https://codereview.appspot.com/57690049/	Andrew Wilkins		10 years ago
2292	[r=waigani] environs/config: expand field names [r=waigani] environs/config: expand field names	Jesse Meek		10 years ago
2291	[r=adeuring],[bug=1273408] destroy-environment for local pro... [r=adeuring],[bug=1273408] destroy-environment for local provider: pass JUJU_HOME to 'sudo juju destroy-environment...'	Abel Deuring		10 years ago
2290	[r=thumper] Have container provisioners support proxies [r=thumper] Have container provisioners support proxies Container provisioners need to get the proxy information, and need to set this in the MachineConfig. This comes from the ContainerConfig api call. Also, the special casing for the containers apt proxy support has been removed, as this is now handled in the much more general case.	Tim Penhey		10 years ago
2289	[r=frankban],[bug=1271341] Add the environment name to the h... [r=frankban],[bug=1271341] Add the environment name to the hooks context. Expose the human friendly environment name to the hooks through the JUJU_ENV_NAME environment variable. This is required e.g. by the GUI in order to give to the user a hint about where to find the password to log in. https://codereview.appspot.com/50090044/	Francesco Banconi		10 years ago
2288	[r=sinzui] Increment juju to 1.17.3. [r=sinzui] Increment juju to 1.17.3.	Curtis Hovey		10 years ago
2287	[r=dimitern],[bug=1273464] apiserver: Charm upload takes arc... [r=dimitern],[bug=1273464] apiserver: Charm upload takes archives w/ subdirs Fixed bug #1273464: HTTPS based API to upload local charms fails for compressed charms with nested dirs inside, instead of having metadata.yaml, etc. in the zip root dir. Changes only how uploads are handled: charms with nested dirs are detected after uploading and then repackaged to remove the nesting, before uploading them to the provider storage. Some code for handling zipped archives for charms is copied from the charm package, but as fwereade requested, the changes do not affect the charm package itself. https://codereview.appspot.com/54680045/ R=natefinch	Dimiter Naydenov		10 years ago
2286	[r=axwalk],[bug=1269120] utils/ssh: implicit -i options for ... [r=axwalk],[bug=1269120] utils/ssh: implicit -i options for client keys Each key in the juju ssh client keys dir should have a corresponding -i option added to all invocations of ssh/scp. This caters for the scenario where OpenSSH is installed, but the user has not generated any keys. Fixes #1269120 https://codereview.appspot.com/58140044/	Andrew Wilkins		10 years ago
2285	[r=axwalk],[bug=1272614] Rename null to manual [r=axwalk],[bug=1272614] Rename null to manual The null provider is renamed to manual. We retain an alias of "null", for existing deployments. Fixes #1272614 https://codereview.appspot.com/53040046/	Andrew Wilkins	juju-1.17.2	10 years ago
2284	[r=wallyworld] Misc small fixes [r=wallyworld] Misc small fixes Follow up to the previous work to address some bootstrap issues. Now ctx.Abspath() is used with command file/dir parameters. Also, make the check for exisitng files in the dummy storage URL() more strict. https://codereview.appspot.com/58870043/	Ian Booth		10 years ago
2283	[r=thumper],[bug=1270671] Fix the local provider all-machine... [r=thumper],[bug=1270671] Fix the local provider all-machines.log There were a number of problems with this as it was, which failed more in trusty as I think the apparmor profile for rsyslogd may have changed or got bugs fixed. The rsyslog config file was being written by root with 0600 perms. The rsyslog process does a setuid to become the syslog user. The syslog user couldn't read the config file. Now this is written with 0644 so it can be read. The apparmor profile is quite strict about where rsyslog can write files. Instead of poking with the profile, the local provider now logs to /var/log/juju-{{user}}-{{env name}}/all-machines.log, and a symlink is made in the local provider log dir to point to that file. The file is also created with 0644 so the user can read it without poking permissions. By default rsyslog creates files with 0644, but in the ubuntu package, the setting is changed to 0640, which means normal users can't read the log file. Using a new action directive (new as in not-legacy), we can specify the file create mode so it doesn't use the default. Also, when a local environment is destroyed, the normal dir is removed, but the all-machines.log is left around (which I find handy) until the environment is bootstrapped again, then it is removed.	Tim Penhey		10 years ago
2282	[r=dimitern],[bug=1257649] bootstrap: Added configurable tim... [r=dimitern],[bug=1257649] bootstrap: Added configurable timeout and delays Fixed bug #1257649: ssh timeout for bootstrap could be configurable, by adding 3 new config settings: - bootstrap-timeout (default: 600s = 10m) - bootstrap-retry-delay (default: 5s) - bootstrap-addresses-delay (default: 10s) All of them are integers, measured in seconds, and also all of them are optional and have the specified defaults. Updated the bootstrap command's help text to include the new config settings. https://codereview.appspot.com/58170045/ R=natefinch	Dimiter Naydenov		10 years ago
2281	[r=dimitern],[bug=1067979] state;apiserver: Fix a race - lp ... [r=dimitern],[bug=1067979] state;apiserver: Fix a race - lp bug #1067979 This introduces some changes to how charm store charms are added through the API (in state and to provider storage). Now PrepareStoreCharmUpload is called before trying to download the charm, repackage it and upload it to storage, in order to reserve a charm URL in state with pending state. Added a test that demonstrates multiple concurrent deployments of the same charm does not cause the race issues, like mentioned in the bug. A few drive-by fixes brought up during review: * Added ReadSHA256 and ReadFileSHA256 helpers in utils, and changed most places where hashes are calculated to use them. * Charms are now uploaded to storage with a randomly generated archive names with the format "<charm-name>-<revision>-<uuid>". This allows multiple concurrent uploads to happen safely, and at the end AddCharm in the API checks to see if the charm info is already updated in state and if so, deletes the duplicated upload. * Added GetEnvironStorage helper to environs/testing. * Fixed potential compatibility issues with older versions and the recently added PendingUpload and Placeholder fields of the charm document. Also tested multiple concurrent deployments with the local provider manually and updated the bug accordingly. https://codereview.appspot.com/53210044/ R=fwereade	Dimiter Naydenov		10 years ago
2280	[r=adeuring],[bug=1250285] juju switch -l: Show all defined [r=adeuring],[bug=1250285] juju switch -l: Show all defined environments when JUJU_ENV is set.	Abel Deuring		10 years ago
2279	[r=axwalk],[bug=1274018] manual: mark specified host as a pu... [r=axwalk],[bug=1274018] manual: mark specified host as a public address Recent work to have machine agents publish their machine addresses has caused a problem for the manual provider. The bootstrap node currently does not record the host/IP bootstrapped with as being public, and so any machine address that comes along may override it as being the "most public address". I have changed manual provisioning/bootstrap to mark the address we specified explicitly as being public. Fixes #1274018 https://codereview.appspot.com/56310044/	Andrew Wilkins		10 years ago
2278	[r=wallyworld] Fix various bootstrap issues [r=wallyworld] Fix various bootstrap issues This fixed bootstrap on MAAS with --upload-tools and also general bootstrap with --metadata-source. It also changes dummy storage so that it mimics MAAS behaviour to hopefully help catch future issues. https://codereview.appspot.com/58510044/	Ian Booth		10 years ago
2277	[r=axwalk] Add option to disable apt in ProvisioningScript [r=axwalk] Add option to disable apt in ProvisioningScript The ProvisioningScript API is updated to take a boolean option DisablePackageCommands to completely disable all package-related commands (apt-get, etc.) This is for the benefit of external provisioners that require more control over which packages are installed and from where. https://codereview.appspot.com/58490043/	Andrew Wilkins		10 years ago