lp:~wallyworld/gwacl/cloud-init-fix

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp:gwacl at revision 229

Ian Booth: Approve on 2013-10-31

Diff: 48 lines (+5/-3)

3 files modified

example/management/run.go (+3/-1)
xmlobjects.go (+1/-1)
xmlobjects_test.go (+1/-1)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #1246320: Azure bootstrap fails: versioning header is not specified

Critical

Fix Released

Link a bug report

Related blueprints

231. By Ian Booth on 2013-10-31

Reorder import

230. By Ian Booth on 2013-10-31

Reorder import

229. By Ian Booth on 2013-10-31

Fix base64 encoding of custom data

228. By Ian Booth on 2013-10-31

[r=wallyworld] [r=][bug=][author=wallyworld] The first fix for the request eof issue worked by setting
Close=true on the http request. This seemed to be not always
work so a different approach is used (same as for goose).
Set DisableKeepAlives=true on the http transport.

227. By Ian Booth on 2013-10-31

[r=wallyworld],[bug=1246320] [r=][bug=1246320][author=wallyworld] Azure now requires certain requests to the management api to have an updated version number in the header. Such changes are supposed to be backwards compatible but msft failed not to break existing applications. This branch includes the new api version. It also base64 encodes customdata sent with the request, as this also appears to have changed from underneath us.

With these changes, juju bootstrap now completes and the bootstrap instance spins up. Previously it would have errored part way through bootstrap. There still may be an issue with cloud-init completing properly but if this is the case, it appears to be a separate issue and I'd like to land this branch to allow others with Azure credentials to help debug.

226. By Ian Booth on 2013-10-17

[r=wallyworld] [r=][bug=][author=wallyworld] Ensure Close = true when sending http requests.

This is needed to avoid this bug in Go's http lib:
https://code.google.com/p/go/issues/detail?id=4677

225. By Jeroen T. Vermeulen on 2013-09-24

[r=][bug=][author=jtv] Update role sizes: new A5 machine type, and new prices.

224. By Ian Booth on 2013-09-12

[r=][bug=][author=wallyworld] GetAnonymousFileURL() was panicing if there was an issue. I changed it to return an error. This is because juju sometimes needs to try and find a URL of a file but it doesn't know if it exists or not. So by using error and not panic, juju can handle the situation accordingly.

223. By Gavin Panella on 2013-08-29

[r=jameinel][bug=][author=allenap] Make tests verbose again.

222. By Gavin Panella on 2013-08-29

[r=allenap] [r=jtv][bug=][author=allenap] Get all tests running.

The logging tests were not running, but there was no obvious
indication that there was a problem; the module was developed in the
top-level gwacl package, moved to a sub-package when it was complete,
but it was missing the hook function from testing into gocheck.

In any case, `make check` would not have run tests in any sub-package.
When -gocheck.v is passed into `go test` the package specification
appears to be ignored, and only tests in the current package are
exercised.