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

FISH-6081-Payara6-cdi-4.0-upgrade #5819

Merged
merged 75 commits into from
Jul 6, 2022

Conversation

breakponchito
Copy link
Contributor

@breakponchito breakponchito commented Jun 21, 2022

FISH-6081 Payara6 CDI 4.0 upgrade

Description

This is the PR to upgrade cdi to the latest version compatible with Jakarta EE 10. This include the following upgraded versions

  • WELD api: 5.0.SP2
  • WELD: 5.0.1.Final
  • CDI API: 4.0.1

Important Info

Blockers

Testing

New tests

CDI full passing tests:
[mvn.test] [INFO] Tests run: 1831, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 1,082.431 s - in TestSuite
[mvn.test] [INFO]
[mvn.test] [INFO] Results:
[mvn.test] [INFO]
[mvn.test] [INFO] Tests run: 1831, Failures: 0, Errors: 0, Skipped: 0

Testing Performed

executing the tck from the cditck-porting project

Testing Environment

Ubuntu 20.04, Azul JDK 11, Maven 3.6.1, ant 1.10.12

Documentation

Notes for Reviewers

Still working to configure correctly the CDI Core profile. I identified an issue that cause to execute around 1295 tests but the correct number should be 725

# Conflicts:
#	appserver/admingui/pom.xml
#	appserver/extras/javaee/manifest-jar/pom.xml
#	appserver/packager/external/jakarta-ee9-shim/pom.xml
#	appserver/web/web-glue/src/main/java/org/glassfish/web/jsp/JSPCompiler.java
#	pom.xml
# Conflicts:
#	appserver/extras/javaee/manifest-jar/pom.xml
#	appserver/packager/external/jakarta-ee9-shim/pom.xml
#	pom.xml
@breakponchito
Copy link
Contributor Author

Jenkins test please

@breakponchito breakponchito changed the base branch from Payara6-tck to Payara6 June 21, 2022 19:03
@breakponchito
Copy link
Contributor Author

Jenkins test please

@breakponchito
Copy link
Contributor Author

Jenkins test please

@breakponchito
Copy link
Contributor Author

Jenkins test please

@breakponchito
Copy link
Contributor Author

Jenkins test please

@breakponchito
Copy link
Contributor Author

Jenkins test please

@breakponchito
Copy link
Contributor Author

Jenkins test please

3 similar comments
@breakponchito
Copy link
Contributor Author

Jenkins test please

@breakponchito
Copy link
Contributor Author

Jenkins test please

@breakponchito
Copy link
Contributor Author

Jenkins test please

@breakponchito
Copy link
Contributor Author

Jenkins test please

@Pandrex247 Pandrex247 changed the base branch from Payara6 to Payara6-tck July 6, 2022 14:55
@Pandrex247 Pandrex247 merged commit 9da842f into payara:Payara6-tck Jul 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: DO NOT MERGE Don't merge PR until further notice
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants