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.

Sign up for GitHub

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

Jump to bottom

Update dataplane license for CNCF requirements #79

Merged
merged 2 commits into from
Mar 6, 2023
Merged

Update dataplane license for CNCF requirements #79

merged 2 commits into from
Mar 6, 2023

Conversation

shaneutt
Copy link
Member

@shaneutt shaneutt commented Mar 3, 2023 •
edited
Loading

Changes the dataplane licensing from GPLv2 to dual-licensed GPLv2 + BSD-2-Clause as per CNCF requirements as outlined in #78.

Resolves #78

@shaneutt shaneutt added area/dataplane area/maintenance do-not-merge DEPRECATED. Indicates that a PR should not merge. Label can only be manually applied/removed. labels Mar 3, 2023
@shaneutt shaneutt added this to the v0.2.0 - TCPRoute Support milestone Mar 3, 2023
@shaneutt
chore: change dataplane licensing
0b48dd7 
This switches the dataplane licensing from GPLv2 to
dual-licensed GPLv2 + BSD-2-Clause as this licensing
scheme is considered accepted for inclusion in the
CNCF which we are migrating to.
@shaneutt shaneutt requested a review from robscott March 3, 2023 18:33
@shaneutt shaneutt removed the do-not-merge DEPRECATED. Indicates that a PR should not merge. Label can only be manually applied/removed. label Mar 3, 2023
astoycos
astoycos approved these changes Mar 6, 2023
View reviewed changes
Copy link
Member

@astoycos astoycos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

Thanks for taking care of this!

mlavacca
mlavacca approved these changes Mar 6, 2023
View reviewed changes
Copy link
Member

@mlavacca mlavacca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm 👍

@robscott
Copy link
Member

robscott commented Mar 6, 2023

Thanks!

/lgtm

@shaneutt shaneutt merged commit 6c192ad into main Mar 6, 2023
@shaneutt shaneutt deleted the dataplane-license-update branch March 6, 2023 16:51
@shaneutt shaneutt mentioned this pull request Mar 6, 2023
Closed
wheelerlaw
wheelerlaw reviewed Apr 4, 2023
View reviewed changes
Copy link
Contributor

@wheelerlaw wheelerlaw left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

as a contributor to the dataplane code, I approve of the change from GPLv2 to GPLv2 or BSD-2-Clause licensing

rushi47
rushi47 reviewed Apr 4, 2023
View reviewed changes
Copy link
Contributor

@rushi47 rushi47 left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks perfect, thanks for taking care of this !
I approve of the change from GPLv2 to GPLv2 or BSD-2-Clause licensing
Lgtm 👍

synthe102
synthe102 reviewed Apr 4, 2023
View reviewed changes
Copy link
Contributor

@synthe102 synthe102 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As a contributor to the dataplane code, I approve of the change from GPLv2 to GPLv2 or BSD-2-Clause licensing

This was referenced Apr 6, 2023
Closed
Closed
shaneutt added a commit to shaneutt/blixt that referenced this pull request Oct 11, 2023
@shaneutt
Update dataplane license for CNCF requirements (kubernetes-sigs#79)
bc599ae 
Changes the dataplane licensing from `GPLv2` to dual-licensed `GPLv2 +
BSD-2-Clause` as per CNCF requirements as outlined in kubernetes-sigs#78.

Resolves kubernetes-sigs#78
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@synthe102 synthe102 synthe102 left review comments

@rushi47 rushi47 rushi47 left review comments

@wheelerlaw wheelerlaw wheelerlaw left review comments

@mlavacca mlavacca mlavacca approved these changes

@astoycos astoycos astoycos approved these changes

@robscott robscott Awaiting requested review from robscott

Assignees
No one assigned
Labels
area/dataplane area/maintenance
Projects
None yet
Milestone
v0.2.0 - TCPRoute Support
Development

Successfully merging this pull request may close these issues.

Change dataplane licensing to meet CNCF requirements
7 participants
@shaneutt @robscott @wheelerlaw @rushi47 @astoycos @mlavacca @synthe102