On Wed, Jul 15, 2015 at 11:36 PM <[email protected]> wrote:

> Hi,
>
> thanks for your feedback. I provided some more information, unfortunately
> the suggested quoting did not work.
>

Oh, I wouldn't recommend quoting numbers; just strings.


>
> installed software
> ##################
> facter                             2.4.1-1
> hiera                              1.3.4-1
> puppet                             3.7.5-1
> puppet-common                      3.7.5-1
>
> scenarios
> #########
>
> 1 - values in quotes
> ====================
> postgresql::server::config_entry { 'max_standby_archive_delay':  value =>
> '-1'}
>
>
> result
> ------
> postgresql.conf contains value in quotes:
> ...
> max_standby_streaming_delay = '-1'
> ...
>
>
> 2 - values without quotes
> =========================
> postgresql::server::config_entry { 'max_standby_archive_delay':  value =>
> -1}
>
>
> result
> ------
> puppet run fails:
>
> Error:
> /Stage[main]/Main/Node[puppet02-node1]/Postgresql::Server::Config_entry[max_standby_archive_delay]/Postgresql_conf[max_standby_archive_delay]:
> Could not evaluate: undefined method `match' for -1:Fixnum
>

Thanks for this. It's probably coming from this line
https://github.com/puppetlabs/puppetlabs-postgresql/blob/master/lib/puppet/provider/postgresql_conf/parsed.rb#L26
but without --trace I can't be sure. Most likely that can just be moved
into the conditional logic when the provider checks for Fixnum and it could
work.


>
>
> 3 - alternative quotes
> ======================
> postgresql::server::config_entry { 'max_standby_archive_delay':  'value =>
> -1'}
>
> result
> ------
> puppet run fails:
>
> Error: Could not retrieve catalog from remote server: Error 400 on SERVER:
> Could not parse for environment production: Syntax error at 'value => -1';
> expected '}' at
> /etc/puppet/environments/production/manifests/nodes/puppet02-node1.pp:16 on
> node puppet02-node1
> Warning: Not using cache on failed catalog
> Error: Could not retrieve catalog; skipping run
>
> 4 - alternative quotes2
> =======================
> postgresql::server::config_entry { 'max_standby_archive_delay':  `value =>
> -1`}
>
> result
> ------
> puppet run fails:
>
> Error: Could not retrieve catalog from remote server: Error 400 on SERVER:
> Could not parse for environment production: Could not match `value at
> /etc/puppet/environments/production/manifests/nodes/puppet02-node1.pp:16 on
> node puppet02-node1
> Warning: Not using cache on failed catalog
> Error: Could not retrieve catalog; skipping run
>
>
>  --
> You received this message because you are subscribed to the Google Groups
> "Puppet Users" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to [email protected].
> To view this discussion on the web visit
> https://groups.google.com/d/msgid/puppet-users/af5f4724-a599-4151-a3c4-adb0488451a2%40googlegroups.com
> <https://groups.google.com/d/msgid/puppet-users/af5f4724-a599-4151-a3c4-adb0488451a2%40googlegroups.com?utm_medium=email&utm_source=footer>
> .
> For more options, visit https://groups.google.com/d/optout.
>

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Users" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/puppet-users/CAJaQvGAXduqsnCQUT3xriFr2JAwrh%3DHOeU0MTNwhTnqHhh1q-g%40mail.gmail.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to