Bug #10997
firewall: 'proto' property needs to support 'ipencap' protocol
| Status: | Closed | Start date: | 11/21/2011 | |
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assignee: | % Done: | 0% |
||
| Category: | firewall | Spent time: | - | |
| Target version: | - | |||
| Keywords: | Branch: | |||
| Votes: | 0 |
Description
IPIP tunneling is not supported. It would be real nice to support more protocols.
Related issues
History
Updated by Sharif Nassar 6 months ago
https://github.com/puppetlabs/puppetlabs-firewall/pull/35
Updated by Ken Barber 6 months ago
- Category set to firewall
- Status changed from Unreviewed to Merged - Pending Release
Merged.
Updated by Ken Barber 6 months ago
- Status changed from Merged - Pending Release to Requires CLA to be signed
Bah – I forgot to ask you to sign the CLA. I guess I presumed you had already signed it. Can you sign it now for me?
Updated by Sharif Nassar 6 months ago
Has nobody pointed out that the CLA is stupid yet ?
The code is Apache licensed. I would not submit patches to a project without an open source license. The CLA reads almost exactly the same as the Apache license. What is the point of a second agreement other than to annoy contributors into NOT contributing to your software.
When the Puppet CLA is on this list, I will reconsider signing it. But even then, extra hurdles to contribution are stupid.
Fondest regards,
Sharif
Updated by James Turnbull 6 months ago
Sharif
The Puppet CLA IS the Apache CLA with some minor modifications. It’s required to effectively use the Apache license for Puppet Labs. Signing the Apache Foundation’s CLA doesn’t have any relationship to signing the Puppet Labs CLA. Every project that uses the Apache license and a CLA (and not an Apache Foundation project) will require you sign a CLA specific to that project.
Updated by Ken Barber 6 months ago
This serves as a reasonable explanation for why CLA’s are used:
http://www.oss-watch.ac.uk/resources/cla.xml
If you still have concerns let us know.
I can’t accept submissions without a signed CLA generally, so let us know what you want to do.
Updated by Sharif Nassar 6 months ago
James,
I haven’t signed the Apache CLA, because I do not desire commit access to their repository. I’m content merely giving back in the form of patches if I should be so lucky as to be useful. I only brought it up, as this module is Apache licensed. However, as long as we are discussing their CLA, I will contrast their documentation with PuppetLabs.
- The description of ASF Roles describes several different roles in their meritocracy. A ‘developer’ contributes code. A ‘committer’ is a developer with write access to the code repository AND has signed the Apache CLA AND has an apache.org email address.
- The ASF desires contributors sign the Apache CLA, but it is evidently not required. See here.
- The Apache HTTP patches project page makes no mention of signing the CLA in order to submit patches.
Meanwhile, the Oracle OCA has the more draconian rules that align with Puppet Labs. (eg: all contributors must sign!) Would you rather be compared to the Apache Software Foundation or Oracle ?
In closing, if you decide to grant me commit access and a @puppetlabs.com email address, I will reconsider signing.
Thanks,
Sharif
Updated by James Turnbull 6 months ago
Sharif – then regretfully we can’t accept your patch. The move to Apache and the reasons behind the CLA are explained here: http://docs.puppetlabs.com/guides/faq.html#change-to-apache-license. The precise section covering CLAs is here:
What does this mean if I or my company have or want to contribute code to Puppet? As part of this license change, Puppet Labs has approached every existing contributor to the project and asked them to sign a Contributor License Agreement or CLA. Puppet Labs has adapted the standard Apache Foundation’s CLA for this purpose. Signing this CLA for yourself or your company provides both you and Puppet Labs with additional legal protections, and confirms: * That you own and are entitled to the code you are contributing to Puppet * That you are willing to have it used in distributions This gives assurance that the origins and ownership of the code cannot be disputed in the event of any legal challenge.
This isn’t draconian or excessive and is common to a large number of open source companies in the industry. I am sorry if this isn’t acceptable to you but we’re unable to make any exceptions.
Kind Regards
James Turnbull
Updated by Sharif Nassar 6 months ago
Good news!
Media Temple has signed the CLA, and faxed it just now.
Now I get to maintain my ridiculous principles, and Puppet Labs gets to do the same.
I submitted a new pull request with a modified LICENSE file.
Respectfully,
Sharif
Updated by James Turnbull 6 months ago
- Status changed from Requires CLA to be signed to In Topic Branch Pending Review
Thanks Sharif!
Updated by Ken Barber 6 months ago
- Status changed from In Topic Branch Pending Review to Merged - Pending Release
Updated by Sharif Nassar 6 months ago
More test fixtures..
https://github.com/puppetlabs/puppetlabs-firewall/pull/39
Updated by Jonathan Boyett 6 months ago
Merged in additional fixtures.
Updated by Jonathan Boyett 6 months ago
- Assignee set to Sharif Nassar
Updated by Ken Barber 6 months ago
- Status changed from Merged - Pending Release to Closed