From: "Alexander Tikhonov" <avtikhon@tarantool.org> To: alexander.turenko@tarantool.org Cc: tarantool-patches@freelists.org Subject: [tarantool-patches] Re: [tarantool-patches] Re: [PATCH v4 0/2] Implement Gitlab-ci testing process Date: Mon, 24 Jun 2019 11:13:03 +0300 [thread overview] Message-ID: <1561363983.14108646@f339.i.mail.ru> (raw) In-Reply-To: <20190621142000.ozbnnu6l4m4pguvj@tkn_work_nb> [-- Attachment #1: Type: text/plain, Size: 4543 bytes --] >Пятница, 21 июня 2019, 17:20 +03:00 от Alexander Turenko <alexander.turenko@tarantool.org>: > >## Changes > >(Updated your branch as we agreed voicely.) > >Fixed spelling and changed wording in the commit messages. > >Splitted goals in .gitlab.mk into sections just like you are made it for >.travis.mk and commented these sections. > >Rewritten 'docker_bootstrap' goal to write docker files in a multiline >way. Reworked tags for these images: use :latest in testing, but always >push both :`md5sum of .travis.mk` and :latest tags. Ok. > >## Comments > >I would remove 'master' tag at all, because of several reasons: > >* It appears over several files and it will be hard to keep it > consistent. >* We rarely change something here and I don't remember case when we > remove some dependency. >* It is not obvious that 'master' tag is intended to be used for all > branches that are based on master, 2.1 for branches based on 2.1), > etc. Ok. > >I thinking whether we should use >tarantool/tarantool/testing/debian-<...> for images naming? I would be >more clear why the image is needed and also will allow use to use this >namespace (tarantool/tarantool) for images of other purposes. I don't do >that in my changes. Fixed, added /testing/ sub-directory into the image name repository. > >Re 'deploy' jobs. I would run the packpack script with environment >variables rather then call its internal makefile inside a docker image >run by gitlab runner. This is how it is performed for Travis CI. Fixed. > >I doubt about a hardcoded repository within a virtual machine image. I >suggest to leave only system things in the image (install and setup >packages and so on), but don't do such special tarantool-related things. Fixed. > >I very against of black boxes like these VM images. Nobody knows what it >is. Nobody can reproduce a problem. So, please implement make goals that >will create vm images identical to ones that is used in testing. There are all needed targets at the .gitlab.mk file, just use the same environment of the job from .gitlab-ci.yml that you need and run from tarantool sources, like: VMS_NAME=osx_14 make -f .gitlab.mk vms_start VMS_USER=tarantool VMS_PORT=2222 make -f .gitlab.mk vms_test_osx NOTE: To run it locally the VBOX should be installed with needed images. > >Also I would look into vagrant. As I see, there are OS X and FreeBSD >images. We need to check their licenses, BTW. Don't sure about pushing >images, but at least local using should be possible. As discussed voicely currently the Virtual Box will be used and the Vagrant images will be integrated during the issue: https://github.com/tarantool/tarantool/issues/4308 > >Maybe I forgotten something that we discussed voicely. > >WBR, Alexander Turenko. > >On Wed, Jun 19, 2019 at 05:56:29PM +0300, Alexander V. Tikhonov wrote: >> Implement Gitlab-ci testing process >> >> Implemented Gitlba-ci testing process additionaly to travis-ci >> that is currently uses. New testing process was added to make >> able to control the high load of the testing processes to avoid >> of flaky fails on timouts, disks layouts and memory swapping. >> Created 2 stages for testing and deploying packages. >> Stage test consist of testing jobs for all branches at: >> Debian 9 (Stretch): release/debug gcc >> Debian 10 (Buster): release clang8 + lto >> OSX 14 (Mojave): release >> FreeBSD 12: release gcc >> and for release branch has additionally: >> OSX 13 (Sierra) release clang >> OSX 14 (Mojave) release clang + lto >> Deploy of packages is the same as in Travis-CI. >> Additional manual work is needed if the image's depends was >> changed at .travis.mk file - in this way need to do: >> make -f .gitlab.mk docker_bootstrap >> >> Fixes #4156 >> >> >> Github: https://github.com/tarantool/tarantool/tree/avtikhon/gh-4156-gitlab-ci-testing >> Issue: https://github.com/tarantool/tarantool/issues/4156 >> >> Alexander V. Tikhonov (2): >> Temporary disabled on_shutdown.test.lua test >> Implement Gitlab-ci testing process >> >> .gitlab-ci.yml | 297 +++++++++++++++++++++++++++++++++++++++++++++ >> .gitlab.mk | 57 +++++++++ >> .travis.mk | 107 +++++++++++----- >> .travis.yml | 3 + >> test/box/suite.ini | 2 +- >> 5 files changed, 438 insertions(+), 28 deletions(-) >> create mode 100644 .gitlab-ci.yml >> create mode 100644 .gitlab.mk >> >> -- >> 2.17.1 >> > -- Alexander Tikhonov [-- Attachment #2: Type: text/html, Size: 7457 bytes --]
prev parent reply other threads:[~2019-06-24 8:13 UTC|newest] Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-06-19 14:56 [tarantool-patches] " Alexander V. Tikhonov 2019-06-19 14:56 ` [tarantool-patches] [PATCH v4] " Alexander V. Tikhonov 2019-06-19 14:56 ` [tarantool-patches] [PATCH v4 1/2] Temporary disabled on_shutdown.test.lua test Alexander V. Tikhonov 2019-06-19 14:56 ` [tarantool-patches] [PATCH v4 2/2] Implement Gitlab-ci testing process Alexander V. Tikhonov 2019-06-21 14:20 ` [tarantool-patches] Re: [PATCH v4 0/2] " Alexander Turenko 2019-06-24 8:13 ` Alexander Tikhonov [this message]
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=1561363983.14108646@f339.i.mail.ru \ --to=avtikhon@tarantool.org \ --cc=alexander.turenko@tarantool.org \ --cc=tarantool-patches@freelists.org \ --subject='[tarantool-patches] Re: [tarantool-patches] Re: [PATCH v4 0/2] Implement Gitlab-ci testing process' \ /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