On Thu, Dec 3, 2020 at 2:29 PM Sean Mooney <smooney@redhat.com> wrote:
On Thu, 2020-12-03 at 13:16 +0100, Radosław Piliszek wrote:
On Thu, Dec 3, 2020 at 11:03 AM Radosław Piliszek <radoslaw.piliszek@gmail.com> wrote:
... I obviously forgot all the little changes I made ...
Just remembered one so writing down for posterity. One of the enhancements is that it supports multiple tenants, i.e. it works for non-openstack/ namespaces, e.g. x/, opendev/ or zuul/ too.
the main issue i still see with this script is it only looks a zuul comments so im worried that if too many peopel install it they will not look at the thirdparty ci result becaue they are uesed to the old ui where they are there.
Yes, this is the original limitation. This is analogous to hrw's comment about AArch64. Open to contributions. :-) I am pretty sure it would be best to treat this as a PoC and build a proper solution on Gerrit's side. As for dynamic results - I bet it is even trickier to handle the TP solutions.
it works on firfox with code injector too by the way.
++ -yoctozepto