On Thu, 2020-11-26 at 17:30 +0000, Tristan Cacqueray wrote:
On Thu, Nov 26, 2020 at 15:59 Sean Mooney wrote:
On Thu, 2020-11-26 at 15:19 +0000, Jeremy Stanley wrote:
On 2020-11-26 15:10:09 +0000 (+0000), Tristan Cacqueray wrote: [...]
Here is a CR to add the test result table back in the new gerrit UI:
https://review.opendev.org/c/opendev/system-config/+/763891
This new implementation uses the javascript plugin extension API.
Thanks--that was quick work!
You're welcome :)
is that going to be moved to the zuul org in opendev as a part of the zuul projects deliverables.
i kind of think it makes sne to develop it there rather then on the rdo gerrit instance.
I'd be happy to move the project to the opendev gerrit.
looking at the impleation it does not look like this owrks teh ssame way by parsing the comment https://softwarefactory-project.io/cgit/software-factory/zuul-results-gerrit...
That is correct, this is using the Gerrit Js plugin api, which let you register a `showChange` callback, which is called with the change info https://gerrit-review.googlesource.com/Documentation/rest-api-changes.html#c...
Thus no need to hack through the dom to get the comments.
Note that the gerrit related code is in another project here: https://softwarefactory-project.io/cgit/software-factory/re-gerrit/
will this just show the results form zuul. it need to pars all the comments form the third party cis too. basicaly you need to parse all coment form a author that end isn CI and comments form zuul
It should do that already, though I haven't tested it with real data. I guess the next step would be to enable such plugin on review-dev. so the author regular expression is only looking for zuul https://softwarefactory-project.io/cgit/software-factory/re-gerrit/tree/src/...
so it lookes liek you would need to refactor that slightly to match comments form autors that end in ci too. each CI systems comemtn then need have there won tabel section with the job results for that ci to mimic the same behavior we had in the old ui.
Cheers, -Tristan