<br><br>On Wednesday, 10 April 2019, Gert Wollny <<a href="mailto:gert.wollny@collabora.com">gert.wollny@collabora.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello Gurchetan, <br>
<br>
Am Mittwoch, den 10.04.2019, 10:44 -0700 schrieb Gurchetan Singh:<br>
> <br>
> <br>
> <br>
> How does one enable the CI?  It's not one of the options when editing<br>
> a merge requests.<br>
I'll add this to the wiki. The CI file must be set for the fork you're<br>
using: <br>
<br>
  Settings/CI <br>
    General pipelines<br>
<br>
   Timeout: 2h <br>
   Custom CI config path: ci/.gitlab-ci.yml  <br>
<br>
Currently the CI is only building and running "make check" for normal<br>
commits, and only for merge requests the full test series is run. </blockquote><div><br></div><div>Actually I add a small doc few days ago.</div><div> <a href="https://gitlab.freedesktop.org/virgl/virglrenderer/blob/master/docs/CI-notes.txt">https://gitlab.freedesktop.org/virgl/virglrenderer/blob/master/docs/CI-notes.txt</a></div><div><br></div><div>We need to setup the timeout to 2h because the CTS run gl and gles in a same job. I will split it in 2 later.</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
> Note with Mesa, Gitlab shows the "Merge when pipeline passes" option<br>
> for me, while virglrenderer only shows "Merge" option.<br>
This is when you didn't set up the CI for your fork. For security<br>
reasons gitlab doesn't allow to run the CI of a merge requests in the<br>
main repo space.  <br>
<br>
>  <br>
> > * After doing a review please tag the MR with one of <br>
> >    - is-reviewed,<br>
> >    - waiting for changes, or  <br>
> >    - needs more reviews<br>
> >   with that I can see the state of a MR without digging through<br>
> > the <br>
> >   comments or through all the emails. <br>
> <br>
> Developer access atleast is needed to add tags, I believe.  I created<br>
> a test account and issued a new merge request it:<br>
> <br>
> <a href="https://gitlab.freedesktop.org/virgl/virglrenderer/merge_requests/219" target="_blank">https://gitlab.freedesktop.<wbr>org/virgl/virglrenderer/merge_<wbr>requests/219</a><br>
> <br>
> Unfortunately, I couldn't edit the tags.<br>
I think it's reporter status that is needed. I'll added Lepton Wu and<br>
Pohsien Wang. <br>
<br>
I've also put this on a wiki page: <br>
<a href="https://gitlab.freedesktop.org/virgl/virglrenderer/wikis/Merge-requests" target="_blank">https://gitlab.freedesktop.<wbr>org/virgl/virglrenderer/wikis/<wbr>Merge-requests</a><br>
<br>
Best, <br>
Gert <br>
<br>
______________________________<wbr>_________________<br>
virglrenderer-devel mailing list<br>
<a href="mailto:virglrenderer-devel@lists.freedesktop.org">virglrenderer-devel@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/virglrenderer-devel" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/<wbr>virglrenderer-devel</a></blockquote>