Skip to content

[DX-3583] Modify legacy nightly system tests triggers#21876

Open
Tofel wants to merge 11 commits intodevelopfrom
dx-3583-nightly-public-images
Open

[DX-3583] Modify legacy nightly system tests triggers#21876
Tofel wants to merge 11 commits intodevelopfrom
dx-3583-nightly-public-images

Conversation

@Tofel
Copy link
Copy Markdown
Contributor

@Tofel Tofel commented Apr 7, 2026

run legacy nightly system tests with:

  1. public images,
  2. private images,
  3. when nightly image has been built (via docker-build)

Tests:

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 7, 2026

✅ No conflicts with other open PRs targeting develop

@Tofel Tofel force-pushed the dx-3583-nightly-public-images branch from b190577 to 7fe73fe Compare April 7, 2026 09:30
@Tofel Tofel force-pushed the dx-3583-nightly-public-images branch from 7fe73fe to 77f5853 Compare April 7, 2026 09:36
@Tofel Tofel marked this pull request as ready for review April 7, 2026 13:11
@Tofel Tofel requested review from a team as code owners April 7, 2026 13:11
Copilot AI review requested due to automatic review settings April 7, 2026 13:11
mchain0
mchain0 previously approved these changes Apr 7, 2026
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 — changes affect nightly CI orchestration and image selection for legacy system tests.

Updates legacy nightly system test triggering so runs can be driven via workflow inputs and can also be invoked after the nightly Docker image build completes.

Changes:

  • Converted legacy-system-tests-nightly.yml to be driven via workflow_dispatch / workflow_call inputs for Chainlink image selection (full image vs repo+tag) and checkout ref.
  • Added Chainlink image resolution + ECR (private/public) login steps to the legacy system tests workflow.
  • Updated docker-build.yml to call the legacy nightly system tests workflow on schedule / workflow_dispatch after docker-core finishes, passing the built image tag.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
.github/workflows/legacy-system-tests-nightly.yml Adds inputs and resolves Chainlink image/tag at runtime; updates AWS auth + ECR login to support private/public registries.
.github/workflows/docker-build.yml Invokes legacy system tests after nightly core image build on scheduled/dispatch runs; aligns chainlink_version passed to downstream workflows with the checked-out ref/SHA.

Scrupulous human review recommended (high-impact blocks):

  • .github/workflows/legacy-system-tests-nightly.yml: the Resolve Chainlink image env wiring in both summary and test-nightly jobs (mutual exclusivity handling and correctness of the selected image reference).
  • .github/workflows/docker-build.yml: the new call-legacy-system-tests job wiring (trigger conditions + ensuring the intended image/tag is passed through from docker-core outputs).

@Tofel Tofel force-pushed the dx-3583-nightly-public-images branch from c33ba84 to 9b84360 Compare April 7, 2026 13:53
@Tofel Tofel marked this pull request as draft April 9, 2026 15:03
@Tofel Tofel marked this pull request as ready for review April 9, 2026 15:03
@Tofel Tofel requested a review from Copilot April 9, 2026 15:04
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

Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.

skudasov
skudasov previously approved these changes Apr 9, 2026
@Tofel Tofel force-pushed the dx-3583-nightly-public-images branch from 868c725 to 2f8d091 Compare April 10, 2026 10:45
@Tofel Tofel force-pushed the dx-3583-nightly-public-images branch from ac5752f to 735c986 Compare April 10, 2026 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants