Closed findmyname666 closed 4 years ago
@findmyname666 curious case, maybe due to custom tinydata. Can you tell me if changing line 338 in tp::conf https://github.com/example42/puppet-tp/blob/master/manifests/conf.pp#L338 from:
Hi @alvagante , Nope still the same. I think the issue is that it tries to set latest as ensure for file. Even though error doesn't match it :man_shrugging:
@findmyname666 oh , you're right, the problem is here: https://github.com/example42/puppet-tp/blob/master/manifests/install.pp#L294 . Honestly I almost never used the conf_hash param in tp::install. Have to fix it, thanks for the workaround anyway.
Honestly I almost never used the conf_hash param in tp::install. Actually i think that it is neat feature ;)
@findmyname666 can you please check if https://github.com/example42/puppet-tp/commit/6157efa46b12d3228ec14d421b5c99ab2a4f481c fixes the issue?
Hi @alvagante , busy time ! sorry to don't get back to you ...
i was thinking can you insert also data_module
to dir_defaults
please?
$dir_defaults = {
'ensure' => $ensure,
'ensure' => tp::ensure2dir($ensure),
'settings_hash' => $settings,
'data_module' => $data_module
}
Good catch @findmyname666 , and good that someone is using custom tinydata. Added in the referenced commit.
@alvagante can you publish it on forge?
@findmyname666 pushed version 2.4.2
Expected Behavior
To Create configuration file.
Actual Behavior
The code is failing on:
With added logging:
We can see that there are really source and content in var $file_params but content is empty.
Steps to Reproduce the Problem
When i change ensure to present everything works + when i passed ensure to conf_hash:
FYI my tinydata:
Specifications
Please add this info: