lp:~thumper/juju-core/more-errgo

Created by Tim Penhey and last modified
Get this branch:
bzr branch lp:~thumper/juju-core/more-errgo
Only Tim Penhey can upload to this branch. If you are Tim Penhey please log in for upload directions.

Branch merges

Related bugs

Related blueprints

Branch information

Owner:
Tim Penhey
Project:
juju-core
Status:
Development

Recent revisions

2295. By Tim Penhey

More errgo usage.

2294. By Tim Penhey

[r=thumper] Add errgo as a dep, and start using it.

errgo adds the ability to capture the location of
the error creation or annotation. Annotated errors
don't lose the original error.

https://codereview.appspot.com/59930043/

2293. By Andrew Wilkins

[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/

2292. By Jesse Meek

[r=waigani] environs/config: expand field names

2291. By Abel Deuring

[r=adeuring],[bug=1273408] destroy-environment for local provider: pass JUJU_HOME to 'sudo juju destroy-environment...'

2290. By Tim Penhey

[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.

2289. By Francesco Banconi

[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/

2288. By Curtis Hovey

[r=sinzui] Increment juju to 1.17.3.

2287. By Dimiter Naydenov

[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

2286. By Andrew Wilkins

[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/

Branch metadata

Branch format:
Branch format 7
Repository format:
Bazaar repository format 2a (needs bzr 1.16 or later)
Stacked on:
lp:~go-bot/juju-core/trunk
This branch contains Public information 
Everyone can see this information.

Subscribers