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

[type:feat] upgrade dockerfile java runtime version 8 to 17 #5374

Merged
merged 124 commits into from
May 6, 2024

Conversation

tomsun28
Copy link
Contributor

  • upgrade dockerfile java runtime version 8 to 17, use image openjdk:17-jdk-alpine

Do not merge now, please consider merge after v2.6.1 version realease.

Make sure that:

  • You have read the contribution guidelines.
  • You submit test cases (unit or integration tests) that back your changes.
  • Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

@codecov-commenter
Copy link

codecov-commenter commented Dec 21, 2023

Codecov Report

Attention: Patch coverage is 27.27273% with 8 lines in your changes are missing coverage. Please review.

Project coverage is 61.50%. Comparing base (1748a0e) to head (0b2cab8).

❗ Current head 0b2cab8 differs from pull request most recent head 151d5a4. Consider uploading reports for the commit 151d5a4 to get more accurate results

Files Patch % Lines
...java/org/apache/shenyu/common/utils/GsonUtils.java 27.27% 8 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##             master    #5374   +/-   ##
=========================================
  Coverage     61.49%   61.50%           
- Complexity     8818     8820    +2     
=========================================
  Files          1261     1261           
  Lines         39273    39284   +11     
  Branches       3725     3726    +1     
=========================================
+ Hits          24151    24161   +10     
- Misses        12952    12954    +2     
+ Partials       2170     2169    -1     

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

tomsun28 and others added 23 commits December 24, 2023 21:28
* Tests

* Updated lib.rs and added a README.md

I updated the `lib.rs` and updated the WASM file and also added a README.md

* Remove TestWasmPluginDiscoveryHandler#handlerDiscoveryUpstreamData

* Revert "Remove TestWasmPluginDiscoveryHandler#handlerDiscoveryUpstreamData"

This reverts commit c85d3db.

* Update AbstractWasmDiscoveryHandlerTest.java

---------

Co-authored-by: loongs-zhang <zhangzicheng@apache.org>
* fix AbstractWasmPluginDataHandlerTest

* [ISSUE apache#5427] add AbstractWasmMetaDataHandlerTest
…he#5463)

Co-authored-by: likeguo <33576070+li-keguo@users.noreply.github.com>
…apache#5462)

* add offline in grpc and websocket

* trigger ci

* trigger ci

---------

Co-authored-by: xiaoyu <xiaoyu@apache.org>
Co-authored-by: loongs-zhang <zhangzicheng@apache.org>
@Aias00
Copy link
Contributor

Aias00 commented Apr 26, 2024

@tomsun28 this job is done

@tomsun28 tomsun28 requested a review from moremind April 29, 2024 14:19
@tomsun28
Copy link
Contributor Author

this job is done

@Aias00 👍👍 great!,@moremind please help to review and merge.

@moremind
Copy link
Member

LGTM

@tomsun28 tomsun28 requested a review from moremind May 2, 2024 04:10
yu199195
yu199195 previously approved these changes May 6, 2024
@moremind moremind added this to the 2.7.0 milestone May 6, 2024
@moremind moremind merged commit 40e91fe into apache:master May 6, 2024
52 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.