Skip to content
This repository has been archived by the owner on Apr 17, 2019. It is now read-only.

CLA munger in each repository #1834

Closed
foxish opened this issue Oct 4, 2016 · 7 comments
Closed

CLA munger in each repository #1834

foxish opened this issue Oct 4, 2016 · 7 comments

Comments

@foxish
Copy link
Contributor

foxish commented Oct 4, 2016

Currently, the model we operate under is one instance of the mungegithub bot per repository (test-infra, contrib, kubernetes).
However, with the CNCF CLA, message posting and applying CLA labels are tied to the bot. We would need to have one instance per repository running, which doesn't seem scalable (or efficient) as we have so many different repositories.

The alternatives include:

  • Separating the CLA functionality into a separate single instance of mungegithub which which can easily run on a list of repos.
  • Adding the label functionality at the CNCF end.

I think the first option is reasonable. As I understand, the latter is not preferable since we want to have control of how/who we notify.
cc @kubernetes/contributor-experience @apelisse @grodrigues3

@apelisse
Copy link
Contributor

apelisse commented Oct 8, 2016

As discussed, I'll take care of that (if time allows)

@foxish
Copy link
Contributor Author

foxish commented Oct 24, 2016

@apelisse If you put out a WIP PR, I can help with this as well, in order to speed it up.

@apelisse
Copy link
Contributor

As discussed in kubernetes/test-infra#930, this will eventually be implemented in prow (please join the discussion if you have an opinion :-).

@foxish
Copy link
Contributor Author

foxish commented Nov 2, 2016

@apelisse Is prow ready for moving the CLA functionality into it?

@apelisse
Copy link
Contributor

apelisse commented Nov 2, 2016

@foxish No, we'll just shove it in there anyway :-/

@spxtr
Copy link
Contributor

spxtr commented Nov 2, 2016

I'll try to have a nice framework in place in the next couple days, then we can put it there without much hassle.

@foxish
Copy link
Contributor Author

foxish commented Nov 2, 2016

Thanks @spxtr

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants