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

Closes #2404 - Upgrade to Spring Boot 3.1.5 #2428

Merged
merged 6 commits into from
Nov 10, 2023
Merged

Conversation

arolfes
Copy link
Contributor

@arolfes arolfes commented Nov 9, 2023

Upgrading to Spring Boot 3.1.x required an upgrade to jakarta.
The Wildfly server 25.0.1 does not yet support jakarta, so it had to be upgraded as well.
With Spring Boot 3.1.X, SLF4J 2.0.X was also added to TASKANA.

We tried to adapt as little production & test code as possible, except for the imports.

However, this was not possible in the CDI module.
Here a new method with an empty body had to be implemented so that the PostConstruct method is executed first.
is executed first.
The other changes in the CDI module concern the switch to Wildfly 27 and the upgrade of the arquillian test module.
After reviewing the sonar-cloud results, I switched from property injection to constructor injection for all cdi related classes.

The taskana-routing-rest module could no longer be built because it had a dependency on the dmn-xlsx-converter from camunda. This community project has not yet been updated to jakarta. So we cloned dmn-xlsx-converter and built it as a submodule of taskana-routing-parent.
We made all necessary changes available to the project as PR and documented them in the clone.

SonarCloud results
https://sonarcloud.io/summary/new_code?id=norman-schmidt_taskana&branch=SPRING_BOOT_3_1_x
All warnings regarding the dmn-xlsx-converter will not be fixed for the time being. How this will be handled has yet to be decided.

Addendum Nr: 1
In the module taskana-routing-rest I switched from com.sun.xml.bind:jaxb-core to org.glassfish.jaxb:jaxb-core and com.sun.xml.bind:jaxb-impl to org.glassfish.jaxb:jaxb-runtime .
There is already a discussion about this. #2421 (comment)
Should we stay with glassfish here or switch back to sun?

Addendum Nr: 2
While upgrading to Wildfly 27.0.1 I noticed that a complete EE server is started here.
In my opinion this is not necessary.
Changing the configuration of the Wildfly server at this point can speed up testing considerably.
However, this is not part of the Spring Boot upgrade and should be addressed in a separate ticket.

Release Notes:

* Move to Spring Boot 3.1.5
* Switch to jakarta instead of javax wherever there is a jakarta equivalent.
* Change to Wildfly 27.0.1
* New submodule `dmn-xlsx-converter` as a clone of https://github.com/camunda-community-hub/camunda-dmn-xlsx/tree/master/xlsx-dmn-converter

For the submitter:

Verified by the reviewer:

  • Commit message format → (Closes) #<Issue Number>: Your commit message.
  • Submitter's update to documentation is sufficient
  • SonarCloud analysis meets our standards
  • Update of the current release notes reflects changes
  • PR fulfills the ticket
  • Edge cases and unwanted side effects are tested
  • Readability

@arolfes arolfes linked an issue Nov 9, 2023 that may be closed by this pull request
@arolfes arolfes merged commit 8c039a0 into master Nov 10, 2023
85 of 86 checks passed
@arolfes arolfes deleted the SPRING_BOOT_3_1_x branch November 10, 2023 12:08
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.

Upgrade to Spring Boot 3.1.X
3 participants