-
Notifications
You must be signed in to change notification settings - Fork 17.8k
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
x/build: “Gerrit User XXX” displayed instead of actual name in Gerrit messages #28663
Comments
This also affects dev.golang.org. |
Change https://golang.org/cl/148557 mentions this issue: |
This change deletes the OwnerName method on GerritCL, since Gerrit's format for its NoteDB backing store changed to display "Gerrit User NNN" instead of the proper display name in the meta commit. Update golang/go#28663 Change-Id: Ifa64ca2b2694b17e888451582b9c63f3f37280a9 Reviewed-on: https://go-review.googlesource.com/c/148557 Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Change https://golang.org/cl/148560 mentions this issue: |
This should take effect on Nov 9 00:00 UTC in that all new Gerrit comments after that time will have the correct display name. dev.golang.org/reviews is fixed now. |
This is now fixed for all new comments. Sorry for the inconvenience, @rsc. |
Thanks for the quick fix. |
#28659 (comment) says "Message from Gerrit User 5056". This needs to be fixed, and soon.
It's easy to fix - the usual Gerrit REST API ChangeInfo uses the right names. I'm perplexed as to why the maintner feed says these things but at least the bot could re-fetch the actual Gerrit comments directly from Gerrit (only when it needs to post one to GitHub, so fairly low rate) to find out real names.
#28662 means that from Gerrit I can't tell who is writing the CL and this bug means that from GitHub I can't tell who is reviewing the CL. This is not really an acceptable review process.
The text was updated successfully, but these errors were encountered: