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

⬆️ [Dependencies] Upgrade Artemis from 2.31.2 to 2.38.0 #4087

Open
wants to merge 9 commits into
base: develop
Choose a base branch
from

Conversation

riccardomodanese
Copy link
Contributor

Brief description of the PR.
see description

Related Issue
none

Description of the solution adopted
just upgrade dependency

Screenshots
none

Any side note on the changes made
none

@riccardomodanese
Copy link
Contributor Author

/request-license-review

@riccardomodanese riccardomodanese added the CQ pending This PR needs a CQ to be approved from Eclipse before merging. label Jul 23, 2024
@Coduz Coduz changed the title :feat: upgrade artemis to 2.35 ⬆️ [Dependencies] Upgrade Artemis from 2.31.2 to 2.35.0 Oct 8, 2024
@riccardomodanese riccardomodanese force-pushed the feat-upgrade_artemis branch 2 times, most recently from 05d11cf to aa1f2b9 Compare November 15, 2024 08:20
@riccardomodanese
Copy link
Contributor Author

/request-license-review

@riccardomodanese
Copy link
Contributor Author

/request-license-review

@riccardomodanese riccardomodanese changed the title ⬆️ [Dependencies] Upgrade Artemis from 2.31.2 to 2.35.0 ⬆️ [Dependencies] Upgrade Artemis from 2.31.2 to 2.38.0 Nov 15, 2024
@riccardomodanese
Copy link
Contributor Author

upgraded versions

  • Netty to 4.1.114.Final
  • Logback 1.5.12
  • Slf4J to 2.0.16

@riccardomodanese riccardomodanese force-pushed the feat-upgrade_artemis branch 5 times, most recently from 13aaa9b to c10489c Compare November 18, 2024 13:47
Copy link

codecov bot commented Nov 18, 2024

Codecov Report

Attention: Patch coverage is 0% with 51 lines in your changes missing coverage. Please review.

Project coverage is 16.72%. Comparing base (9970735) to head (02403b1).
Report is 4 commits behind head on develop.

Files with missing lines Patch % Lines
...mis/plugin/security/connector/AcceptorHandler.java 0.00% 20 Missing ⚠️
...oker/artemis/plugin/security/ActivateCallback.java 0.00% 13 Missing ⚠️
...kapua/client/security/bean/KapuaPrincipalImpl.java 0.00% 6 Missing ⚠️
...vice/authentication/authentication/AclCreator.java 0.00% 6 Missing ⚠️
...ua/broker/artemis/plugin/security/context/Acl.java 0.00% 4 Missing ⚠️
...broker/artemis/plugin/security/SecurityPlugin.java 0.00% 1 Missing ⚠️
...a/broker/artemis/plugin/security/ServerPlugin.java 0.00% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@              Coverage Diff              @@
##             develop    #4087      +/-   ##
=============================================
- Coverage      16.72%   16.72%   -0.01%     
  Complexity        22       22              
=============================================
  Files           2032     2033       +1     
  Lines          52745    52751       +6     
  Branches        4439     4438       -1     
=============================================
  Hits            8821     8821              
- Misses         43526    43532       +6     
  Partials         398      398              
Files with missing lines Coverage Δ
...broker/artemis/plugin/security/SecurityPlugin.java 0.00% <0.00%> (ø)
...a/broker/artemis/plugin/security/ServerPlugin.java 0.00% <0.00%> (ø)
...ua/broker/artemis/plugin/security/context/Acl.java 0.00% <0.00%> (ø)
...kapua/client/security/bean/KapuaPrincipalImpl.java 0.00% <0.00%> (ø)
...vice/authentication/authentication/AclCreator.java 0.00% <0.00%> (ø)
...oker/artemis/plugin/security/ActivateCallback.java 0.00% <0.00%> (ø)
...mis/plugin/security/connector/AcceptorHandler.java 0.00% <0.00%> (ø)
---- 🚨 Try these New Features:

@riccardomodanese riccardomodanese force-pushed the feat-upgrade_artemis branch 2 times, most recently from 7876e89 to e8a20a7 Compare November 19, 2024 11:50
@Coduz
Copy link
Contributor

Coduz commented Nov 19, 2024

/request-license-review

Copy link

/request-license-review

License review requests:

After all reviews have concluded, re-run the license-vetting check from the Github Actions web-interface to update its status.

Workflow run (with attached summary files):
https://github.com/eclipse/kapua/actions/runs/11917641780

@riccardomodanese
Copy link
Contributor Author

/request-license-review

Copy link

/request-license-review

⚠️ Failed to request review of not vetted licenses.

Workflow run (with attached summary files):
https://github.com/eclipse/kapua/actions/runs/11932396047

Signed-off-by: riccardomodanese <riccardo.modanese@eurotech.com>
Signed-off-by: riccardomodanese <riccardo.modanese@eurotech.com>
riccardomodanese and others added 7 commits November 21, 2024 14:26
Signed-off-by: riccardomodanese <riccardo.modanese@eurotech.com>
…s the bridges)

Signed-off-by: riccardomodanese <riccardo.modanese@eurotech.com>
Signed-off-by: riccardomodanese <riccardo.modanese@eurotech.com>
Signed-off-by: riccardomodanese <riccardo.modanese@eurotech.com>
Signed-off-by: riccardomodanese <riccardo.modanese@eurotech.com>
Signed-off-by: riccardomodanese <riccardo.modanese@eurotech.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CQ pending This PR needs a CQ to be approved from Eclipse before merging.
Projects
Status: In Review
Development

Successfully merging this pull request may close these issues.

3 participants