Tarantool development patches archive
 help / color / mirror / Atom feed
From: Alexander Turenko <alexander.turenko@tarantool.org>
To: "Alexander V. Tikhonov" <avtikhon@tarantool.org>
Cc: tarantool-patches@freelists.org, Kirill Yukhin <kyukhin@tarantool.org>
Subject: [tarantool-patches] Re: [PATCH v1] gitlab-ci: add static build
Date: Wed, 21 Aug 2019 01:02:00 +0300	[thread overview]
Message-ID: <20190820220200.7swuiaxdgrk2gffl@tkn_work_nb> (raw)
In-Reply-To: <adeca83a27bdced81c7e7fefe6f3dd9e1ad545bd.1566293538.git.avtikhon@tarantool.org>

LGTM.

Alexander, did you test it on 1.10? If not, I suggest to do so before
the patch will land.

Kirill, please, push to all release branches down to 1.10. You can test
it before push using `RUN_TESTS=ON make -f .gitlab.mk static_build` (the
docker daemon should be running).

WBR, Alexander Turenko.

On Tue, Aug 20, 2019 at 12:32:32PM +0300, Alexander V. Tikhonov wrote:
> Added static build using Dockerfile on Centos 7 for release
> commit criteria only. Added the cleanup for cmake generating
> CMakeCache.txt files and CMakeFiles directories to avoid of
> cmake localy created setup failing inside the docker after
> the whole tarantool path was copied into it. Added testing
> into the static build, running only when RUN_TESTS environment
> variable set to non empty value, used in gitlab-ci job to run
> the testing after the build.
> 
> Closes #3668
> ---
> 
> Github: https://github.com/tarantool/tarantool/tree/avtikhon/gh-3668-static-build-full-ci
> Issue: https://github.com/tarantool/tarantool/issues/3668
> 
>  .gitlab-ci.yml         | 13 +++++++++++++
>  .gitlab.mk             |  7 +++++++
>  Dockerfile.staticbuild | 12 ++++++++++++
>  3 files changed, 32 insertions(+)
> 
> diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
> index a7b2b3804..f160c276b 100644
> --- a/.gitlab-ci.yml
> +++ b/.gitlab-ci.yml
> @@ -326,3 +326,16 @@ debian_10:
>      DIST: 'buster'
>    script:
>      - ${GITLAB_MAKE} package
> +
> +static_build:
> +  only:
> +    refs:
> +      - master
> +      - /^.*-full-ci$/
> +  stage: test
> +  tags:
> +    - deploy
> +  variables:
> +    RUN_TESTS: 'ON'
> +  script:
> +    - ${GITLAB_MAKE} static_build
> diff --git a/.gitlab.mk b/.gitlab.mk
> index 7199adf94..cb66df4c2 100644
> --- a/.gitlab.mk
> +++ b/.gitlab.mk
> @@ -105,3 +105,10 @@ vms_shutdown:
>  package: git_submodule_update
>  	git clone https://github.com/packpack/packpack.git packpack
>  	PACKPACK_EXTRA_DOCKER_RUN_PARAMS='--network=host' ./packpack/packpack
> +
> +# ############
> +# Static build
> +# ############
> +
> +static_build:
> +	docker build --network=host -f Dockerfile.staticbuild .
> diff --git a/Dockerfile.staticbuild b/Dockerfile.staticbuild
> index 6a784990b..679cce560 100644
> --- a/Dockerfile.staticbuild
> +++ b/Dockerfile.staticbuild
> @@ -78,6 +78,12 @@ RUN set -x && \
>      git submodule init && \
>      git submodule update
>  
> +WORKDIR /tarantool
> +
> +RUN set -x && \
> +    find . -name 'CMakeFiles' -type d -exec rm -rf {} + && \
> +    find . -name 'CMakeCache.txt' -type f -delete
> +
>  RUN pip install -r /tarantool/test-run/requirements.txt
>  
>  RUN set -x \
> @@ -91,4 +97,10 @@ RUN set -x \
>  
>  RUN cd /tarantool && make install
>  
> +ARG RUN_TESTS
> +RUN [ -n ${RUN_TESTS} ] && \
> +    set -x && \
> +    cd test && \
> +    /usr/bin/python test-run.py --force
> +
>  ENTRYPOINT /bin/bash
> -- 
> 2.17.1
> 

  reply	other threads:[~2019-08-20 22:02 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-08-20  9:32 [tarantool-patches] " Alexander V. Tikhonov
2019-08-20 22:02 ` Alexander Turenko [this message]
2019-08-21 11:07   ` [tarantool-patches] " Alexander Turenko
  -- strict thread matches above, loose matches on Subject: below --
2019-08-16  9:55 [tarantool-patches] " Alexander V. Tikhonov
2019-08-19 12:00 ` [tarantool-patches] " Alexander Turenko
2019-08-09  4:32 [tarantool-patches] " Alexander V. Tikhonov
2019-08-16  2:30 ` [tarantool-patches] " Alexander Turenko

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20190820220200.7swuiaxdgrk2gffl@tkn_work_nb \
    --to=alexander.turenko@tarantool.org \
    --cc=avtikhon@tarantool.org \
    --cc=kyukhin@tarantool.org \
    --cc=tarantool-patches@freelists.org \
    --subject='[tarantool-patches] Re: [PATCH v1] gitlab-ci: add static build' \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox