Skip to content

Commit

Permalink
Define separator explicitly in set_value call
Browse files Browse the repository at this point in the history
set_value method of Puppet::Util::IniFile uses '' for separator when
called without the separator argument[1]. Set the argument explicitly
to avoid broken rendering. Note that the argument was added in
puppetlabs-inifile 2.0.0 so we don't need to bump the min version.

[1] puppetlabs/puppetlabs-inifile#544

Closes-Bug: #2081303
Change-Id: I6112c6d12e860bd4b4030e5f784952a7358f617f
(cherry picked from commit 93f15aa)
(cherry picked from commit 5769e2a)
(cherry picked from commit a0dbaca)
  • Loading branch information
kajinamit committed Sep 24, 2024
1 parent 897269e commit 3f54cf7
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion lib/puppet/provider/swift_storage_policy/ruby.rb
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,7 @@ def write_policy
end
self.class.policy_settings.each do |property_name, setting_name|
unless @property_flush[property_name].nil?
config.set_value(policy_title, setting_name, @property_flush[property_name])
config.set_value(policy_title, setting_name, '=', @property_flush[property_name])
end
end
config.save
Expand Down

0 comments on commit 3f54cf7

Please sign in to comment.