diff --git a/manifests/scheduler/filter.pp b/manifests/scheduler/filter.pp index 457d49d16..0be1ffb0c 100644 --- a/manifests/scheduler/filter.pp +++ b/manifests/scheduler/filter.pp @@ -92,21 +92,6 @@ # (optional) Separator character(s) for image property namespace and name # Defaults to $::os_service_default # -# DEPRECATED -# -# [*baremetal_scheduler_default_filters*] -# An array of filters to be used by default for baremetal hosts -# No longer used. Defaults to undef -# -# [*scheduler_use_baremetal_filters*] -# Use baremetal_scheduler_default_filters or not. -# No longer used. Defaults to undef -# -# [*scheduler_host_manager*] -# The scheduler host manager class to use. -# No longer used. Defaults to undef -# - class nova::scheduler::filter ( $scheduler_max_attempts = '3', $scheduler_host_subset_size = '1', @@ -128,10 +113,6 @@ class nova::scheduler::filter ( $restrict_isolated_hosts_to_isolated_images = $::os_service_default, $aggregate_image_properties_isolation_namespace = $::os_service_default, $aggregate_image_properties_isolation_separator = $::os_service_default, - # DEPRECATED - $baremetal_scheduler_default_filters = undef, - $scheduler_use_baremetal_filters = undef, - $scheduler_host_manager = undef, ) { include nova::deps @@ -158,12 +139,6 @@ class nova::scheduler::filter ( $scheduler_available_filters_real = any2array($scheduler_available_filters) } - if $baremetal_scheduler_default_filters or $scheduler_use_baremetal_filters or $scheduler_host_manager { - warning('The scheduler_host_manager, baremetal_scheduler_default_filters and \ -scheduler_use_baremetal_filters parameters are deprecated and will have \ -no effect. Baremetal scheduling now uses custom resource classes.') - } - if !is_service_default($isolated_images) and !empty($isolated_images){ validate_legacy(Array, 'validate_array', $isolated_images) $isolated_images_real = join($isolated_images, ',') diff --git a/releasenotes/notes/cleanup-baremetal-scheduling-opts-250e52ff28ae2fc3.yaml b/releasenotes/notes/cleanup-baremetal-scheduling-opts-250e52ff28ae2fc3.yaml new file mode 100644 index 000000000..f2aaf1afb --- /dev/null +++ b/releasenotes/notes/cleanup-baremetal-scheduling-opts-250e52ff28ae2fc3.yaml @@ -0,0 +1,8 @@ +--- +upgrade: + - | + The following deprecated parameters have been removed. + + - ``nova::scheduler::filter::baremetal_scheduler_default_filters`` + - ``nova::scheduler::filter::scheduler_use_baremetal_filters`` + - ``nova::scheduler::filter::scheduler_host_manager`` diff --git a/spec/classes/nova_scheduler_filter_spec.rb b/spec/classes/nova_scheduler_filter_spec.rb index d7d1dfba7..732365b45 100644 --- a/spec/classes/nova_scheduler_filter_spec.rb +++ b/spec/classes/nova_scheduler_filter_spec.rb @@ -52,18 +52,6 @@ describe 'nova::scheduler::filter' do it { is_expected.to contain_nova_config('filter_scheduler/available_filters').with_value(['nova_filter1','nova_filter2']) } end - context 'when overriding deprecated params' do - let :params do - { - :scheduler_use_baremetal_filters => true, - :baremetal_scheduler_default_filters => ['ExactRamFilter','ExactDiskFilter','ExactCoreFilter'], - } - end - - it { is_expected.to_not contain_nova_config('filter_scheduler/use_baremetal_filters') } - it { is_expected.to_not contain_nova_config('filter_scheduler/baremetal_enabled_filters') } - end - context 'when overriding params with empty arrays' do let :params do { :isolated_images => [],