Skip to content

[pull] develop from smartcontractkit:develop#544

Merged
pull[bot] merged 6 commits intoMarcelRaschke:developfrom
smartcontractkit:develop
Apr 13, 2026
Merged

[pull] develop from smartcontractkit:develop#544
pull[bot] merged 6 commits intoMarcelRaschke:developfrom
smartcontractkit:develop

Conversation

@pull
Copy link
Copy Markdown

@pull pull bot commented Apr 13, 2026

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

Tofel and others added 6 commits April 13, 2026 17:01
…sts (#21968)

* fix(test): prevent goroutine leak in PausableTicker and CronTicker tests

PausableTicker.Destroy() stops the ticker but doesn't close the
channel. Goroutines running for-range on Ticks() block forever,
accumulating across -count=N iterations.

Add done channels so goroutines exit when the test completes. Also
reduce CronTicker verification sleep from 1s to 200ms.

Fixes: CORE-2401

* revert CronTicker verification sleep back to 1s
…ision (#21977)

* fix(ci): pass unique artifact-name per matrix job to prevent 409 collision

Multiple matrix jobs with trunk-auto-quarantine enabled (go_core_tests,
go_core_tests_integration, go_core_ccip_deployment_tests) all upload an
artifact named 'individual_test_logs' in the same workflow run, causing a
deterministic 409 conflict that fails CI even when tests pass.

Pass artifact-name: ${{ matrix.type.cmd }}_test_logs so each job gets a
unique artifact name. Depends on smartcontractkit/.github#1512.

Fixes: CORE-2418

* fix(ci): pass unique artifact-name to branch-out-upload in CRE workflows

Prevents 409 conflict when multiple matrix jobs fail simultaneously and
attempt to upload artifacts with the same hardcoded name.

Fixes: smartcontractkit/.github/actions/branch-out-upload (artifact-name input)
…eport in nodetestutils (#21980)

* fix(deployment): replace hashicorp freeport with smartcontractkit/freeport in nodetestutils

The move commit fd31642 re-introduced github.com/hashicorp/consul/sdk/freeport
into node.go, undoing the repo-wide swap done in 8315b51. The hashicorp library
has a TOCTOU race — it briefly binds to check port availability then releases,
allowing other CI processes to steal the port before the test binds. The
smartcontractkit fork avoids this by holding a TCP listener on an entire port block
for the lifetime of the process.

Fixes CORE-2375.

* fix(deployment): fix goimports and go mod tidy after freeport swap

Run goimports to fix import ordering in nodetestutils/node.go and
run go mod tidy in deployment, integration-tests, and
integration-tests/load to remove the now-unused
github.com/hashicorp/consul/sdk dependency.
…n OCR3 (#21959)

Global Prometheus counters/histograms registered via promauto accumulate
across test iterations when running with -count=N. Tests that assert
exact absolute values fail on the 2nd+ iteration.

Fix by capturing initial metric values before test operations and
asserting on the delta instead.

Fixes: CORE-2374
@pull pull bot locked and limited conversation to collaborators Apr 13, 2026
@pull pull bot added the ⤵️ pull label Apr 13, 2026
@pull pull bot merged commit cdc60a7 into MarcelRaschke:develop Apr 13, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants