search

voxpupuli / puppet-firewalld

Puppet module for managing firewalld
Apache License 2.0
40 stars 77 forks source link

Implement `instances` method allowing `puppet resource` to return current resources #295

Open n3mawashi opened 4 years ago

n3mawashi commented 4 years ago

I guess this sits somewhere between bug and feature enhancement. We use this command to check the syntax but also to understand what information is being returned.

How to reproduce (e.g Puppet code you use)

puppet resource firewalld_service

What are you seeing

Error: Could not run: To support listing resources of this type the 'firewall_cmd' provider needs to implement an 'instances' class method returning the current set of resources. We recommend porting your module to the simpler Resource API instead: https://puppet.com/search/docs?keys=resource+api

What behaviour did you expect instead

to return the list of services currently configured

nasa-dan commented 2 years ago

BUMP This would be really nice to have.