Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

nftables::set can only be assigned to 1 table #100

Closed
AlexSamad opened this issue Jul 5, 2021 · 1 comment · Fixed by #102
Closed

nftables::set can only be assigned to 1 table #100

AlexSamad opened this issue Jul 5, 2021 · 1 comment · Fixed by #102
Labels
enhancement New feature or request

Comments

@AlexSamad
Copy link

Hi

I have a set of sets that I want to use in my
inet filter table as well as my ip nat table

the class only takes 1 table as input and as its not part of the title I can only have 1

nftables::set{ 'management_ip':
...
table => 'STRING'
}

maybe if it took an array of string.

@nbarrientos nbarrientos added the enhancement New feature or request label Jul 6, 2021
nbarrientos added a commit to nbarrientos/puppet-nftables that referenced this issue Jul 6, 2021
nbarrientos added a commit to nbarrientos/puppet-nftables that referenced this issue Jul 6, 2021
nbarrientos added a commit to nbarrientos/puppet-nftables that referenced this issue Jul 6, 2021
nbarrientos added a commit to nbarrientos/puppet-nftables that referenced this issue Jul 6, 2021
nbarrientos added a commit to nbarrientos/puppet-nftables that referenced this issue Jul 6, 2021
@nbarrientos
Copy link
Collaborator

Hi @AlexSamad

Thanks for the suggestion. I've submitted a change request covering your request :)

nbarrientos added a commit to nbarrientos/puppet-nftables that referenced this issue Jul 6, 2021
nbarrientos added a commit to nbarrientos/puppet-nftables that referenced this issue Jul 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants