17:42 <karolherbst> wait... marge-bot requires admin priviliges?

17:43 <imirkin> i think it requires same privs as you would for pushing to repo

17:43 <karolherbst> ehh, no, it's for something super stupid, getting email addresses of accounts

17:44 <imirkin> i meant wrt who can assign to marge-bot

17:45 <karolherbst> mhh, not sure if the gitlab API requires admin privs to query this information

17:45 <karolherbst> it seems marge-bot only needs it after force pushing if it wants to re-ack as well, as otherwise the acks would get lost

17:46 <karolherbst> but that's understandable, because to impersonate accounts you kind of need admin privs

17:46 <karolherbst> but for fetching the email address?

17:46 <imirkin> email addresses aren't public?

17:46 <karolherbst> apparently not via the API

17:46 <imirkin> (that wasn't meant as a question)

17:46 <karolherbst> well, they are

17:46 <imirkin> you can't find out the email address of an arbitrary account, afaik

17:46 <karolherbst> I think..

17:47 <__tim> you can probably query those that are advertised in the public profile

17:47 <karolherbst> there is a "Public email" option

17:47 <karolherbst> __tim: nope

17:47 <karolherbst> "All existing Reviewed-by: trailers on commits in the branch will be stripped. This feature requires marge to run with admin privileges due to a peculiarity of the GitLab API: only admin users can obtain email addresses of other users, even ones explicitly declared as public (strangely this limitation is particular to email, Skype handles etc. are visible to everyone)."

17:48 <karolherbst> maybe it was a bug and got fixed? dunno

17:48 <karolherbst> but I guess I'd kind of want to have a "account -> email" mapping anyway... even though that's super painful to maintain

17:49 <__tim> I think we're talking about different things anyway, I don't think you can assume that the e-mail address used for commit author or Reviewed-by matches the one in gitlab (public or not)

17:50 <karolherbst> I am talking about a bot adding those things