-
Notifications
You must be signed in to change notification settings - Fork 210
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
Ensure Kubeflow meets the CNCF IP Policy #661
Comments
Hi WG leads, could you confirm that your components are licensed under an OSI-approved open source license to meet the required (e) stated under CNCF IP Policy? This is related to https://github.com/cncf/foundation/blob/main/allowed-third-party-license-policy.md#approved-licenses-for-allowlist which is part of #656 |
I can confirm that mpi-operator is licensed as Apache 2.0 |
KFP-Tekton is licensed under Apache 2.0. |
Sorry folks, looks like simple look up of LICENSE files in each repo would be sufficient for the requirement (e).
No additional action is needed for requirement (e), unless you want to add any comments. |
Hi everyone, I am no longer able to continue on this issue, but @rimolive has volunteered to work on Kubeflow CLA/DCO migration with the interim steering committee to ensure Kubeflow meets the CNCF IP Policy and to update GitHub to ensure DCO or CLA are enabled for all GitHub repositories of the project. Completion of the tasks should allow this issue to be closed and should be reported back to the main issue cncf/toc#1139 as well. Thanks! |
@rimolive - James Wu and I discussed in Kubeflow Steering Committee call. We approved this migration to the CNCF CLA/DCO. Do we need to enable one or both of these options ? |
@jbottum The CLA is much more comprehensive than the standard DCO (and we currently don't have a DCO requirement, so it would require contributors to start using the "sign-off" section at the end of their commits if we adopted it). |
@thesuperzapper I tend to agree with your analysis. I propose to start with the CNCF CLA option (only). If some contributors find that unusable and they want/need the DCO, then we can add DCO. @james-jwu - your thoughts ? |
@jbottum I added an item in the next Kubeflow Community meeting to discuss the differences. |
I'd recommend using DCO. Argo project has switched from CLA to DCO before and here's our rationale:
|
@terrytangyuan The Linux kernel as well as most CNCF projects use the DCO. - do you have a sense of how DCO adoption percentage-wise? |
@james-jwu I don't know specific percentage (perhaps CNCF can get us a number) but I just randomly picked several graduated projects and they all seem to use DCO. |
@james-jwu I did that research. 22 out of 24 Graduated CNCF projects are confirmed that they use DCO. I have more info to share in the next Kubeflow community meeting. |
Bumping this issue. Since KSC agreed to use DCO and it is implemented the PR checks in Kubeflow repositories, can we consider it done? Or there are still missing action items? |
This is probably something that can be decided in the community call or KSC meeting. |
DCO/CLA related requirements are met. There's one remaining item from this list though:
Looks like @zijianjoy is working on it based on cncf/toc#1139 (comment). @zijianjoy Can you confirm that this is done? |
https://github.com/amye is going to be the best person for sharing the progress on this item. |
@amye Do you have an update on that? Thanks! |
You'll want to do this over in the TOC repo, we have cncf/toc#1139 open for you! |
We have enabled DCO for Kubeflow, as a result, I believe the CNCF IP Policy item is completed. I am going to close this item as finished, feel free to comment if not the case. |
Part of cncf/toc#1139
As part of CNCF onboarding, the Kubeflow community must fulfill the CNCF IP Policy.
This issue will be used to track progress toward meeting the CNCF IP Policy requirements.
Interim Steering Committee @james-jwu @theadactyl @jbottum
Katib: @andreyvelich @gaocegege @johnugeorge @tenzen-y
Manifests: @elikatsis @kimwnasptd @PatrickXYS @StefanoFioravanzo @yanniszark @juliusvonkohout
MPI Operator: @alculquicondor @rongou @terrytangyuan
Notebooks: @elikatsis @kimwnasptd @thesuperzapper @StefanoFioravanzo @yanniszark
Pipelines: @chensun @zijianjoy @james-jwu @IronPan
Pipelines on Tekton: @animeshsingh @ckadner @fenglixa @pugangxa @ScrapCodes @Tomcli @yhwang
Training Operators: @gaocegege @Jeffwan @johnugeorge @zw0610 @terrytangyuan @tenzen-y @andreyvelich
The text was updated successfully, but these errors were encountered: