From: "Alexander V. Tikhonov" <avtikhon@tarantool.org> To: Alexander Turenko <alexander.turenko@tarantool.org> Cc: tarantool-patches@freelists.org, tarantool-patches@dev.tarantool.org Subject: [Tarantool-patches] [PATCH v1] build: added centos 8 Date: Tue, 29 Oct 2019 13:23:53 +0300 [thread overview] Message-ID: <a45bd670a45d59f5a2ffc758675507291cbd87bc.1572344561.git.avtikhon@tarantool.org> (raw) Added CentOS 8 image build, ready for Packpack use. Found that CentOS 8 completely moved to the use of python3 while Tarantool testing still uses python2, also found that CentOS 8 changed naming of the python packages to python2. To make these changes workable with Tarantool build added use of the python2 packages instead of python. Found that old packages like python-gevent and python-greenlet were completely removed from CentOS 8. To fix it the sources if these packages were downloaded from https://cbs.centos.org/ and rebuilt, to make them available the binaries were saved at the PackageCloud packpack backport repository. Met the issue with app-tap/pwd.test.lua which was allready fixed in local commit 7732daded14f86a314d44486e306e17bbc6ab293 (lua: treat ENOENT as success in getpwall/getgrall), but it wasn't merged into trunk - decided to block temporary the test till the correct fix will be created with the new issue created: https://github.com/tarantool/tarantool/issues/4592 Added CentOS 8 into regular testing at gitlab-ci. Closes #4543 --- Github: https://github.com/tarantool/tarantool/tree/avtikhon/gh-4543-centos8 Issue: https://github.com/tarantool/tarantool/issues/4543 .gitlab-ci.yml | 6 ++++++ rpm/tarantool.spec | 11 ++++++++++- test/app-tap/pwd.skipcond | 7 +++++++ 3 files changed, 23 insertions(+), 1 deletion(-) create mode 100644 test/app-tap/pwd.skipcond diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index 431730b67..655e68cdc 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -152,6 +152,12 @@ centos_7: OS: 'el' DIST: '7' +centos_8: + <<: *deploy_test_definition + variables: + OS: 'el' + DIST: '8' + fedora_28: <<: *deploy_test_definition variables: diff --git a/rpm/tarantool.spec b/rpm/tarantool.spec index 10daf1458..5930f79d7 100644 --- a/rpm/tarantool.spec +++ b/rpm/tarantool.spec @@ -15,7 +15,9 @@ BuildRequires: gcc-c++ >= 4.5 BuildRequires: coreutils BuildRequires: sed BuildRequires: readline-devel +%if (0%{?fedora} > 0 || 0%{?rhel} <= 7) BuildRequires: libyaml-devel +%endif BuildRequires: openssl-devel BuildRequires: libicu-devel #BuildRequires: msgpuck-devel @@ -64,12 +66,19 @@ BuildRequires: libunwind-devel %endif # For tests -%if (0%{?fedora} >= 22 || 0%{?rhel} >= 7) +%if (0%{?fedora} >= 22 || 0%{?rhel} == 7) BuildRequires: python >= 2.7 BuildRequires: python-six >= 1.9.0 BuildRequires: python-gevent >= 1.0 BuildRequires: python-yaml >= 3.0.9 %endif +%if (0%{?rhel} >= 8) +BuildRequires: python2 >= 2.7 +BuildRequires: python2-six >= 1.9.0 +BuildRequires: python2-greenlet +BuildRequires: python2-gevent >= 1.0 +BuildRequires: python2-yaml >= 3.0.9 +%endif Name: tarantool # ${major}.${major}.${minor}.${patch}, e.g. 1.6.8.175 diff --git a/test/app-tap/pwd.skipcond b/test/app-tap/pwd.skipcond new file mode 100644 index 000000000..b001fb82c --- /dev/null +++ b/test/app-tap/pwd.skipcond @@ -0,0 +1,7 @@ +import subprocess + +# Disabled at CentOS 8 build due to issue #4592. +if subprocess.check_output(["rpm --eval '%{centos_ver}'"], shell=True).strip() == '8': + self.skip = 1 + +# vim: set ft=python: -- 2.17.1
next reply other threads:[~2019-10-29 10:23 UTC|newest] Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-10-29 10:23 Alexander V. Tikhonov [this message] 2019-10-31 13:09 ` Alexander Turenko 2019-10-31 15:09 ` [Tarantool-patches] [tarantool-patches] " Alexander Tikhonov [not found] <702377579a50dbd8566542652d0b05baa62a0e2d.1570103318.git.avtikhon@tarantool.org> 2019-10-23 23:40 ` [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=a45bd670a45d59f5a2ffc758675507291cbd87bc.1572344561.git.avtikhon@tarantool.org \ --to=avtikhon@tarantool.org \ --cc=alexander.turenko@tarantool.org \ --cc=tarantool-patches@dev.tarantool.org \ --cc=tarantool-patches@freelists.org \ --subject='Re: [Tarantool-patches] [PATCH v1] build: added centos 8' \ /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