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

Audit needed for the Hashicorp MPL -> BUSL license change #305

Closed
5 tasks done
Tracked by #308
leogr opened this issue Aug 28, 2023 · 4 comments
Closed
5 tasks done
Tracked by #308

Audit needed for the Hashicorp MPL -> BUSL license change #305

leogr opened this issue Aug 28, 2023 · 4 comments
Assignees
@leogr
Labels
kind/documentation Improvements or additions to documentation

Comments

@leogr
Copy link
Member

leogr commented Aug 28, 2023 •
edited
Loading

We've conducted an initial scan following the CNCF input request after the Hashicorp MPL -> BUSL license change.

The lists below were generated after examining the go.sum packages across the entire Falcosecurity organization.

From this initial audit, we've determined that we are NOT using any BUSL-licensed packages 🥳 Thus, we are unaffected by the MPL -> BUSL license transition, which is positive.

Additionally, we identified some Hashicorp packages under MPL 2.0 that we are using without a CNCF Governing Board exception.

For context, according to the CNCF IP Policy, all 3rd-party dependencies must either be Apache 2.0 licensed OR listed in the Approved Licenses for Allowlist OR have an exception approved by the Governing Board (see already approved license exceptions).

MPL2'd packages being used without an exception

[ACTION NEEDED]! @falcosecurity/core-maintainers

cc @falcosecurity/driverkit-maintainers @falcosecurity/event-generator-maintainers @falcosecurity/falcoctl-maintainers @falcosecurity/falcosidekick-maintainers @falcosecurity/kilt-maintainers @falcosecurity/plugins-maintainers

Please carefully evaluate the possibility of removal for these Go dependencies listed in the table below. If that's not feasible, we must submit a ticket to the CNCF for review and request a license exception (I can take care of that once we have completed the evaluation of them one by one).

It's worth noting that some of these packages might not be in active use. A straightforward cleanup might suffice to remove them:

go get -u
go mod tidy
License Package Used by Note
MPL-2.0 github.com/hashicorp/consul/sdk falcosidekick plugins An exception request for github.com/hashicorp/consul/api exists already
MPL-2.0 github.com/hashicorp/logutils event-generator falcoctl falcosidekick plugins This is likely an indirect dependency. Need investigation.
MPL-2.0 github.com/hashicorp/memberlist event-generator falcoctl falcosidekick plugins See cncf/foundation#624
MPL-2.0 github.com/hashicorp/terraform-plugin-framework kilt See cncf/foundation#187 and cncf/foundation#619. Evaluate replacing: https://github.com/ko-build/terraform-provider-ko
MPL-2.0 github.com/hashicorp/terraform-plugin-go kilt same as above

Cleanups in progress
Beta Give feedback
  1. chore: updated project deps. driverkit#286
    approved area/build dco-signoff: yes kind/cleanup lgtm size/XXL
    EXONER4TED leogr
  2. chore: bumped deps through go get -u falcoctl#315
    approved dco-signoff: yes kind/cleanup lgtm size/XXL
    leogr
  3. build: switch to Go 1.21 and tidy deps event-generator#85
    approved dco-signoff: yes kind/cleanup lgtm size/XXL
    FedeDP
  4. chore(build/registry): updated go deps. plugins#324
    approved area/registry dco-signoff: yes kind/cleanup lgtm size/XXL
    leogr
  5. update dependencies falcosidekick#619
    approved area/build area/outputs dco-signoff: yes kind/cleanup lgtm size/XXL
    fjogeleit leogr

Already allowed Hashicorp packages.

The packages listed below are already permitted, either due to inclusion in the allowlist or because they have a GB-approved exception. Therefore, no additional action is required. We can continue to use them without concerns.

Allowlist / Exception License Package
2023-06-27 MPL-2.0 github.com/hashicorp/consul/api
2019-03-11 MPL-2.0 github.com/hashicorp/errwrap
2019-03-11 MPL-2.0 github.com/hashicorp/go-cleanhttp
Allowlist MIT github.com/hashicorp/go-hclog
2023-06-27 MPL-2.0 github.com/hashicorp/go-immutable-radix
Allowlist MIT github.com/hashicorp/go-msgpack
2019-03-11 MPL-2.0 github.com/hashicorp/go-multierror
2023-06-27 MPL-2.0 github.com/hashicorp/go-plugin
2021-07-19 MPL-2.0 github.com/hashicorp/go-retryablehttp
2023-06-27 MPL-2.0 github.com/hashicorp/go-rootcerts
2023-06-27 MPL github.com/hashicorp/go-secure-stdlib/parseutil
2023-06-27 MPL github.com/hashicorp/go-secure-stdlib/strutil
2023-06-27 MPL-2.0 github.com/hashicorp/go-sockaddr
Allowlist MIT github.com/hashicorp/go-syslog
2023-06-27 MPL-2.0 github.com/hashicorp/go-uuid
Allowlist BSD-3-Clause github.com/hashicorp/go.net
2019-03-11 MPL-2.0 github.com/hashicorp/golang-lru
2019-03-11 MPL-2.0 github.com/hashicorp/hcl
Allowlist MIT github.com/hashicorp/mdns
2023-06-27 MPL-2.0 github.com/hashicorp/raft
2023-06-27 MPL-2.0 github.com/hashicorp/serf
2023-06-27 MPL-2.0 github.com/hashicorp/vault/api
2023-06-27 MPL-2.0 github.com/hashicorp/yamux

