cluster-api-provider-bringyourownhost icon indicating copy to clipboard operation
cluster-api-provider-bringyourownhost copied to clipboard

Investigate why /lgtm label is not working

Open shamsher31 opened this issue 3 years ago • 2 comments

What steps did you take and what happened: Add /lgtm label to any PR. You will see the following error.

Error: error handling issue comment: Error: commenter is not a org member or collaborator. Cannot add lgtm label

See https://github.com/vmware-tanzu/cluster-api-provider-bringyourownhost/actions/runs/1490117401

What did you expect to happen: If /lgtm label is present along with /approve, the PR should merge automatically.

Anything else you would like to add: The problem might be because of org label or reviewer group is not specified anywhere in merge actions.

shamsher31 avatar Nov 22 '21 12:11 shamsher31

@shamsher31 lgtm chat command uses API checkMembership in the current code base of https://github.com/jpmcb/prow-github-actions but this API is changed to checkMembershipForUser.

I raised a PR https://github.com/jpmcb/prow-github-actions/pull/37 to modify this API as per the new version. Let us wait to get this PR merged.

Akasurde avatar Dec 27 '21 06:12 Akasurde

Thanks for surfacing this!

The prow bot is actually using an older version of @actions/github and the checkMembershipForUser API wasn't added until a later version. The currently implementation should still work.

I do see that you are a member of this org where you had commented the lgtm label command. But i don't think the error gets surfaced out of the checkMembership job when there's problems. I think that's the first thing to do since GitHub org authentication can be really really tricky.

For reference:

  • [ ] https://github.com/jpmcb/prow-github-actions/issues/39

jpmcb avatar Dec 29 '21 18:12 jpmcb