Skip to content

Commit

Permalink
SM-Openstack: don't install nova-compute on openstack nodes
Browse files Browse the repository at this point in the history
Closes-Bug: #1638321

nova::compute and nova::compute::neutron are not needed for openstack roles.
only needed for compute.

Change-Id: If36e794c246dccb3ea44446d3815517809c60627
  • Loading branch information
Dheeraj Gautam committed Nov 1, 2016
1 parent 90b50a9 commit 49e2d74
Showing 1 changed file with 15 additions and 12 deletions.
Expand Up @@ -15,6 +15,7 @@
$sriov_enable = $::contrail::params::sriov_enable,
$enable_ceilometer = $::contrail::params::enable_ceilometer,
$package_sku = $::contrail::params::package_sku,
$host_roles = $::contrail::params::host_roles,
$contrail_internal_vip = $::contrail::params::contrail_internal_vip,
$openstack_rabbit_servers = $::contrail::params::openstack_rabbit_hosts,
$neutron_shared_secret = $::contrail::params::os_neutron_shared_secret,
Expand Down Expand Up @@ -147,20 +148,22 @@
enabled => 'true',
}

# TODO: it's important to set up the vnc properly
class { '::nova::compute':
enabled => $contrail_is_compute,
vnc_enabled => true,
vncserver_proxyclient_address => $management_address,
vncproxy_host => $address_api,
instance_usage_audit => $instance_usage_audit,
instance_usage_audit_period => $instance_usage_audit_period
}
if ('compute' in $host_roles) {
# TODO: it's important to set up the vnc properly
class { '::nova::compute':
enabled => $contrail_is_compute,
vnc_enabled => true,
vncserver_proxyclient_address => $management_address,
vncproxy_host => $address_api,
instance_usage_audit => $instance_usage_audit,
instance_usage_audit_period => $instance_usage_audit_period
}

#TODO make sure we have vif package
#TODO make sure we have vif package

class { '::nova::compute::neutron':
libvirt_vif_driver => "nova_contrail_vif.contrailvif.VRouterVIFDriver"
class { '::nova::compute::neutron':
libvirt_vif_driver => "nova_contrail_vif.contrailvif.VRouterVIFDriver"
}
}

class { '::nova::network::neutron':
Expand Down

0 comments on commit 49e2d74

Please sign in to comment.