Puppet

Applied in changeset commit:“3f61df8716d1ed66fb24bf7bcf64fcc2ebc711b9”.

I was getting this issue, so I applied this fix to my 0.24.8 and tried it out but I’m still getting the error:

err: Could not retrieve catalog: Failed to parse template backupninja/rdiff.conf.erb: private method `versioncmp' called for Puppet::Parser::TemplateWrapper::VersionComparer:Class at /etc/puppet/modules/backupninja/manifests/rdiff.pp:52

The code I’m using is:

<%- include Puppet::Util::Package
    class VersionComparer; extend Puppet::Util::Package; end
    
    if VersionComparer.versioncmp(Puppet.version,'0.24.6') >= 0
        %w{type host directory user sshoptions}.each do |v|
                if has_variable?(v) and instance_variable_get("@#{v}").to_s != "false"
                -%>
<%= v + ' = ' + instance_variable_get("@#{v}").to_s %>

I doubt what you’re experiencing can fall under this bug.

I’d suggest what you do is instead of trying to use the versioncmp function in your template, use it in the manifest.

something like:

$ver = versioncmp($puppetversion)

and then simply use the $ver variable in an if statement inside your template, this should be much cleaner and more readable as well.

As volcane says, versioncmp is meant to be used in the manifest.

If you use it in a template, you need to use ‘function_versioncmp’, or just do what the function does: ‘Puppet::Util::Package.versioncmp(args[0], args[1])’.