N.B. The 2023-06-27 license execptions file inaccurately indicates that that github.com/hashicorp/vault is licensed under MPL-2.0 and has GB exception approval as of 2023-06-27. In reality, github.com/hashicorp/vault is BUSL-1.1, while only its sub-package github.com/hashicorp/vault/api is MPL-2.0.
@leogr leogr added the kind/documentation Improvements or additions to documentation label Aug 28, 2023
@leogr leogr self-assigned this Aug 28, 2023
@leogr leogr mentioned this issue Aug 28, 2023
Closed
This was referenced Aug 29, 2023
Merged
Merged
Merged
@leogr leogr mentioned this issue Aug 29, 2023
Merged
@leogr
Copy link
Member Author

leogr commented Aug 29, 2023

It seems that consul/sdk, logutils, and memberlist were transitive dependencies of other packages.

This was the dep graph for them in the event-generator (before I fixed it):

❯ go mod graph | grep consul/sdk                                                                                                                                                                          
github.com/hashicorp/consul/[email protected] github.com/hashicorp/consul/[email protected]                                                                                                                           
github.com/hashicorp/consul/[email protected] github.com/hashicorp/[email protected]                                                                                                                          
github.com/hashicorp/consul/[email protected] github.com/hashicorp/[email protected]                                                                                                                             
github.com/hashicorp/consul/[email protected] github.com/hashicorp/[email protected]                                                                                                                               
github.com/hashicorp/consul/[email protected] github.com/hashicorp/[email protected]                                                                                                                            
github.com/hashicorp/consul/[email protected] github.com/pkg/[email protected]                                                                                                                                      
github.com/hashicorp/consul/[email protected] github.com/stretchr/[email protected]                                                                                                                                
github.com/hashicorp/consul/[email protected] golang.org/x/[email protected]                                                                                                               
github.com/hashicorp/consul/[email protected] github.com/davecgh/[email protected]                                                                                                                                 
github.com/hashicorp/consul/[email protected] github.com/fatih/[email protected]                                                                                                                                     
github.com/hashicorp/consul/[email protected] github.com/kr/[email protected]                                                                                                                                       
github.com/hashicorp/consul/[email protected] github.com/mattn/[email protected]                                                                                                                              
github.com/hashicorp/consul/[email protected] github.com/mattn/[email protected]                                                                                                                                
github.com/hashicorp/consul/[email protected] github.com/pmezard/[email protected]                                                                                                                              
github.com/hashicorp/consul/[email protected] gopkg.in/[email protected]                                                                                                              
github.com/hashicorp/consul/[email protected] gopkg.in/[email protected]
                                                                                                                                           
❯ go mod graph | grep github.com/hashicorp/logutils                                                                                                                                                       
github.com/hashicorp/[email protected] github.com/hashicorp/[email protected]                                                                                                                                    

❯ go mod graph | grep github.com/hashicorp/memberlist                                                                                                                                                     
github.com/hashicorp/consul/[email protected] github.com/hashicorp/[email protected]                                                                                                                            
github.com/hashicorp/[email protected] github.com/hashicorp/[email protected]                                                                                                                                  
github.com/hashicorp/[email protected] github.com/armon/[email protected]                                                                                                     
github.com/hashicorp/[email protected] github.com/davecgh/[email protected]                                                                                                                                  
github.com/hashicorp/[email protected] github.com/google/[email protected]                                                                                                         
github.com/hashicorp/[email protected] github.com/hashicorp/[email protected]
github.com/hashicorp/[email protected] github.com/hashicorp/[email protected]
github.com/hashicorp/[email protected] github.com/hashicorp/[email protected]
github.com/hashicorp/[email protected] github.com/hashicorp/[email protected]
github.com/hashicorp/[email protected] github.com/miekg/[email protected]
github.com/hashicorp/[email protected] github.com/pascaldekloe/[email protected] 
github.com/hashicorp/[email protected] github.com/pmezard/[email protected]
github.com/hashicorp/[email protected] github.com/sean-/[email protected]
github.com/hashicorp/[email protected] github.com/stretchr/[email protected]
github.com/hashicorp/[email protected] golang.org/x/[email protected]

I had to reset the go.mod and switch to Go 1.21 to remove them from the event generator. Likely, the latest versions of required packages do not carry any unwanted Hashicorp packages anymore. See falcosecurity/event-generator#85

@FedeDP
Copy link
Contributor

FedeDP commented Aug 29, 2023

I am doing the same (ie: switching to new go and running go get -u and go mod tidy) on:

Consul is no more greppable in them.

@leogr leogr mentioned this issue Aug 31, 2023
Closed
@leogr leogr mentioned this issue Dec 15, 2023
Closed
@leogr
Copy link
Member Author

leogr commented Jan 11, 2024

With #347, all tasks are done now!
/close

@poiana
Copy link
Contributor

poiana commented Jan 11, 2024

@leogr: Closing this issue.

In response to this:

With #347, all tasks are done now!
/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@poiana poiana closed this as completed Jan 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@leogr leogr

Labels
kind/documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@leogr @FedeDP @poiana