Co-authored-by checking fails unexpectedly

[ilrudie]

Summary

When I accept suggestions by maintainers of the istio project it causes the easyCLA check for that commit to fail with a message:

The email address for the commit (istio/istio.io@4f5e466) is not linked to the GitHub account, preventing the EasyCLA check. Consult this Help Article and GitHub Help to resolve. (To view the commit's email address, add .patch at the end of this PR page's URL.) For further assistance with EasyCLA, please submit a support request ticket.

Following the troubleshooting steps I do see both authors as valid "clickable" links to github profiles. The co-author's email address looks to be the same as his regular contributions to the project so the only thing I can figure is easyCLA isn't validating these co-authored entries correctly.

Background

Provide context to the issue - provide steps to reproduce the behavior, such as:

1. Go to 'Ambient policy enforcement istio/istio.io#15072 (comment)' to view the error. I will try to leave it up for a while but we do need to move the PR along at some point.

This was created by:

1. Submitted a PR to istio/istio.io repo
2. Receive and accept suggestions from a maintainer of the repository using the github PR web ui.

Expected behavior

I would expect suggestions from maintainers who have signed the CLA would not trigger a failure when their email becomes associated with the commit by addition of a "Co-authored-by" line in the commit message.

Screenshots

Environment

Please complete the following information:

• Environment:
  • ALL
  • DEV
  • STAGING
  • PROD
  • CI
• Browser:
  • Chrome/Brave
  • Firefox
  • Opera
  • Vivaldi
  • LibreWolf
  • SRware Iron
  • Dissenter
  • Slimjet
  • Midori
  • Edge
  • Lynx
  • Safari

[jarias-lfx]

Thank you for the report @ilrudie. I am adding additional PRs with the same issue:

• Add event listener support to the Desktop Agent API finos/FDC3#1207
• [podmanreceiver] add scraper's shutdown method open-telemetry/opentelemetry-collector-contrib#32981
• Add baggage span processor component open-telemetry/opentelemetry-java-contrib#1290
• Add credential consumption for protected buildpacks cloudfoundry/cnbapplifecycle#2

[svrnm]

Thanks for raising this @ilrudie, we are facing the same issue with at least 2 issues:

• Project Proposal: Feature Flag Semantic Conventions open-telemetry/community#2042
• Add "using instrumentation libraries" for cpp open-telemetry/opentelemetry.io#4388

[jarias-lfx]

Update:
We have identified the root cause of the issue and decided to rollback the co-author changes to unblock the community, since the fix will take longer engineering time. The PR should update by adding /easycla comment, which I added the in the reported PRs and confirmed EasyCLA passing.
Thank you for your patience.

[svrnm]

thanks @jarias-lfx

[ilrudie]

Thanks @jarias-lfx!