3.2 looks really nice, thanks for the smooth update!
There's a "Monitoring" tab in the top bar of the new Gerrit UI, maybe we should hide it by default (if that's possible). It's confusingly named (I initially assumed it allowed me to subscribed to some repo's activity) and the internal JavaMelody metrics are only relevant for people who keep Gerrit running, but not Gerrit users.
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | QChris | T186406 Polygerrit Outgoing reviews, Incoming reviews and Recently closed headings should be more different from commits | |||
| Resolved | None | T220801 PolyGerrit 2.15 UI does not support custom dashboards (supported from in 2.16+) | |||
| Resolved | QChris | T206453 Big empty space under patch description | |||
| Restricted Task | |||||
| Resolved | None | T210515 context links in diff don't use pointer cursor | |||
| Resolved | Paladox | T223278 "Server error: line 1:33 no viable alternative at character '['" when visiting particular patch | |||
| Restricted Task | |||||
| Resolved | hashar | T103990 Gerrit upload-pack send ALL references causing massive network I/O on common operations | |||
| Resolved | None | T195280 New Gerrit UI does not contain copyable Change-Id or parent commit id | |||
| Declined | Feature | None | T201246 Make a PolyGerrit plugin to welcome new users | ||
| Resolved | QChris | T254158 Gerrit 3.2 upgrade | |||
| Invalid | None | T256601 Hide the "Monitoring" tab? |
If it was visible by default, it would not be ideal. Agreed.
However, the Monitoring tab is hidden by default (see below).
It's only visible to users who can administrate the server (or who explicitly got access to JavaMelody).
And that includes you, as you are in the ops LDAP group, which are explicitly enabled access to JavaMelody. Hence, you get to see it.
Here are the relevant code links in case someone wants to verify:
https://gerrit.googlesource.com/plugins/javamelody/+/06e4d1ae08e470e1a4f7c394b852053e5e831c3b/src/main/java/com/googlesource/gerrit/plugins/javamelody/MonitoringTopMenu.java#29
https://gerrit.googlesource.com/plugins/javamelody/+/06e4d1ae08e470e1a4f7c394b852053e5e831c3b/src/main/java/com/googlesource/gerrit/plugins/javamelody/CapabilityChecker.java#50