Fix centos 7 ci #3614
Merged
Fix centos 7 ci #3614
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
AkihiroSuda
added
area/ci
backport/1.1-todo
kolyshkin
force-pushed
the
fix-centos-7-ci
branch
3 times, most recently
from
1936333 to
82ae1fb
Compare
|
OK, the issue seems that in CentOS 7 image used by Cirrus CI, EPEL is no longer enabled, despite the docs at https://cloud.google.com/compute/docs/images/os-details (under "Image configuration") saying
I am pretty sure it used to be enabled, but apparently Google Cloud has changed something. The other part of the story is |
kolyshkin
force-pushed
the
fix-centos-7-ci
branch
from
82ae1fb to
9043435
Compare
The situation when /sys/fs/cgroup/unified is not present normal and should not result in anything on stderr. Suppress it. Fixes: cc15b88 Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
We use awk in other 9 or so places, and here it's gawk. Since this is on Linux, most probably awk is gawk. So s/gawk/awk/. Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
kolyshkin
force-pushed
the
fix-centos-7-ci
branch
from
9043435 to
f73a02a
Compare
It used to be enabled by default, but not as of last few weeks. While at it, add rpm -q command to make sure all required RPMS were in fact installed (at least CentOS 7 yum exits with 0 when some packages requested are not available). Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
kolyshkin
force-pushed
the
fix-centos-7-ci
branch
from
f73a02a to
491713e
Compare
Merged
kolyshkin
added
backport/1.1-done
AkihiroSuda
approved these changes
Sep 28, 2022
mrunalp
approved these changes
Sep 28, 2022
This was referenced Oct 3, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Turns out, CentOS 7 image used in Cirrus CI had EPEL repo configured, and now it's not. This, together with a "feature" of older yum NOT returning an error when some of the packages requested are not found, resulted in failures in tests that use
jqandfuse-sshfs.The fix is to add/enable EPEL repo.
1.1 backport: #3618