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.
100%
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
Updated by Tomáš Doischer almost 2 years ago
- Description updated (diff)
- Estimated time set to 60.00 h
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
Updated by Tomáš Doischer almost 2 years ago
- Status changed from New to In Progress
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
Updated by Tomáš Doischer almost 2 years ago
Swagger needs to be upgraded to 3.0.0.
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.
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)
Updated by Tomáš Doischer over 1 year ago
Swagger works.
Ic module has no description for some reason.
LRTs do cannot be started.
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.
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)
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?
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)
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)
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
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)
Updated by Peter Štrunc about 1 year ago
- Status changed from Resolved to Closed