lp:~dave-cheney/juju-core/054-sidestep-bootstrap-gofmt-issue

Branch merges

Propose for merging

No branches dependent on this one.

Rejected for merging into lp:~juju/juju-core/trunk

The Go Language Gophers: Pending requested 2012-11-26

Diff: 123 lines (+33/-22)

4 files modified

environs/ec2/config.go (+18/-9)
environs/ec2/ec2.go (+2/-1)
environs/ec2/local_test.go (+12/-11)
juju/bootstrap.go (+1/-1)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Related blueprints

747. By Dave Cheney on 2012-11-26

gofmt

746. By Dave Cheney on 2012-11-26

fix tests

745. By Dave Cheney on 2012-11-26

gofmt

744. By Dave Cheney on 2012-11-26

added public-bucket-region

743. By Roger Peppe on 2012-11-24

environs: fix config test

It slipped through the cracks somehow.
Submitting because it's trivial and also unbreaks trunk.

R=
CC=
https://codereview.appspot.com/6842094

742. By Roger Peppe on 2012-11-24

environs/config: adjust Root->CA in test descriptions.

Trivial enough that I'm going to submit it immediately.

R=
CC=
https://codereview.appspot.com/6855085

741. By Roger Peppe on 2012-11-23

environs/config: use "" instead of nil

As discussed on line.
Tests now pass with inaccessible ~/.ssh and ~/.juju.

R=niemeyer
CC=
https://codereview.appspot.com/6854088

740. By Roger Peppe on 2012-11-23

environs/config: make CA cert optional

As discussed online.
Also make cmd/juju not use juju.Bootstrap, because
it's generating a PEM file we can't use.

R=TheMue, niemeyer
CC=
https://codereview.appspot.com/6850087

739. By Roger Peppe on 2012-11-23

environs/config: old (reverted) CL

This is here for reference purposes, so we
can view all the old changes.

R=niemeyer
CC=
https://codereview.appspot.com/6852081

738. By Dimiter Naydenov on 2012-11-23

Trivial fix of a failing test.

After making the region mandatory in a previous CL, the failing test is now fixed.

R=rog
CC=
https://codereview.appspot.com/6843109