nova-adminclient

Merge lp:~anotherjesse/nova-adminclient/set-region-in-connection-for into lp:nova-adminclient

  1. set-region-in-connection-for
  2. Merge into trunk
Proposed by Jesse Andrews
Status: Merged
Approved by: Devin Carlen
Approved revision: 6
Merged at revision: 5
Proposed branch: lp:~anotherjesse/nova-adminclient/set-region-in-connection-for
Merge into: lp:nova-adminclient
Diff against target: 25 lines (+2/-2)
2 files modified
nova_adminclient/client.py (+1/-1)
setup.py (+1/-1)
To merge this branch: bzr merge lp:~anotherjesse/nova-adminclient/set-region-in-connection-for
Reviewer Review Type Date Requested Status
Devin Carlen Approve
Review via email: mp+61192@code.launchpad.net

Description of the change

use passed region instead of self.region when creating new boto connection

To post a comment you must log in.
lp:~anotherjesse/nova-adminclient/set-region-in-connection-for updated
6. By Jesse Andrews

bump rev

Revision history for this message
Devin Carlen (devcamcar) wrote :

lgtm

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'nova_adminclient/client.py'
2--- nova_adminclient/client.py 2011-03-30 18:30:04 +0000
3+++ nova_adminclient/client.py 2011-05-17 03:12:41 +0000
4@@ -319,7 +319,7 @@
5 aws_secret_access_key=user.secretkey,
6 is_secure=parts['is_secure'],
7 region=RegionInfo(None,
8- self.region,
9+ region,
10 parts['ip']),
11 port=parts['port'],
12 path='/services/Cloud',
13
14=== modified file 'setup.py'
15--- setup.py 2011-03-31 01:40:43 +0000
16+++ setup.py 2011-05-17 03:12:41 +0000
17@@ -20,7 +20,7 @@
18 from setuptools import setup, find_packages
19
20 setup(name = "nova-adminclient",
21- version = "0.1.5",
22+ version = "0.1.6",
23 license = 'Apache License (2.0)',
24 description = "client for administering OpenStack Nova",
25 author = 'OpenStack',

Subscribers

People subscribed via source and target branches