dhparam doesn't work without 'ensure'

[anthonyheckmann]

This doesn't create the file unless ensure is uncommented, which should be the default operation.

dhparam { '/etc/openvpn/dh2048.pem': 
    size         => 2048,
    require    => Package['openvpn'],
    # ensure => 'present'    
 }

[anthonyheckmann]

It might be I'm just not using it correctly. Though the docs say this:

dhparam { '/etc/openvpn/dh2048.pem': 
    size         => 2048
}

should work.

[raphink]

OK. Requiring ensure doesn't shock me, so would it be acceptable to update the docs?

[anthonyheckmann]

Seems reasonable. Not sure what happens when no params are used at all. Both cases might need to be updated.

…

On Aug 9, 2017 5:15 AM, "Raphaël Pinson" ***@***.***> wrote: OK. Requiring ensure doesn't shock me, so would it be acceptable to update the docs? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#90 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAXmRMl4DEskdX5Q08oCDAo2Zh5LjQtWks5sWXiagaJpZM4OwxR5> .

[tlcowling]

@anthonyheckmann I think sticking the openssl:: at the beginning is what you want so it collects 'present' by default from the puppet type. This happened to me too just before AND also while doing stuff with openvpn!

openssl::dhparam { '/etc/openvpn/dh2048.pem': 
    size         => 2048
}

Agree with updating the doc to reflect this!