Wrong path to JavaMelody monitoring in PolyGerrit UI

Discussion:

j***@ql.ventures

2018-11-30 06:32:51 UTC

Hello,

/MONITORING/JavaMelody in PolyGerrit UI points
to https://gerrit.my.company/admin/monitoring, while
the old ui is https://gerrit.my.company/monitoring.

Is this a bug or configurable?

Thanks,

--
--
To unsubscribe, email repo-discuss+***@googlegroups.com
More info at http://groups.google.com/group/repo-discuss?hl=en

---
You received this message because you are subscribed to the Google Groups "Repo and Gerrit Discussion" group.
To unsubscribe from this group and stop receiving emails from it, send an email to repo-discuss+***@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

David Ostrovsky

2018-11-30 10:10:48 UTC

Permalink

Post by j***@ql.ventures
Hello,
/MONITORING/JavaMelody in PolyGerrit UI points to
https://gerrit.my.company/admin/monitoring, while
the old ui is https://gerrit.my.company/monitoring.
Is this a bug or configurable?

Sounds like a bug to me. Extension point is providing the URL without
"admin" part: [1].

https://gerrit.googlesource.com/plugins/javamelody/+/master/src/main/java/com/googlesource/gerrit/plugins/javamelody/MonitoringTopMenu.java#32

David Ostrovsky

2018-12-03 21:39:11 UTC

Permalink

Post by David Ostrovsky

Sounds like a bug to me. Extension point is providing the URL without
"admin" part: [1].

Should be fixed with: [2].

[2] https://gerrit-review.googlesource.com/c/plugins/javamelody/+/205993