Skip to content

chore: 25580-blocks-mode-xts + test-ci merge#25661

Open
AlexKehayov wants to merge 31 commits into
mainfrom
25580-blocks-mode-xts-test-ci
Open

chore: 25580-blocks-mode-xts + test-ci merge#25661
AlexKehayov wants to merge 31 commits into
mainfrom
25580-blocks-mode-xts-test-ci

Conversation

@AlexKehayov
Copy link
Copy Markdown
Contributor

@AlexKehayov AlexKehayov commented May 29, 2026

Description:
Merges branch test-ci into 25580-blocks-mode-xts to test the fixes on the XTS BLOCKS job.

XTS here, here and here.

Fixes #

petreze and others added 22 commits May 27, 2026 22:40
Signed-off-by: Petar Tonev <petar.tonev@limechain.tech>
Signed-off-by: Petar Tonev <petar.tonev@limechain.tech>
Signed-off-by: Petar Tonev <petar.tonev@limechain.tech>
BLOCKS-mode embedded node can't sustain 10 concurrent contract
deployments after heavy preceding test classes (PrecompileSCTest).
Receipt polling times out with UNKNOWN on FileCreate/CryptoCreate.

Signed-off-by: aderevets <artem.derevets@limechain.tech>
In BLOCKS mode the legacy BlockInfo singleton is not maintained, so
ContractCallLocal queries read block number 0 from BlockRecordInfoImpl,
making blockhash(block.number - 1) underflow and revert. Mirror the
handle path (ParentTxnFactory) by deriving query block info from
BlockStreamInfo state. Read from the immutable snapshot, not the live
BlockStreamManager, whose fields are mutated off the query thread.

Signed-off-by: aderevets <artem.derevets@limechain.tech>
Signed-off-by: Petar Tonev <petar.tonev@limechain.tech>
Signed-off-by: Alex Kehayov <aleks.kehayov@limechain.tech>
Signed-off-by: Petar Tonev <petar.tonev@limechain.tech>
Signed-off-by: Alex Kehayov <aleks.kehayov@limechain.tech>
Signed-off-by: Petar Tonev <petar.tonev@limechain.tech>
Signed-off-by: aderevets <artem.derevets@limechain.tech>
Signed-off-by: Petar Tonev <petar.tonev@limechain.tech>
Signed-off-by: Petar Tonev <petar.tonev@limechain.tech>
Signed-off-by: Petar Tonev <petar.tonev@limechain.tech>
Signed-off-by: Petar Tonev <petar.tonev@limechain.tech>
Signed-off-by: Petar Tonev <petar.tonev@limechain.tech>
Signed-off-by: Petar Tonev <petar.tonev@limechain.tech>
Signed-off-by: Alex Kehayov <aleks.kehayov@limechain.tech>
@AlexKehayov AlexKehayov self-assigned this May 29, 2026
@trunk-io
Copy link
Copy Markdown

trunk-io Bot commented May 29, 2026

Merging to main in this repository is managed by Trunk.

  • To merge this pull request, check the box to the left or comment /trunk merge below.

After your PR is submitted to the merge queue, this comment will be automatically updated with its status. If the PR fails, failure details will also be posted here

@lfdt-bot
Copy link
Copy Markdown

lfdt-bot commented May 29, 2026

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@codecov
Copy link
Copy Markdown

codecov Bot commented May 29, 2026

Codecov Report

❌ Patch coverage is 43.47826% with 26 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...era/node/app/records/impl/BlockStreamInfoImpl.java 0.00% 14 Missing ⚠️
...edera/node/app/blocks/impl/BlockStreamBuilder.java 0.00% 5 Missing ⚠️
...era/node/app/workflows/query/QueryContextImpl.java 0.00% 4 Missing ⚠️
...hedera/node/app/workflows/handle/HandleOutput.java 33.33% 1 Missing and 1 partial ⚠️
.../node/app/state/recordcache/BlockRecordSource.java 92.30% 0 Missing and 1 partial ⚠️

Impacted file tree graph

@@             Coverage Diff              @@
##               main   #25661      +/-   ##
============================================
- Coverage     75.11%   74.84%   -0.27%     
- Complexity    11548    11551       +3     
============================================
  Files          2585     2586       +1     
  Lines        101419   101455      +36     
  Branches      11241    11244       +3     
============================================
- Hits          76179    75935     -244     
- Misses        21335    21617     +282     
+ Partials       3905     3903       -2     
Files with missing lines Coverage Δ Complexity Δ
...app/workflows/standalone/TransactionExecutors.java 85.82% <100.00%> (+0.34%) 0.00 <0.00> (ø)
...com/hedera/node/config/data/BlockStreamConfig.java 100.00% <ø> (ø) 0.00 <0.00> (ø)
.../node/app/state/recordcache/BlockRecordSource.java 98.11% <92.30%> (-1.89%) 0.00 <0.00> (ø)
...hedera/node/app/workflows/handle/HandleOutput.java 20.45% <33.33%> (+0.94%) 0.00 <0.00> (ø)
...era/node/app/workflows/query/QueryContextImpl.java 70.96% <0.00%> (-7.61%) 0.00 <0.00> (ø)
...edera/node/app/blocks/impl/BlockStreamBuilder.java 80.07% <0.00%> (+2.95%) 0.00 <0.00> (ø)
...era/node/app/records/impl/BlockStreamInfoImpl.java 0.00% <0.00%> (ø) 0.00 <0.00> (?)

... and 27 files with indirect coverage changes

Impacted file tree graph

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 29, 2026

Node: HAPI Test (Misc) Results

  157 files  ± 0      1 errors  156 suites   - 1   1h 24m 20s ⏱️ +6s
  174 tests  -  1    172 ✅  -  1  2 💤 ±0  0 ❌ ±0 
1 168 runs   - 38  1 166 ✅  - 38  2 💤 ±0  0 ❌ ±0 

For more details on these parsing errors, see this check.

Results for commit cd7c53f. ± Comparison against base commit f3f6ed6.

This pull request removes 1 test.
com.hedera.services.bdd.suites.contract.leaky.LeakyContractTestsSuite ‑ as written

♻️ This comment has been updated with latest results.

petreze added 2 commits May 29, 2026 15:25
Signed-off-by: Petar Tonev <petar.tonev@limechain.tech>
@AlexKehayov AlexKehayov marked this pull request as ready for review June 1, 2026 12:19
@AlexKehayov AlexKehayov requested review from a team and tinker-michaelj as code owners June 1, 2026 12:19
@AlexKehayov AlexKehayov requested a review from andrewb1269 June 1, 2026 12:19
@testlens-app
Copy link
Copy Markdown

testlens-app Bot commented Jun 1, 2026

✅ All tests passed ✅

⚠️ TestLens detected flakiness ⚠️

Test Summary

Check Project/Task Test Runs
Node: PR Checks / MATS / HAPI Tests / HAPI Tests (Misc) :test-clients:testSubprocessConcurrent ConcurrentSubprocessValidationTest > validateLogsAndStreams() > as written ❌ ✅

🏷️ Commit: cd7c53f
▶️ Tests: 6668 executed
⚪️ Checks: 96/96 completed


Learn more about TestLens at testlens.app.

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