Skip to content

fix(ci): force build integ test images on pushes to develop#21726

Merged
erikburt merged 4 commits intodevelopfrom
fix/push-events-force-integ-test-image-build
Apr 2, 2026
Merged

fix(ci): force build integ test images on pushes to develop#21726
erikburt merged 4 commits intodevelopfrom
fix/push-events-force-integ-test-image-build

Conversation

@erikburt
Copy link
Copy Markdown
Collaborator

@erikburt erikburt commented Mar 26, 2026

Ensure we build integ tests images on pushes to develop, to populate docker layer cache.

Pushes to develop (which is where we populate the cache) skip the image build step because the image with the same tag was already built in the merge queue: https://github.com/smartcontractkit/chainlink/actions/runs/23861050974/job/69567608626

  • But merge queue caches are not available to PR branches

@erikburt erikburt self-assigned this Mar 26, 2026
@erikburt erikburt added the run-e2e-tests Use on your PR to run E2E tests before the merge queue label Mar 26, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 26, 2026

✅ No conflicts with other open PRs targeting develop

@trunk-io
Copy link
Copy Markdown

trunk-io Bot commented Mar 26, 2026

Static BadgeStatic BadgeStatic BadgeStatic Badge

View Full Report ↗︎Docs

@erikburt erikburt requested a review from kalverra March 26, 2026 21:43
@erikburt erikburt marked this pull request as ready for review March 26, 2026 21:44
@erikburt erikburt requested a review from a team as a code owner March 26, 2026 21:44
Copilot AI review requested due to automatic review settings March 26, 2026 21:44
@erikburt erikburt requested review from a team as code owners March 26, 2026 21:44
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Risk Rating: MEDIUM — modifies CI control flow for building/pushing integration-test Docker images on push events to develop, which can impact CI cost and reliability.

This PR updates the Integration Tests workflow so that Chainlink integration test images are force-built on pushes to develop to help populate Docker layer cache.

Changes:

  • Rename the matrix boolean from should-build to any-should-run and route build gating through a dedicated “explain” step output (build-image).
  • Force image builds on push events to the develop branch (regardless of whether tests will run / whether the image already exists).
  • Update downstream steps to use steps.explain.outputs.build-image for conditional checkout/build.

Comment thread .github/workflows/integration-tests.yml Outdated
Comment thread .github/workflows/integration-tests.yml
Comment thread .github/workflows/integration-tests.yml Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@erikburt erikburt enabled auto-merge April 1, 2026 18:56
@erikburt erikburt added this pull request to the merge queue Apr 2, 2026
Merged via the queue into develop with commit 916ce4b Apr 2, 2026
255 of 257 checks passed
@erikburt erikburt deleted the fix/push-events-force-integ-test-image-build branch April 2, 2026 13:17
prashantkumar1982 pushed a commit that referenced this pull request Apr 2, 2026
* fix(ci): force build integ test images on pushes to develop

* Update .github/workflows/integration-tests.yml

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* fix: merge

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run-e2e-tests Use on your PR to run E2E tests before the merge queue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants