[ovirt-devel] Re: Add "ci system-test" command
On Thu, Jun 24, 2021 at 2:11 PM Yedidyah Bar David <didi(a)redhat.com> wrote:
On Thu, Jun 24, 2021 at 1:51 PM Marcin Sobczyk <msobczyk(a)redhat.com> wrote:
>
>
>
> On 6/23/21 5:44 PM, Nir Soffer wrote:
> > Similar to "ci build", "ci test", "ci merge" add
a new command that
> > triggers OST run.
> >
> > Running OST is tied now in vdsm (and engine?) to Code-Review: +2.
> > This causes trouble and does not allow non-maintainers to use the convenient
OST
> > infrastructure.
> >
> > Expected flow:
> >
> > 1. User add a comment with "ci system-test"
> "ci system-test" is sooo long, I vote for "ci ost".
+1.
Perhaps we can add an optional suite name? E.g. 'ci ost ansible-suite-master'
+1