Project

General

Profile

Actions

Task #3271

closed

Task #3270: Upgrade Spring Boot

Spring Boot 2.1 > 2.2

Added by Tomáš Doischer almost 2 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Peter Štrunc
Category:
Maintenance
Target version:
Start date:
03/07/2023
Due date:
% Done:

100%

Estimated time:
60.00 h
Owner:
Peter Štrunc

Description

Pretty massive changes in this version. There may be some painful refactoring.

https://github.com/spring-projects/spring-boot/wiki/Spring-Boot-2.2-Release-Notes

Actions #1

Updated by Tomáš Doischer almost 2 years ago

  • Parent task set to #3270
Actions #2

Updated by Tomáš Doischer almost 2 years ago

  • Description updated (diff)
  • Estimated time set to 60.00 h
Actions #3

Updated by Tomáš Doischer almost 2 years ago

  • Sprint set to Sprint 13.1-2 (bře 08 - bře 22)
  • Assignee set to Tomáš Doischer
Actions #4

Updated by Tomáš Doischer almost 2 years ago

  • Status changed from New to In Progress
Actions #5

Updated by Tomáš Doischer almost 2 years ago

  • Category set to Maintenance
  • % Done changed from 0 to 50

Current state:

  • Flyway (5 > 6) now works but a minor change in each module is needed
  • Quartz did not work at some point but now seems to work fine
  • massive changes in HATEOAS - I used a migration shell script supplied by Spring but it is a little trigger happy so a little more careful replacement will be used for standard module migration
  • REST API does not quite work
Actions #6

Updated by Tomáš Doischer almost 2 years ago

Swagger needs to be upgraded to 3.0.0.

Actions #7

Updated by Tomáš Doischer almost 2 years ago

  • % Done changed from 50 to 70

The application now starts and works roughly as expected.

Tests needed some change in flyway config (just disabling flyway is not enough) and now seem to work well.

Swagger currently does not work.

Actions #8

Updated by Tomáš Doischer almost 2 years ago

  • Sprint changed from Sprint 13.1-2 (bře 08 - bře 22) to Sprint 13.1-3 (bře 22 - dub 05)
Actions #9

Updated by Tomáš Doischer over 1 year ago

Swagger works.

Ic module has no description for some reason.
LRTs do cannot be started.

Actions #10

Updated by Tomáš Doischer over 1 year ago

The issue with nonstarting LRTs was caused by the shift to lazy bean initialization (by default!!!) in 2.2. As a result, annotation processing looked at interfaces an not proxies. LRTs work fine now.

Some tests are failing due to changes in Mockito and will require rewriting, @InjectMock does not work as it used to.

Actions #11

Updated by Tomáš Doischer over 1 year ago

  • Sprint changed from Sprint 13.1-3 (bře 22 - dub 05) to Sprint 13.1-4 (dub 05 - dub 19)
Actions #12

Updated by Tomáš Doischer over 1 year ago

  • Status changed from In Progress to Needs feedback
  • Assignee changed from Tomáš Doischer to Peter Štrunc
  • % Done changed from 70 to 80

All tests finally passed.

PR: https://github.com/bcvsolutions/CzechIdMng/pull/349

Can you take a look?

Actions #13

Updated by Tomáš Doischer over 1 year ago

  • Sprint changed from Sprint 13.1-4 (dub 05 - dub 19) to Sprint 13.1-5 (dub 19 - kvě 03)
Actions #14

Updated by Peter Štrunc over 1 year ago

  • Sprint changed from Sprint 13.1-5 (Apr 19 - May 03) to Appliance / Cross domain / IdStory Hub - 9 (Sep 18 - Oct 02)
Actions #15

Updated by Peter Štrunc about 1 year ago

  • Status changed from Needs feedback to Resolved
  • % Done changed from 80 to 100
  • Owner set to Peter Štrunc

Looks good. Needed to fix a few test and rebase on current develop. Merged. Closing ticket

Actions #16

Updated by Martin Kolombo about 1 year ago

  • Sprint changed from Appliance / Cross domain / IdStory Hub - 9 (Sep 18 - Oct 02) to Next Backlog (Oct 30 - Nov 13)
Actions #17

Updated by Peter Štrunc about 1 year ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF