ci: Switch to amd64 container in “ARM” task #27886

pull hebasto wants to merge 1 commits into bitcoin:master from hebasto:230614-ci-arm changing 1 files +1 −5
  1. hebasto commented at 12:32 pm on June 14, 2023: member

    The arm_container does not support 32-bit mode anymore.

    Fixes #27879.

    Also, the arm_container could be used for testing aarch64 binaries, which are the part of our releases. Leaving that for another PR.

  2. ci: Switch to `amd64` container in "ARM" task
    Tee `arm_container` does not support 32-bit mode anymore.
    See: https://github.com/bitcoin/bitcoin/issues/27879
    016fe6d828
  3. DrahtBot commented at 12:32 pm on June 14, 2023: contributor

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK MarcoFalke

    If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

  4. DrahtBot added the label Tests on Jun 14, 2023
  5. maflcko commented at 12:41 pm on June 14, 2023: member

    review ACK 016fe6d8280768917081894dfca233c2f06e78d9 if CI is green

    A smaller diff would be to just unset the QEMU_USER_CMD and thus run qemu-arm on aarch64 and keep the image as-is, but that seems to run into unrelated timeouts, as pointed out in #27879 (comment)

    Longer term we can also think about whether we want to test aarch64 or arm32 in this CI task, but this can be done later.

  6. maflcko added this to the milestone 25.1 on Jun 14, 2023
  7. maflcko added the label Needs backport (25.x) on Jun 14, 2023
  8. maflcko commented at 1:14 pm on June 14, 2023: member
    Looks like this was, as expected, wrapped in qemu and the task passed: https://cirrus-ci.com/task/5196388229709824?logs=ci#L3590
  9. fanquake merged this on Jun 14, 2023
  10. fanquake closed this on Jun 14, 2023

  11. fanquake referenced this in commit e6141298e3 on Jun 14, 2023
  12. fanquake removed the label Needs backport (25.x) on Jun 14, 2023
  13. fanquake commented at 1:39 pm on June 14, 2023: member
    Backported to 25.x in #27887.
  14. hebasto deleted the branch on Jun 14, 2023
  15. fanquake referenced this in commit 6233049709 on Jun 15, 2023
  16. sidhujag referenced this in commit fa9c9c7b9e on Jun 15, 2023
  17. fanquake referenced this in commit 642b5dd1b4 on Jun 16, 2023
  18. fanquake referenced this in commit 43596499b2 on Oct 2, 2023
  19. fanquake referenced this in commit 1416d09cba on Oct 6, 2023
  20. bitcoin locked this on Jun 13, 2024


hebasto DrahtBot maflcko fanquake

Labels
Tests

Milestone
25.1


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-11-21 18:12 UTC

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