diff options
author | Mark McLoughlin <markmc@redhat.com> | 2012-11-21 23:29:35 +0000 |
---|---|---|
committer | Mark McLoughlin <markmc@redhat.com> | 2012-11-28 15:04:34 +0000 |
commit | 4408b6bb43353624b525ff674f5aaad46efc8438 (patch) | |
tree | 07669175f28f21339989f29bf9ccb85bf19b63c1 /nova | |
parent | 33d7f56cc424071a90f78ebd21913ce2bc9e6c0d (diff) | |
download | nova-4408b6bb43353624b525ff674f5aaad46efc8438.tar.gz |
Move ec2 config opts to nova.api.ec2.cloud
Only ec2_dmz_host and ec2_port are used outside of ec2.cloud and in
both cases pipelib is the other module that uses them.
Move all the ec2 options to ec2.cloud and make pipelib explicitly
depend on the ones it needs.
blueprint: scope-config-opts
Change-Id: Ib187ec3421a25bdd86b13dfeae1dc9a07873eb43
Diffstat (limited to 'nova')
-rw-r--r-- | nova/api/ec2/cloud.py | 28 | ||||
-rw-r--r-- | nova/cloudpipe/pipelib.py | 4 | ||||
-rw-r--r-- | nova/config.py | 19 |
3 files changed, 25 insertions, 26 deletions
diff --git a/nova/api/ec2/cloud.py b/nova/api/ec2/cloud.py index ad922118e9..df6f17fb05 100644 --- a/nova/api/ec2/cloud.py +++ b/nova/api/ec2/cloud.py @@ -45,13 +45,31 @@ from nova import servicegroup from nova import utils from nova import volume +ec2_opts = [ + cfg.StrOpt('ec2_host', + default='$my_ip', + help='the ip of the ec2 api server'), + cfg.StrOpt('ec2_dmz_host', + default='$my_ip', + help='the internal ip of the ec2 api server'), + cfg.IntOpt('ec2_port', + default=8773, + help='the port of the ec2 api server'), + cfg.StrOpt('ec2_scheme', + default='http', + help='the protocol to use when connecting to the ec2 api ' + 'server (http, https)'), + cfg.StrOpt('ec2_path', + default='/services/Cloud', + help='the path prefix used to call the ec2 api server'), + cfg.ListOpt('region_list', + default=[], + help='list of region=fqdn pairs separated by commas'), +] CONF = cfg.CONF -CONF.import_opt('ec2_host', 'nova.config') -CONF.import_opt('ec2_path', 'nova.config') -CONF.import_opt('ec2_port', 'nova.config') -CONF.import_opt('ec2_scheme', 'nova.config') -CONF.import_opt('region_list', 'nova.config') +CONF.register_opts(ec2_opts) +CONF.import_opt('my_ip', 'nova.config') CONF.import_opt('vpn_image_id', 'nova.config') CONF.import_opt('vpn_key_suffix', 'nova.config') diff --git a/nova/cloudpipe/pipelib.py b/nova/cloudpipe/pipelib.py index 274bdf5b57..8b8ae00c6c 100644 --- a/nova/cloudpipe/pipelib.py +++ b/nova/cloudpipe/pipelib.py @@ -54,8 +54,8 @@ cloudpipe_opts = [ CONF = cfg.CONF CONF.register_opts(cloudpipe_opts) -CONF.import_opt('ec2_dmz_host', 'nova.config') -CONF.import_opt('ec2_port', 'nova.config') +CONF.import_opt('ec2_dmz_host', 'nova.api.ec2.cloud') +CONF.import_opt('ec2_port', 'nova.api.ec2.cloud') CONF.import_opt('vpn_image_id', 'nova.config') CONF.import_opt('vpn_key_suffix', 'nova.config') CONF.import_opt('pybasedir', 'nova.config') diff --git a/nova/config.py b/nova/config.py index d1878365ba..a55a776ae6 100644 --- a/nova/config.py +++ b/nova/config.py @@ -71,9 +71,6 @@ global_opts = [ cfg.StrOpt('my_ip', default=_get_my_ip(), help='ip address of this host'), - cfg.ListOpt('region_list', - default=[], - help='list of region=fqdn pairs separated by commas'), cfg.StrOpt('aws_access_key_id', default='admin', help='AWS Access ID'), @@ -130,22 +127,6 @@ global_opts = [ cfg.ListOpt('enabled_apis', default=['ec2', 'osapi_compute', 'metadata'], help='a list of APIs to enable by default'), - cfg.StrOpt('ec2_host', - default='$my_ip', - help='the ip of the ec2 api server'), - cfg.StrOpt('ec2_dmz_host', - default='$my_ip', - help='the internal ip of the ec2 api server'), - cfg.IntOpt('ec2_port', - default=8773, - help='the port of the ec2 api server'), - cfg.StrOpt('ec2_scheme', - default='http', - help='the protocol to use when connecting to the ec2 api ' - 'server (http, https)'), - cfg.StrOpt('ec2_path', - default='/services/Cloud', - help='the path prefix used to call the ec2 api server'), cfg.ListOpt('osapi_compute_ext_list', default=[], help='Specify list of extensions to load when using osapi_' |