ci: [lint] Use Cirrus dockerfile cache #31948

pull maflcko wants to merge 1 commits into bitcoin:master from maflcko:2502-ci-l changing 3 files +17 −9
  1. maflcko commented at 12:12 pm on February 25, 2025: member

    The lint task is problematic, because:

    • It doesn’t check modifications to ci/lint_imagefile
    • It calls a separate script that installs packages on every run (taking time)
    • It uses *_cache instructions to cache some installed parts, but not all

    Fix all issues by using ci/lint_imagefile (https://cirrus-ci.org/guide/docker-builder-vm/#dockerfile-as-a-ci-environment)

  2. DrahtBot commented at 12:12 pm on February 25, 2025: contributor

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

    Code Coverage & Benchmarks

    For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/31948.

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK willcl-ark

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

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #31787 (ci: run in worktrees by willcl-ark)

    If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

  3. DrahtBot added the label Tests on Feb 25, 2025
  4. maflcko force-pushed on Feb 25, 2025
  5. maflcko force-pushed on Feb 25, 2025
  6. DrahtBot added the label CI failed on Feb 25, 2025
  7. DrahtBot removed the label CI failed on Feb 25, 2025
  8. maflcko requested review from willcl-ark on Feb 28, 2025
  9. ci: Use Cirrus dockerfile cache fa3b442715
  10. maflcko force-pushed on Mar 13, 2025
  11. maflcko commented at 7:53 am on March 17, 2025: member
  12. willcl-ark approved
  13. willcl-ark commented at 11:41 am on March 17, 2025: member

    ACK fa3b4427158d48f7d899582580f8f6a7b1bc981d

    Sensible change to speedup the lint task startup.

    The change looks correct, and to me it looks like it also works, judging by the log lines:

    011:51:41 PM Successfully pulled image "gcr.io/cirrus-ci-community/bitcoin/bitcoin/ci/lint_imagefile:714fc33ccc0872eef4aa515b2c226494648ec0f52aa6d3afbaf779c6d1a976b3" in 17.583s (18.557s including waiting). Image size: 699506090 bytes.
    

    …and from the “dependency run” step, which is building an image for this imagefile: https://cirrus-ci.com/task/5351025877450752

    I do notice the linked doc page claims that:

    Under the hood, for every Dockerfile that is needed to be built, Cirrus CI will create a Docker Builder task as a dependency. You will see such build_docker_image_HASH tasks in the UI.

    And I wasn’t able to see any of those lines anywhere in the UI; but perhaps that’s changed since it was documented, or they don’t appear if the image is already cached.

  14. fanquake merged this on Mar 18, 2025
  15. fanquake closed this on Mar 18, 2025

  16. maflcko deleted the branch on Mar 18, 2025

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: 2025-03-29 06:12 UTC

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