[all] CI test result table in the new gerrit review UI
balazs.gibizer at est.tech
Thu Nov 26 14:14:31 UTC 2020
On Thu, Nov 26, 2020 at 05:57, Sorin Sbarnea <ssbarnea at redhat.com>
> Thanks! i works!
> tMaybe now you can propose a CR to update the old script that no
> longer worked?
Sorry, I have no intention to do a full update on the original script.
As far as I remember the old gerrit CI table did a lot more than what I
hacked together and I have not enough time and knowledge to cover that.
Also I'm not sure how the above script relates the solution in the old
gerrit that was injected server side instead of injected from the
> There is one example where it does not play very nice due to lots of
> changes sharing the same topic:
What I see here is that there is a lot of related changes and the table
is added at the end of the long list. But overall it works for me on
this patch too. What is your specific problem?
> Probably you can just replace the script, but I would bother to
> increase the versioning because once we merge it, existing users may
> be able to detect an update.
> PS. Add me as reviewer.
> On 26 Nov 2020 at 13:00:01, Lucio Seki <lucioseki at gmail.com> wrote:
>> Thanks a lot Balázs!
>> I minified  the script and put it in a bookmarklet . It works
>> like a charm  :-)
>>  https://imgur.com/a/iy5jPIn
>> On Thu, 26 Nov 2020 at 09:42, Balázs Gibizer
>> <balazs.gibizer at est.tech> wrote:
>>> I understand that adapting the old CI test result table to the new
>>> gerrit review UI is not a simple task. As a temporary measure I
>>> together a tampermonkey user script that does a limited version
>>> the CI table injection from the browser side. I tested it with
>>> chrome and firefox. Let me know if you have problems with it and
>>> try to help. Also feel free to adapt it to your own needs.
More information about the openstack-discuss