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

(#15038) add gre protocol to list of acceptable protocols #85

Merged
merged 1 commit into from
Jun 14, 2012
Merged

(#15038) add gre protocol to list of acceptable protocols #85

merged 1 commit into from
Jun 14, 2012

Conversation

jasonhancock
Copy link
Contributor

Add gre protocol to list of acceptable protocols. A resource such as this:

firewall { '001 accept gre':
  proto   => 'gre',
  action  => 'accept',
}

Was previously thowing an error:

Failed to apply catalog: Parameter proto failed: Invalid value "gre". Valid values are tcp, udp, icmp, ipv6-icmp, esp, ah, vrrp, igmp, ipencap, all.  at /etc/puppet/somemanifest.pp:27

With my patch applied, the rule gets created correctly:

# iptables -L | grep gre
ACCEPT     gre  --  anywhere             anywhere            /* 001 accept gre */ 

kbarber added a commit that referenced this pull request Jun 14, 2012
(#15038) add gre protocol to list of acceptable protocols
@kbarber kbarber merged commit 5355bc9 into puppetlabs:master Jun 14, 2012
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.

3 participants