openstack/puppet-heat
Code Issues Proposed changes

Remove sahara class

Browse Source 
It was formally deprecated during the 2024.2 cycle[1].

[1] 7651af1cb3105395175bbb237ffdc0b5878d95a1

Change-Id: Ied60b5efe996462774484d98aa3156f662c4b14c
This commit is contained in:
Takashi Kajinami 2025-04-17 23:44:22 +09:00
parent 17585cffcf
commit 74e1d708e6
3 changed files with 4 additions and 99 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

48
manifests/clients/sahara.pp
View File

@ -1,48 +0,0 @@
# Class heat::clients::sahara
#
# DEPRECATED !!
# sahara client configuration
#
# == Parameters
#
# [*endpoint_type*]
# (Optional) Type of endpoint in Identity service catalog to use for
# communication with the OpenStack service.
# Defaults to $facts['os_service_default'].
#
# [*ca_file*]
# (Optional) Optional CA cert file to use in SSL communications.
# Defaults to $facts['os_service_default'].
#
# [*cert_file*]
# (Optional) Optional PEM-formatted certificate chain file.
# Defaults to $facts['os_service_default'].
#
# [*key_file*]
# (Optional) Optional PEM-formatted file that contains the private key.
# Defaults to $facts['os_service_default'].
#
# [*insecure*]
# (Optional) If set, then the server's certificate will not be verified.
# Defaults to $facts['os_service_default'].
#
class heat::clients::sahara (
$endpoint_type = $facts['os_service_default'],
$ca_file = $facts['os_service_default'],
$cert_file = $facts['os_service_default'],
$key_file = $facts['os_service_default'],
$insecure = $facts['os_service_default'],
) {
include heat::deps
warning('The heat::clients::sahara class is deprecated.')
heat::clients::base { 'clients_sahara':
endpoint_type => $endpoint_type,
ca_file => $ca_file,
cert_file => $cert_file,
key_file => $key_file,
insecure => $insecure,
}
}

4
releasenotes/notes/remove-sahara-3dfd15ae038ab838.yaml Normal file
View File

@ -0,0 +1,4 @@
---
upgrade:
- |
The ``heat::clients::sahara`` class has been removed.

51
spec/classes/heat_clients_sahara_spec.rb
View File

@ -1,51 +0,0 @@
require 'spec_helper'
describe 'heat::clients::sahara' do
shared_examples 'heat::clients::sahara' do
context 'with defaults' do
it 'configures defaults' do
is_expected.to contain_heat__clients__base('clients_sahara').with(
:endpoint_type => '<SERVICE DEFAULT>',
:ca_file => '<SERVICE DEFAULT>',
:cert_file => '<SERVICE DEFAULT>',
:key_file => '<SERVICE DEFAULT>',
:insecure => '<SERVICE DEFAULT>',
)
end
end
context 'with parameters' do
let :params do
{
:endpoint_type => 'publicURL',
:ca_file => '/path/to/ca.cert',
:cert_file => '/path/to/certfile',
:key_file => '/path/to/key',
:insecure => false,
}
end
it 'configures client parameters' do
is_expected.to contain_heat__clients__base('clients_sahara').with(
:endpoint_type => 'publicURL',
:ca_file => '/path/to/ca.cert',
:cert_file => '/path/to/certfile',
:key_file => '/path/to/key',
:insecure => false,
)
end
end
end
on_supported_os({
:supported_os => OSDefaults.get_supported_os
}).each do |os,facts|
context "on #{os}" do
let (:facts) do
facts.merge!(OSDefaults.get_facts())
end
it_behaves_like 'heat::clients::sahara'
end
end
end