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

fix kernel incompatibilities #72

Merged
merged 1 commit into from
Sep 13, 2019

Conversation

foxxx0
Copy link
Member

@foxxx0 foxxx0 commented Sep 13, 2019

Certain kernel modules and thus iptables functionality was introduced at
later releases, so we need to properly reflect that in our default chain
initialization procedure.

INPUT chain for nat table was introduced with 2.6.36

ip6table_nat kernel module for NAT functionality with IPv6 was
introduced with 3.17

This commit implements the required conditional constraints and includes
the rspec tests to validate it.

@foxxx0 foxxx0 added the bug Something isn't working label Sep 13, 2019
@foxxx0 foxxx0 self-assigned this Sep 13, 2019
manifests/config.pp Outdated Show resolved Hide resolved
manifests/config.pp Outdated Show resolved Hide resolved
@foxxx0 foxxx0 force-pushed the fix-kernel-incompatibilities branch 2 times, most recently from c3b29e9 to f4d4351 Compare September 13, 2019 10:32
Certain kernel modules and thus iptables functionality was introduced at
later releases, so we need to properly reflect that in our default chain
initialization procedure.

`INPUT` chain for `nat` table was introduced with 2.6.36

`ip6table_nat` kernel module for NAT functionality with IPv6 was
introduced with 3.17

This commit implements the required conditional constraints and includes
the rspec tests to validate it.
@bastelfreak bastelfreak merged commit b9e465c into voxpupuli:master Sep 13, 2019
@foxxx0 foxxx0 deleted the fix-kernel-incompatibilities branch September 13, 2019 11:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants