Matthias Baur

Errr, I can't explain the CI failures and are unable to locate the actual error in the CI logs. I don't think it's related, as I've only changed the indention...

@ozerovandrei I should have probably done this when creating this PR, sorry! I've updated the description with an explanation of the motivation behind this MR. Hope that helps! :)

@nikParasyr You're right! This worked out pretty easy! ```terraform locals { _rules_keys = [for rule in var.rules : "${coalesce(rule.direction, "x")}-${coalesce(rule.ethertype, "x")}-${coalesce(rule.description, "x")}-${coalesce(rule.protocol, "x")}-${coalesce(rule.port_range_min, "x")}-${coalesce(rule.port_range_max, "x")}-${coalesce(rule.remote_ip_prefix, "x")}-${coalesce(rule.remote_group_id, "x")}"] _rules = {...

I'm fine closing this PR. Thanks for your help! :)

I have to reopen this as I've found another usecase for the implementation of this PR. The suggested solution (https://github.com/terraform-provider-openstack/terraform-provider-openstack/pull/1458#issuecomment-1346311175) above doesn't work if keys derived from resource attributes e.g....

Hey @nikParasyr and @ozerovandrei, what do you think about merging/releasing this PR? :)

I guess since this PR is a total mess. There were 2 commits added which having nothing to do with the actual PR. I've cleaned it up myself (https://github.com/baurmatt/puppet-logrotate/tree/fix_pr15) but...

@thias What do you think about this? :)

Hey @thias, any update on this? :)

@thias Any update on this? Perhaps this module should be moved to @voxpupuli? https://voxpupuli.org/docs/#migrating-a-module-to-voxpupuli