[Spice-devel] [PATCH libcacard 44/45] Set up the CI in gitlab including coverage reports

Marc-André Lureau marcandre.lureau at gmail.com
Thu Aug 2 13:04:58 UTC 2018


On Wed, Aug 1, 2018 at 10:00 AM, Jakub Jelen <jjelen at redhat.com> wrote:
> On Tue, 2018-07-31 at 18:14 +0200, Marc-André Lureau wrote:
>> Hi
>>
>> On Tue, Jul 31, 2018 at 4:50 PM, Jakub Jelen <jjelen at redhat.com>
>> wrote:
>> > Signed-off-by: Jakub Jelen <jjelen at redhat.com>
>> > Reviewed-by: Robert Relyea <rrelyea at redhat.com>
>> > ---
>> >  .gitlab-ci.yml | 53
>> > ++++++++++++++++++++++++++++++++++++++++++++++++++
>> >  1 file changed, 53 insertions(+)
>> >  create mode 100644 .gitlab-ci.yml
>> >
>>
>> This will pass on freedesktop instance with the following changes:
>>
>> diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
>> index 7eb7a22..5910594 100644
>> --- a/.gitlab-ci.yml
>> +++ b/.gitlab-ci.yml
>> @@ -1,3 +1,9 @@
>> +image: fedora:latest
>> +
>> +before_script:
>> +  - dnf update -y
>> +  - dnf install -y gcc automake libtool make pkgconfig
>> autoconf-archive nss-devel glib2-devel opensc gnutls-utils nss-tools
>> openssl softhsm lcov
>> +
>>  .base: &base_job
>>    artifacts:
>>      expire_in: '1 week'
>> @@ -24,8 +30,6 @@ Check:
>>      - bash run_tests.sh
>>      - make code-coverage-capture
>>      - lcov --summary libcacard-*-coverage.info
>> -  tags:
>> -    - linux
>>
>>  Memcheck:
>>    <<: *base_job
>> @@ -35,8 +39,6 @@ Memcheck:
>>      - ./configure
>>      - sed -e "s/make check/make memcheck-local/" -i run_tests.sh
>>      - bash run_tests.sh
>> -  tags:
>> -    - linux
>>    allow_failure: true
>>
>>
>>
>> (ideally, we would use a saved docker instead of reinstalling)
>>
>> Do you mind if I squash the changes and push this patch before the
>> rest of the series?
>
> Sure. Feel free to squash this first. The reason I did this later was
> that the testsuite was very limited before I started working on that.
>

Ok, I'll push it now with a few minor changes, they will require minor
update for you later in the series.

thanks

>> thanks
>>
>> > diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
>> > new file mode 100644
>> > index 0000000..7eb7a22
>> > --- /dev/null
>> > +++ b/.gitlab-ci.yml
>> > @@ -0,0 +1,53 @@
>> > +.base: &base_job
>> > +  artifacts:
>> > +    expire_in: '1 week'
>> > +    paths:
>> > +      - test-suite.log
>> > +      - libcacard-*-coverage
>> > +  only:
>> > +    - master
>> > +
>> > +Build:
>> > +  <<: *base_job
>> > +  stage: build
>> > +  script:
>> > +    - autoreconf -i
>> > +    - ./configure
>> > +    - make
>> > +
>> > +Check:
>> > +  <<: *base_job
>> > +  stage: test
>> > +  script:
>> > +    - autoreconf -i
>> > +    - ./configure --enable-code-coverage
>> > +    - bash run_tests.sh
>> > +    - make code-coverage-capture
>> > +    - lcov --summary libcacard-*-coverage.info
>> > +  tags:
>> > +    - linux
>> > +
>> > +Memcheck:
>> > +  <<: *base_job
>> > +  stage: test
>> > +  script:
>> > +    - autoreconf -i
>> > +    - ./configure
>> > +    - sed -e "s/make check/make memcheck-local/" -i run_tests.sh
>> > +    - bash run_tests.sh
>> > +  tags:
>> > +    - linux
>> > +  allow_failure: true
>> > +
>> > +pages:
>> > +  stage: deploy
>> > +  dependencies:
>> > +    - Check
>> > +  script:
>> > +    - mv *-coverage/ public/
>> > +  artifacts:
>> > +    paths:
>> > +      - public
>> > +    expire_in: 30 days
>> > +  only:
>> > +    - master
>> > --
>> > 2.17.1
>> >
>> > _______________________________________________
>> > Spice-devel mailing list
>> > Spice-devel at lists.freedesktop.org
>> > https://lists.freedesktop.org/mailman/listinfo/spice-devel
>>
>>
>>
> --
> Jakub Jelen
> Software Engineer
> Security Technologies
> Red Hat, Inc.
>



-- 
Marc-André Lureau


More information about the Spice-devel mailing list