ci: Bump CentOS 8 image #24212

pull MarcoFalke wants to merge 3 commits into bitcoin:master from MarcoFalke:2201-ciBC changing 6 files +72 −72
  1. MarcoFalke commented at 9:34 am on January 31, 2022: member

    CentOS 8 was just shut down:

    0$ pob centos:8 
    1[root_e606059da4d9 /]# dnf update
    2Failed to set locale, defaulting to C.UTF-8
    3CentOS Linux 8 - AppStream
    4Error: Failed to download metadata for repo 'appstream': Cannot prepare internal mirrorlist: No URLs in mirrorlist
    

    Fix that by bumping the image.

  2. hebasto commented at 9:56 am on January 31, 2022: member

    Concept ACK, considering:

    • CentOS Linux 8 EOL: 2021-12-31
    • CentOS Stream 8 EOL: 2024-05-31
  3. DrahtBot added the label Tests on Jan 31, 2022
  4. fanquake commented at 10:38 am on January 31, 2022: member
    0  zeromq-4.3.4-2.el8.x86_64                                                     
    1Removed:
    2  coreutils-single-8.30-12.el8.x86_64                                           
    3
    4Complete!
    5bash: free: command not found
    6
    7Exit status: 127
    
  5. MarcoFalke commented at 10:44 am on January 31, 2022: member
    Trying with procps-ng …
  6. MarcoFalke force-pushed on Jan 31, 2022
  7. MarcoFalke force-pushed on Jan 31, 2022
  8. MarcoFalke commented at 12:18 pm on January 31, 2022: member
    0Building libevent...
    1make[1]: Entering directory '/tmp/cirrus-ci-build/depends/work/build/i686-pc-linux-gnu/libevent/2.1.12-stable-dbc4e9c2bf0'
    2make[1]: -c: Command not found
    3make[1]: *** [Makefile:1425: config.h] Error 127
    4make[1]: Leaving directory '/tmp/cirrus-ci-build/depends/work/build/i686-pc-linux-gnu/libevent/2.1.12-stable-dbc4e9c2bf0'
    5make: *** [funcs.mk:282: /tmp/cirrus-ci-build/depends/work/build/i686-pc-linux-gnu/libevent/2.1.12-stable-dbc4e9c2bf0/./.stamp_built] Error 2
    6make: Leaving directory '/tmp/cirrus-ci-build/depends'
    7Exit status: 2
    
  9. MarcoFalke added the label Up for grabs on Jan 31, 2022
  10. hebasto commented at 9:39 pm on January 31, 2022: member

    @MarcoFalke

    libevent build can be fixed with the following patch:

     0--- a/ci/test/05_before_script.sh
     1+++ b/ci/test/05_before_script.sh
     2@@ -47,7 +47,7 @@ if [[ $HOST = *-mingw32 ]]; then
     3   DOCKER_EXEC update-alternatives --set "${HOST}-g++" \$\(which "${HOST}-g++-posix"\)
     4 fi
     5 if [ -z "$NO_DEPENDS" ]; then
     6-  if [[ $DOCKER_NAME_TAG == centos* ]]; then
     7+  if [[ $DOCKER_NAME_TAG == *centos* ]]; then
     8     # CentOS has problems building the depends if the config shell is not explicitly set
     9     # (i.e. for libevent a Makefile with an empty SHELL variable is generated, leading to
    10     #  an error as the first command is executed)
    
  11. MarcoFalke force-pushed on Feb 1, 2022
  12. MarcoFalke removed the label Up for grabs on Feb 1, 2022
  13. ci: Bump CentOS 8 image fa5457e64a
  14. scripted-diff: Rename DOCKER_EXEC to CI_EXEC
    -BEGIN VERIFY SCRIPT-
     sed -i "s/DOCKER_EXEC/CI_EXEC/g" $(git grep -l DOCKER_EXEC)
    -END VERIFY SCRIPT-
    fa33236e77
  15. ci: Use dash when building depends in centos build fafc55a489
  16. MarcoFalke force-pushed on Feb 1, 2022
  17. Sjors commented at 9:42 am on February 1, 2022: member
    utACK fafc55a with the caveat that I’ve never used CentOS.
  18. MarcoFalke commented at 10:00 am on February 1, 2022: member

    Just to give some more context:

    • Only the first commit is needed as a bug fix
    • The second commit is an unrelated rename. docker->ci, since it is possible to run the ci without docker
    • The third commit makes sense because we use the centos build to test with dash
  19. MarcoFalke merged this on Feb 1, 2022
  20. MarcoFalke closed this on Feb 1, 2022

  21. MarcoFalke deleted the branch on Feb 1, 2022
  22. sidhujag referenced this in commit 7f9f55a049 on Feb 1, 2022
  23. fanquake referenced this in commit c9d0a52e8c on Feb 4, 2022
  24. fanquake referenced this in commit 95f726f942 on Feb 4, 2022
  25. hebasto referenced this in commit ed2769135a on Feb 6, 2022
  26. hebasto referenced this in commit d25c9c7b67 on Feb 6, 2022
  27. fanquake referenced this in commit 201ed8096f on Feb 9, 2022
  28. fanquake referenced this in commit 40cc39a819 on Feb 9, 2022
  29. fanquake referenced this in commit c362722418 on Feb 9, 2022
  30. fanquake referenced this in commit 611a1d6115 on Feb 9, 2022
  31. fanquake referenced this in commit b1f38581ac on Feb 14, 2022
  32. fanquake referenced this in commit 37e8ea5753 on Feb 14, 2022
  33. fanquake referenced this in commit bba5856963 on Feb 14, 2022
  34. fanquake referenced this in commit 0075274c77 on Feb 14, 2022
  35. fanquake referenced this in commit b5d12edb08 on Feb 14, 2022
  36. fanquake referenced this in commit d34691ce0c on Feb 14, 2022
  37. fanquake commented at 8:08 am on February 15, 2022: member
    Being backported in #24256.
  38. fanquake referenced this in commit eff9709723 on Feb 15, 2022
  39. DrahtBot locked this on Feb 15, 2023

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-12-18 15:12 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me