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

Need legal review, licenses compatibility of third party dependencies with the OpenTelemetry license #2383

Closed
marcalff opened this issue Oct 7, 2024 · 4 comments

Comments

@marcalff
Copy link
Member

marcalff commented Oct 7, 2024

Context

In the opentelemetry-cpp repository, all the code is covered by the Apache-2.0 License.

For a new feature, to implement the yaml file configuration as defined in the opentelemetry-configuration repository, we need ... a YAML parser in C++.

Not many parsers are available in the C++ world, and after evaluating several, the library that meets our technical requirements is rapidyaml.

This parser, rapidyaml, is covered by the MIT License.

This raises the question whether the MIT and Apache-2.0 licenses are compatible, and can be used in the same product (the OpenTelemetry-cpp library).

Request

Disclaimer: I am not a layer.

Hence, we need some legal advice, to clarify if using:

Furthermore, it would be very desirable to document a list of licenses that are pre-approved to be used with Apache-2.0 and OpenTelemetry, to provide early guidance to maintainers.

This will help every SIG, when evaluating candidate third party dependencies, to make sure these dependencies not only meet technical requirements but also legal requirements.

Thanks for your help.

cc @open-telemetry/cpp-maintainers @open-telemetry/configuration-maintainers @svrnm

@lalitb
Copy link
Member

lalitb commented Oct 8, 2024

According to the CNCF policy outlined here, it appears that the MIT license is included in the approved set of licenses for third-party components.
disclaimer - I am not a legal expert on licensing or copyright matters :)

@marcalff
Copy link
Member Author

marcalff commented Oct 8, 2024

Thanks @lalitb , this is exactly what I was looking for.

@trask
Copy link
Member

trask commented Oct 8, 2024

@marcalff does this address your issue? thanks!

@marcalff
Copy link
Member Author

marcalff commented Oct 8, 2024

Yes, and closing this issue then.

@marcalff marcalff closed this as completed Oct 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants