fix(ci): properly check image build status

[erikburt]

Changes

• Switches from templated literals to env vars
• More robust checks for chainlink image build

[github-actions]

✅ No conflicts with other open PRs targeting develop

[trunk-io]

     

Failed Test	Failure Summary	Logs
Test_CRE_V2_EVM_CallContract_Invalid_Addr_To_Read_Regression/[v2]_EVM.CallContract_-_invalid_address_to_read_fails_with_long_add...	The test failed because an unexpected error occurred during EVM contract call, specifically a reentrancy out-of-gas error, and a workflow could not...	Logs ↗︎
Test_CRE_V2_EVM_CallContract_Invalid_Addr_To_Read_Regression	The test failed without providing specific error details, indicating an unspecified failure in the test case.	Logs ↗︎

View Full Report ↗︎ ⋅ Docs

[Copilot]

Pull request overview

Updates the integration-tests GitHub Actions workflow to reduce brittle expression interpolation in bash and to more accurately fail when the Chainlink image build is cancelled/failed.

Changes:

• Pass needs.* values into steps via env: and reference them as shell variables instead of embedding expressions in bash.
• Expand the Chainlink image build gate to fail on cancelled (and still fail on failure), while warning on skipped.

Comments suppressed due to low confidence (1)

.github/workflows/integration-tests.yml:630

• node_migration_tests_failed is computed using echo $results | jq ... with an unquoted variable expansion. This allows word-splitting/globbing and can corrupt the JSON (or change it) before it reaches jq, producing incorrect results or parse errors. Quote the variable (or use printf '%s' "$results") when piping into jq.

          results="$TEST_RESULTS"
          echo "Core e2e test results:"
          echo "$results" | jq .

          node_migration_tests_failed=$(echo $results | jq '[.[] | select(.id == "integration-tests/migration/upgrade_version_test.go:*" ) | select(.result != "success")] | length > 0')