Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: update local node images to Java 21 #10834

Merged
merged 2 commits into from Jan 12, 2024

Conversation

isavov
Copy link
Contributor

@isavov isavov commented Jan 9, 2024

Description:

This PR updates the Dockerfile for local node in order to support Java 21

  • Update Dockerfile

Related issue(s):

Fixes #10833

Notes for reviewer:

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

@isavov isavov linked an issue Jan 9, 2024 that may be closed by this pull request
@isavov isavov force-pushed the 10833-docker-images-for-local-node-dont-work branch from 35ff637 to 3e0cf50 Compare January 9, 2024 13:00
@isavov isavov self-assigned this Jan 9, 2024
@isavov isavov added Bug An error that causes the feature to behave differently than what was expected based on design. Release Issues tied to release management functions. labels Jan 9, 2024
Copy link

github-actions bot commented Jan 9, 2024

Node: HAPI Test (Restart) Results

1 tests  ±0   1 ✔️ ±0   4m 49s ⏱️ -47s
1 suites ±0   0 💤 ±0 
1 files   ±0   0 ±0 

Results for commit bb97591. ± Comparison against base commit 688f3d5.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Jan 9, 2024

Node: HAPI Test (Node Death Reconnect) Results

1 tests  ±0   1 ✔️ ±0   5m 57s ⏱️ +31s
1 suites ±0   0 💤 ±0 
1 files   ±0   0 ±0 

Results for commit bb97591. ± Comparison against base commit 688f3d5.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Jan 9, 2024

Node: HAPI Test (Token) Results

189 tests  ±0   189 ✔️ +1   20m 4s ⏱️ +12s
  13 suites ±0       0 💤 ±0 
  13 files   ±0       0  - 1 

Results for commit bb97591. ± Comparison against base commit 688f3d5.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Jan 9, 2024

Node: HAPI Test (Crypto) Results

210 tests  ±0   210 ✔️ +1   21m 41s ⏱️ - 1m 30s
  22 suites ±0       0 💤 ±0 
  22 files   ±0       0  - 1 

Results for commit bb97591. ± Comparison against base commit 688f3d5.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Jan 9, 2024

Node: E2E Test Results

    1 files  ±    0      1 suites  ±0   23m 11s ⏱️ + 23m 11s
311 tests +310  311 ✔️ +311  0 💤 ±0  0  - 1 
333 runs  +332  333 ✔️ +333  0 💤 ±0  0  - 1 

Results for commit bb97591. ± Comparison against base commit 688f3d5.

This pull request removes 1 and adds 311 tests. Note that renamed tests count towards both.
EndToEndTests ‑ initializationError
EndToEndTests ‑ ADDRESS_BOOK_CONTROLCanUpdateADDRESS_BOOK
EndToEndTests ‑ ADDRESS_BOOK_CONTROLCanUpdateNODE_DETAILS
EndToEndTests ‑ AccountsGetPayerRecordsIfSoConfigured
EndToEndTests ‑ Acct57CanMakeSmallChanges
EndToEndTests ‑ Acct57CantMakeLargeChanges
EndToEndTests ‑ AddingSignaturesToExecutedTxFails
EndToEndTests ‑ AddingSignaturesToExecutedTxFailsWithLongTermEnabled
EndToEndTests ‑ AddingSignaturesToNonExistingTxFails
EndToEndTests ‑ AddingSignaturesToNonExistingTxFailsWithLongTermEnabled
EndToEndTests ‑ AddressAliasIdFuzzing
…

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Jan 9, 2024

Node: Unit Test Results

    2 292 files  ±0      2 292 suites  ±0   1h 0m 40s ⏱️ + 9m 50s
118 792 tests ±0  118 753 ✔️ +1  39 💤 ±0  0  - 1 
127 313 runs  ±0  127 274 ✔️ +1  39 💤 ±0  0  - 1 

Results for commit bb97591. ± Comparison against base commit 688f3d5.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Jan 9, 2024

Node: HAPI Test (Misc) Results

423 tests  ±0   409 ✔️ ±0   35m 20s ⏱️ + 1m 24s
  73 suites ±0     14 💤 ±0 
  73 files   ±0       0 ±0 

Results for commit bb97591. ± Comparison against base commit 688f3d5.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Jan 9, 2024

Node: HAPI Test (Time Consuming) Results

20 tests  ±0   20 ✔️ ±0   53m 50s ⏱️ +3s
  2 suites ±0     0 💤 ±0 
  2 files   ±0     0 ±0 

Results for commit bb97591. ± Comparison against base commit 688f3d5.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Jan 9, 2024

Node: HAPI Test (Smart Contract) Results

445 tests  ±0   444 ✔️ ±0   59m 10s ⏱️ -12s
  52 suites ±0       0 💤 ±0 
  52 files   ±0       1 ±0 

For more details on these failures, see this check.

Results for commit bb97591. ± Comparison against base commit 688f3d5.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Jan 9, 2024

Node: Integration Test Results

    5 files  ±0      5 suites  ±0   1h 6m 2s ⏱️ +13s
318 tests ±0  318 ✔️ +1  0 💤 ±0  0  - 1 
330 runs  ±0  330 ✔️ +1  0 💤 ±0  0  - 1 

Results for commit bb97591. ± Comparison against base commit 688f3d5.

♻️ This comment has been updated with latest results.

@isavov isavov marked this pull request as ready for review January 10, 2024 10:52
@isavov isavov requested review from a team as code owners January 10, 2024 10:52
@isavov isavov force-pushed the 10833-docker-images-for-local-node-dont-work branch from 3e0cf50 to e0ed1ef Compare January 10, 2024 10:53
Copy link

codecov bot commented Jan 10, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (688f3d5) 63.26% compared to head (bb97591) 63.26%.

Additional details and impacted files
@@            Coverage Diff             @@
##             develop   #10834   +/-   ##
==========================================
  Coverage      63.26%   63.26%           
+ Complexity     31301    31299    -2     
==========================================
  Files           3390     3390           
  Lines         137447   137447           
  Branches       14405    14405           
==========================================
+ Hits           86951    86955    +4     
+ Misses         46906    46903    -3     
+ Partials        3590     3589    -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@povolev15 povolev15 left a comment

Choose a reason for hiding this comment

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

LGTM

@isavov isavov force-pushed the 10833-docker-images-for-local-node-dont-work branch 3 times, most recently from ad423ff to c9b8b9e Compare January 11, 2024 13:43
@isavov isavov force-pushed the 10833-docker-images-for-local-node-dont-work branch 2 times, most recently from b2579d0 to c063cc9 Compare January 11, 2024 15:37
Signed-off-by: Iliya Savov <isavov@users.noreply.github.com>
Signed-off-by: Iliya Savov <isavov@users.noreply.github.com>
@isavov isavov force-pushed the 10833-docker-images-for-local-node-dont-work branch from c063cc9 to bb97591 Compare January 12, 2024 05:49
@isavov isavov merged commit f179127 into develop Jan 12, 2024
41 of 43 checks passed
@isavov isavov deleted the 10833-docker-images-for-local-node-dont-work branch January 12, 2024 08:42
nathanklick pushed a commit that referenced this pull request Jan 16, 2024
#10968)

Signed-off-by: Iliya Savov <isavov@users.noreply.github.com>
Co-authored-by: Iliya Savov <isavov@users.noreply.github.com>
timo0 pushed a commit that referenced this pull request Jan 22, 2024
Signed-off-by: Iliya Savov <isavov@users.noreply.github.com>
Signed-off-by: Timo Brandstätter <timo@swirldslabs.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug An error that causes the feature to behave differently than what was expected based on design. Release Issues tied to release management functions.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Docker Images for Local Node don't work
2 participants