lp:~dave-cheney/goose/003-fix-fd-leak

Created by Dave Cheney and last modified
Get this branch:
bzr branch lp:~dave-cheney/goose/003-fix-fd-leak
Only Dave Cheney can upload to this branch. If you are Dave Cheney please log in for upload directions.

Branch merges

Related bugs

Related blueprints

Branch information

Owner:
Dave Cheney
Project:
Go OpenStack Exchange
Status:
Development

Recent revisions

86. By Dave Cheney

always close response body before returning from sendRequest

85. By John A Meinel

[r=jam] goose/{errors,http}: fix format strings in errors.

https://codereview.appspot.com/8818043/

84. By Ian Booth

[r=wallyworld] Simplify some complicated code

This branch cleans up some complicated code which was left after a previous branch landed.

https://codereview.appspot.com/8079043/

83. By Ian Booth

[r=wallyworld] Better error when region is misconfigured.

This branch prints a better error message if the user hasn't configured their
region correctly. For example, if they specify the swift region, but not the
fully qualified compute region, they are told that and the list of correct
regions is displayed. Similarly, if the region is totally wrong, they are
told that too.

The nexted error have also been reformatted to make them more readable, eg

error: failed to GET object provider-state from container juju-50548874395ac32d8896f65ab33f3e17
caused by: cannot create service URLs
caused by: invalid region "xegion-a.geo-1"
one of these regions may be suitable instead: "az-1.region-a.geo-1, az-2.region-a.geo-1, az-3.region-a.geo-1"

https://codereview.appspot.com/8067044/

82. By Ian Booth

[r=wallyworld] Retain traling slashes on URLs

When the final URL for the request was being constructed, all trailing slashes were
being stripped, even if the URL was supposed to have one.

https://codereview.appspot.com/7963043/

81. By Ian Booth

[r=wallyworld] Fix json unmarshalling of large int ids

Large integer ids used in Entity, FlavorDetail, ServerDetail etc were
being unmarsalled as strings in scientific format instead of unformatted
numerics. This is fixed, and some tests are added.

https://codereview.appspot.com/7937044/

80. By John A Meinel

[r=jameinel] identityservice: out-of-random nil error

On some circumstances we can't read all 16-bytes of random data for a hex token in one request. Instead we can use rand.Reader and ioutil functionality to retry reads, etc.
This avoids one of the random failures in the test suite.

79. By Ian Booth

[r=wallyworld] Add prefix support to swift test double

The swift test double did not support using prefix=xxx to filter container listings.
This branch adds that support which is required to make some juju-core tests pass.

https://codereview.appspot.com/7388055/

78. By Ian Booth

[r=wallyworld] Fix server id gen for noav double

A quick change to generate the nova server id on the test double using a counter not a random value.

https://codereview.appspot.com/7396058/

77. By Ian Booth

[r=wallyworld] Try and re-authorise automatically

If a request returns 401 or 403, attempt to re-authorise and then retry the request.
The service double hook code was moved to a new package so that some tests could be
written without circular import issues.

https://codereview.appspot.com/7374044/

Branch metadata

Branch format:
Branch format 7
Repository format:
Bazaar repository format 2a (needs bzr 1.16 or later)
Stacked on:
lp:goose
This branch contains Public information 
Everyone can see this information.

Subscribers