-
Notifications
You must be signed in to change notification settings - Fork 256
Disable old linuxfoundation CLA #379
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
Comments
Can we please hold off on this for at least another week? At least Splunk and Grafana Labs are impacted by the new CLA and will require time to get it formally approved. |
Yup, just a tracking issue, will wait another 1.5 weeks. |
@flands would it help if instead of re-signing, I asked the Linux Foundation to migrate your existing CLA? |
@open-telemetry/admins - We should undo the Community repository from enforcing the EasyCLA for now. I think Community should be the last repository to enforce the new CLA as we have seen that there are large gaps in who got the communication around the new process. (#309) |
I'll revert! |
@flands it is my understanding that the CLA itself is unchanged, it's only the mechanism for checking it. When I did a test PR, all I had to do was to confirm that I still work at Uber, I was not required to re-sign the CLA. Are you seeing something different? |
Filed https://jira.linuxfoundation.org/servicedesk/customer/portal/4/SUPPORT-1402 to ask the existing CLAs to be migrated to avoid the need to re-sign |
|
Grafana's CLA has been migrated. I'm going to re-enable new CLA on community now. |
@bogdandrutu how did discussion at maintainer meeting go? Do we need another week or am I good to swap the CLAs as of Friday? |
It's now been 3 weeks, I'm going to go ahead and globally change, unless people object. |
@open-telemetry/admins I've updated all but the ones for which I have no access as a GC member. |
open-telemetry/opentelemetry-operator doesn't show EasyCLA as a check |
same with open-telemetry/opentelemetry-python-contrib - no checks open-telemetry/opentelemetry-auto-instr-ruby has no files |
A dummy PR needs to be opened to force EasyCLA checks to run then. I'll do that. |
@open-telemetry/ruby-maintainers do you need the auto-instr repo? or can we delete it until we need it? p.s. the name is out of spec with the naming of other language auto-instr where the language comes first. |
@yurishkuro dummy PRs opened against otel-operator and otel-python-contrib, it should show now. |
thanks, Liz, I enabled the checked in those two. |
Great. I think we can declare this migration done. I've submitted a ticket to Linux Foundation support to ask them to disable the old CLA bot. |
I deleted |
one last item, @yurishkuro: Heather from CNCF/LF says we need to:
|
that is correct, the github app is how we've integrated EasyCLA so the legacy webhook can be removed. |
deleted webhook. |
Opening a test PR to verify, but we can consider this closed. |
Tracking issue for completion of the EasyCLA migration (to be completed on June 18)
The text was updated successfully, but these errors were encountered: