Remove workaround for resume_guests option
Change-Id: Ib3100f17d17fca7fe948cfff2b6b8500be18b322 Closes-Bug: #1651101
This commit is contained in:
parent
13897f7db8
commit
a5b336a50c
@ -313,19 +313,20 @@ class openstack_tasks::roles::compute {
|
||||
# after the deployment is done.
|
||||
# NOTE(bogdando) This maybe be changed, if the host aggregates implemented, bp disable-new-computes
|
||||
class { '::nova::compute':
|
||||
enabled => false,
|
||||
vncserver_proxyclient_address => get_network_role_property('nova/api', 'ipaddr'),
|
||||
vncproxy_protocol => $vncproxy_protocol,
|
||||
vncproxy_host => $vncproxy_host,
|
||||
vncproxy_port => $nova_hash_real['vncproxy_port'],
|
||||
force_config_drive => $force_config_drive,
|
||||
pci_passthrough => nic_whitelist_to_json(get_nic_passthrough_whitelist('sriov')),
|
||||
instance_usage_audit => $instance_usage_audit,
|
||||
instance_usage_audit_period => $instance_usage_audit_period,
|
||||
reserved_host_memory => $nova_hash_real['reserved_host_memory'],
|
||||
config_drive_format => $config_drive_format,
|
||||
allow_resize_to_same_host => true,
|
||||
vcpu_pin_set => $nova_hash_real['cpu_pinning'],
|
||||
enabled => false,
|
||||
vncserver_proxyclient_address => get_network_role_property('nova/api', 'ipaddr'),
|
||||
vncproxy_protocol => $vncproxy_protocol,
|
||||
vncproxy_host => $vncproxy_host,
|
||||
vncproxy_port => $nova_hash_real['vncproxy_port'],
|
||||
force_config_drive => $force_config_drive,
|
||||
pci_passthrough => nic_whitelist_to_json(get_nic_passthrough_whitelist('sriov')),
|
||||
instance_usage_audit => $instance_usage_audit,
|
||||
instance_usage_audit_period => $instance_usage_audit_period,
|
||||
reserved_host_memory => $nova_hash_real['reserved_host_memory'],
|
||||
config_drive_format => $config_drive_format,
|
||||
allow_resize_to_same_host => true,
|
||||
vcpu_pin_set => $nova_hash_real['cpu_pinning'],
|
||||
resume_guests_state_on_host_boot => hiera('resume_guests_state_on_host_boot', 'False'),
|
||||
}
|
||||
|
||||
nova_config {
|
||||
@ -341,17 +342,6 @@ class openstack_tasks::roles::compute {
|
||||
}
|
||||
}
|
||||
|
||||
# TODO (mkarpin): rework this option management once it's available in puppet-nova module (LP #1651101)
|
||||
if !defined(Nova_config['DEFAULT/resume_guests_state_on_host_boot']) {
|
||||
nova_config {
|
||||
'DEFAULT/resume_guests_state_on_host_boot': value => hiera('resume_guests_state_on_host_boot', 'False')
|
||||
}
|
||||
} else {
|
||||
Nova_config<| title == 'DEFAULT/resume_guests_state_on_hosts_boot' |> {
|
||||
value => hiera('resume_guests_state_on_host_boot', 'False')
|
||||
}
|
||||
}
|
||||
|
||||
if $use_syslog {
|
||||
nova_config {
|
||||
'DEFAULT/use_syslog_rfc_format': value => true;
|
||||
|
@ -69,6 +69,8 @@ describe manifest do
|
||||
|
||||
let(:rabbit_hash) { Noop.hiera_structure 'rabbit', {} }
|
||||
|
||||
let(:resume_guests_state_on_host_boot) { Noop.hiera 'resume_guests_state_on_host_boot', 'False' }
|
||||
|
||||
# Legacy openstack-compute tests
|
||||
|
||||
if ironic_enabled
|
||||
@ -369,6 +371,14 @@ describe manifest do
|
||||
)
|
||||
end
|
||||
|
||||
it 'should properly configure resuming guests on host boot' do
|
||||
should contain_class('nova::compute').with(
|
||||
'resume_guests_state_on_host_boot' => resume_guests_state_on_host_boot
|
||||
)
|
||||
should contain_nova_config('DEFAULT/resume_guests_state_on_host_boot').with(
|
||||
'value' => resume_guests_state_on_host_boot)
|
||||
end
|
||||
|
||||
it 'should properly configure glance api servers with (non-)ssl' do
|
||||
should contain_class('nova').with(
|
||||
'glance_api_servers' => glance_api_servers
|
||||
|
Loading…
Reference in New Issue
Block a user