openstack/puppet-swift
Code Issues Proposed changes

Merge "Avoid deprecated config_file_path"

Browse Source
This commit is contained in:
Zuul 2024-04-25 15:24:50 +00:00 committed by Gerrit Code Review
commit 1347526af1
3 changed files with 24 additions and 33 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
manifests/storage/all.pp
View File

@ -220,22 +220,19 @@ from 6002 to 6202 and will be changed in a later release")
}
swift::storage::server { "${account_port}":
type => 'account',
config_file_path => 'account-server.conf',
pipeline => $account_pipeline,
workers => $account_server_workers,
type => 'account',
pipeline => $account_pipeline,
workers => $account_server_workers,
}
swift::storage::server { "${container_port}":
type => 'container',
config_file_path => 'container-server.conf',
pipeline => $container_pipeline,
workers => $container_server_workers,
type => 'container',
pipeline => $container_pipeline,
workers => $container_server_workers,
}
swift::storage::server { "${object_port}":
type => 'object',
config_file_path => 'object-server.conf',
pipeline => $object_pipeline,
workers => $object_server_workers,
splice => $splice,

9
manifests/storage/node.pp
View File

@ -63,8 +63,7 @@ define swift::storage::node(
}
swift::storage::server { "60${name}0":
type => 'object',
config_file_path => 'object-server.conf',
type => 'object',
}
$ring_host = normalize_ip_for_uri($storage_local_net_ip)
@ -81,8 +80,7 @@ define swift::storage::node(
}
swift::storage::server { "60${name}1":
type => 'container',
config_file_path => 'container-server.conf',
type => 'container',
}
ring_container_device { "${ring_host}:60${name}1/${name}":
zone => $zone,
@ -90,8 +88,7 @@ define swift::storage::node(
}
swift::storage::server { "60${name}2":
type => 'account',
config_file_path => 'account-server.conf',
type => 'account',
}
ring_account_device { "${ring_host}:60${name}2/${name}":
zone => $zone,

33
spec/classes/swift_storage_all_spec.rb
View File

@ -81,28 +81,25 @@ describe 'swift::storage::all' do
end
it { is_expected.to contain_swift__storage__server(param_hash[:account_port]).with(
:type => 'account',
:config_file_path => 'account-server.conf',
:incoming_chmod => param_hash[:incoming_chmod],
:outgoing_chmod => param_hash[:outgoing_chmod],
:pipeline => param_hash[:account_pipeline] || ['account-server']
:type => 'account',
:incoming_chmod => param_hash[:incoming_chmod],
:outgoing_chmod => param_hash[:outgoing_chmod],
:pipeline => param_hash[:account_pipeline] || ['account-server']
)}
it { is_expected.to contain_swift__storage__server(param_hash[:object_port]).with(
:type => 'object',
:config_file_path => 'object-server.conf',
:incoming_chmod => param_hash[:incoming_chmod],
:outgoing_chmod => param_hash[:outgoing_chmod],
:pipeline => param_hash[:object_pipeline] || ['object-server'],
:splice => param_hash[:splice],
:rsync_timeout => param_hash[:rsync_timeout],
:rsync_bwlimit => param_hash[:rsync_bwlimit],
:type => 'object',
:incoming_chmod => param_hash[:incoming_chmod],
:outgoing_chmod => param_hash[:outgoing_chmod],
:pipeline => param_hash[:object_pipeline] || ['object-server'],
:splice => param_hash[:splice],
:rsync_timeout => param_hash[:rsync_timeout],
:rsync_bwlimit => param_hash[:rsync_bwlimit],
)}
it { is_expected.to contain_swift__storage__server(param_hash[:container_port]).with(
:type => 'container',
:config_file_path => 'container-server.conf',
:incoming_chmod => param_hash[:incoming_chmod],
:outgoing_chmod => param_hash[:outgoing_chmod],
:pipeline => param_hash[:container_pipeline] || ['container-server']
:type => 'container',
:incoming_chmod => param_hash[:incoming_chmod],
:outgoing_chmod => param_hash[:outgoing_chmod],
:pipeline => param_hash[:container_pipeline] || ['container-server']
)}
it { is_expected.to contain_class('rsync::server').with(