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

[7.67.x-blue] RHPAM-3709: upgrade maven dependencies to address CVE-2021-26291 #3824

Merged
merged 7 commits into from
Apr 3, 2024

Conversation

baldimir
Copy link
Member

Replaces #3823

@baldimir
Copy link
Member Author

jenkins retest this

@mareknovotny
Copy link
Member

there is enforce duplicated classes error in kie-wb-common-server-ui-backend. this needs to be resolved by exclusions or to ignore those deps in enforcer rules.

17:33:33  [WARNING] Rule 0: org.apache.maven.plugins.enforcer.BanDuplicateClasses failed with message:
17:33:33  Duplicate classes found:
17:33:33  
17:33:33    Found in:
17:33:33      com.google.inject:guice:jar:no_aop:4.0:compile
17:33:33      com.google.inject:guice:jar:4.0:test

@baldimir
Copy link
Member Author

jenkins retest this

@baldimir
Copy link
Member Author

I see the GH action checks failing in the logs after a successful build. Could someone please check? Or should I just focus on the Jenkins build?

@mareknovotny
Copy link
Member

I see the GH action checks failing in the logs after a successful build. Could someone please check? Or should I just focus on the Jenkins build?

@baldimir it builds "Succesfully" without errors, but there are 14 failed tests in both GHA jdk8/11 ;) and it is in Jenkins CI the same so it is correct

Copy link

sonarcloud bot commented Mar 28, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
3.1% Duplication on New Code (required ≤ 3%)
50.0% Condition Coverage on New Code (required ≥ 80%)

See analysis details on SonarCloud

@mareknovotny mareknovotny merged commit 54b973d into kiegroup:7.67.x-blue Apr 3, 2024
3 of 4 checks passed
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