Ewoud Kohl van Wijngaarden

Results 1159 comments of Ewoud Kohl van Wijngaarden

Any luck so far? I'd happy to make suggestions when you're stuck.

A parameter `exec_restorecon_on_change` (or similar) makes sense to me. Just for compatibility with previous versions I'm leaning to defaulting to false. An additional reason to do it inside the define...

This is a backwards incompatible change, but I can't apply the label.

No urgency. I'm fine with keeping this till the next major version.

In https://github.com/theforeman/puppet-pulpcore/pull/199 I'm testing if the approach works. A draft until then.

This now fails on https://tickets.puppetlabs.com/browse/MODULES-11018. I'll add a workaround.

All classes already have a `$config` parameter (or similar). Those can be set via Hiera and the additional lookup is redundant.

That suggests a packaging bug or possibly a locally installed pypi package where futures doesn't match that version spec. Not sure there's a lot we can do here.

I'd like to avoid adding workarounds here instead of fixing it upstream. A trivial reproducer on CentOS 7: ``` yum -y install epel-release yum -y install certbot certbot --help #...