openstack/puppet-nova
Code Issues Proposed changes

Add a few TODOs

Browse Source
This commit is contained in:
Dan Bode
2012-03-30 23:37:15 -07:00
parent ce46fb14dc
commit 034bc51db5
2 changed files with 3 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
TODO Normal file
View File

@@ -0,0 +1 @@
cert support has only been added for Redhat. Should this be added for Ubuntu?

2
manifests/init.pp
View File

@@ -2,6 +2,7 @@ class nova(
# this is how to query all resources from our clutser # this is how to query all resources from our clutser
$nova_cluster_id='localcluster', $nova_cluster_id='localcluster',
$sql_connection = false, $sql_connection = false,
# TODO maybe this should default to glance?
$image_service = 'nova.image.local.LocalImageService', $image_service = 'nova.image.local.LocalImageService',
# these glance params should be optional # these glance params should be optional
# this should probably just be configured as a glance client # this should probably just be configured as a glance client
@@ -97,6 +98,7 @@ class nova(
# query out the config for our db connection # query out the config for our db connection
# TODO - I am not sure if resource collection should be the default
if $sql_connection { if $sql_connection {
nova_config { 'sql_connection': value => $sql_connection } nova_config { 'sql_connection': value => $sql_connection }
} else{ } else{