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

Add nftables wrapper support for RHEL8 #794

Merged
merged 1 commit into from
Nov 29, 2018

Conversation

mwhahaha
Copy link
Contributor

In RHEL8, iptables is replaced with nftables under the covers. In order
to allow for the firewall module to continue to function, this change
updates the redhat firewall configuration to pull in the nftables
packages.

https://access.redhat.com/documentation/en-us/red_hat_enterprise_linux/8-beta/html/8.0_beta_release_notes/new-features#networking_2

@EmilienM
Copy link
Contributor

👍

@HelenCampbell
Copy link
Contributor

Hey @mwhahaha , this is looking really good! Would you be able to address the errors thrown by travis? I think they should be purely syntactic changes. If it goes green I can look at getting this merged in. Thanks!

In RHEL8, iptables is replaced with nftables under the covers. In order
to allow for the firewall module to continue to function, this change
updates the redhat firewall configuration to pull in the nftables
packages.

https://access.redhat.com/documentation/en-us/red_hat_enterprise_linux/8-beta/html/8.0_beta_release_notes/new-features#networking_2
@mwhahaha
Copy link
Contributor Author

@HelenCampbell addressed

@HelenCampbell
Copy link
Contributor

This went green through our adhoc pipeline so merging this now. Thank you for addressing those issues and for your contribution!

@HelenCampbell HelenCampbell merged commit e400c80 into puppetlabs:master Nov 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants