Skip to content

Conversation

maflcko
Copy link
Member

@maflcko maflcko commented May 3, 2023

This allows to cache the image and thus speed up the CI task

@DrahtBot
Copy link
Contributor

DrahtBot commented May 3, 2023

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 fanquake, hebasto

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

@DrahtBot DrahtBot added the Tests label May 3, 2023
Copy link
Member

@fanquake fanquake left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice ACK fa6e2bf

Copy link
Member

@hebasto hebasto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK fa6e2bf

@fanquake fanquake merged commit 1d7f1ad into bitcoin:master May 3, 2023
@maflcko maflcko deleted the 2305-ci-arm-docker- branch May 4, 2023 08:35
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request May 4, 2023
fa6e2bf ci: Use arm_container.dockerfile (MarcoFalke)

Pull request description:

  This allows to cache the image and thus speed up the CI task

ACKs for top commit:
  fanquake:
    nice ACK fa6e2bf
  hebasto:
    ACK fa6e2bf

Tree-SHA512: 3b7f734799bd8a3ca9166e85d2cd8f68c134c32a496757eafb118250c0b158d0d76ead0c461e307d6440a622fe51bc599aa13e67766ce232b750ed0a66d4b450
fanquake added a commit to bitcoin-core/gui that referenced this pull request May 18, 2023
fa3761d ci: Reduce tsan CPU and memory for faster scheduling (MarcoFalke)
aaaa07b ci: Use credits for ARM task (MarcoFalke)

Pull request description:

  After bitcoin/bitcoin#27562 the task should finish in less than 10 minutes, so also using credits for it will be cheap and improve dev experience.

ACKs for top commit:
  fanquake:
    ACK fa3761d

Tree-SHA512: 98ba76eaa63bcbab674076bb9877b3a20d1b90e6d51e2fd7b97ae245414e984d734006b15ea60228a1f4f9bc72fbe2a1a8910fd05e7c52dd2a8b223bfaa25b50
@stickies-v stickies-v mentioned this pull request Jun 15, 2023
fanquake added a commit that referenced this pull request Jun 16, 2023
6233049 ci: Switch to `amd64` container in "ARM" task (Hennadii Stepanov)
d845a3e test: add coverage for `/deploymentinfo` passing a blockhash (brunoerg)
72ead86 rest: bugfix, fix crash error when calling `/deploymentinfo` (brunoerg)
6f7a0ae ci: Use podman stop over podman kill (MarcoFalke)
de56daa ci: Use podman for persistent workers (MarcoFalke)
71f626e ci: Prune dangling images on RESTART_CI_DOCKER_BEFORE_RUN (MarcoFalke)

Pull request description:

  Currently backports:
  * #27777
  * #27844
  * #27853
  * #27886

  Effectively also backports: #27562.

ACKs for top commit:
  stickies-v:
    ACK 6233049

Tree-SHA512: d0f3d5c4cd0cf9792f3270078b49ad6661e28e7a883f91e5981e8cfe95d01c6e415762ee2e3a5fea17d198299989a8b5412961c2cc788cef975b2ee45d84550c
@bitcoin bitcoin locked and limited conversation to collaborators May 3, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants