openstack/puppet-swift
Code Issues Proposed changes

Fix missing argument of builder_file_path

Browse Source 
The change 63688a14e5 introduced support
of storage policies and since the change was merged builder_file_path
takes one argument which is policy_index .

This change fixes the function call without argument to avoid failure
caused by missing argument.

Closes-Bug: #1949334
Change-Id: I987a37cb0606b6c217f906fbae17a765e40f4580
(cherry picked from commit 350778570c)
(cherry picked from commit e7d560738d)
This commit is contained in:
Takashi Kajinami
2021-11-01 16:43:58 +09:00
parent 236a482391
commit a8dc75e183
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
lib/puppet/provider/swift_ring_builder.rb
View File

@@ -221,7 +221,7 @@ class Puppet::Provider::SwiftRingBuilder < Puppet::Provider
def weight=(weight) def weight=(weight)
swift_ring_builder( swift_ring_builder(
builder_file_path, builder_file_path(policy_index),
'set_weight', 'set_weight',
"d#{ring[device_path][:id]}", "d#{ring[device_path][:id]}",
resource[:weight] resource[:weight]
@@ -251,7 +251,7 @@ class Puppet::Provider::SwiftRingBuilder < Puppet::Provider
def meta=(meta) def meta=(meta)
swift_ring_builder( swift_ring_builder(
builder_file_path, builder_file_path(policy_index),
'set_info', 'set_info',
"d#{ring[device_path][:id]}", "d#{ring[device_path][:id]}",
"_#{resource[:meta]}" "_#{resource[:meta]}"