Skip to content

Conversation

ahoehma
Copy link

@ahoehma ahoehma commented Aug 9, 2025

No description provided.

Andreas Höhmann added 3 commits August 9, 2025 13:55
- I did "mvn install -DskipTests=true -nsu -Dcheckstyle.skip=false -pl '!spring-boot-admin-docs'"
- spring-boot-admin-docs doesn't build successfully no glue why
@ahoehma ahoehma requested a review from a team as a code owner August 9, 2025 11:56
Andreas Höhmann added 2 commits August 9, 2025 14:10
…re.MailSenderAutoConfiguration]

- some tests failing because of missing MailSenderAutoConfiguration
- adding spring-boot-starter-mail in test-scope
@ahoehma
Copy link
Author

ahoehma commented Aug 10, 2025

Hmmm ... The de.codecentric.boot.admin.client.ClientServletApplicationTest hangs forever (in my fork build) ... Is this normal?

image

Locally its "working" ...

image

- the CountDownLatch.await should have a timeout too
@ahoehma
Copy link
Author

ahoehma commented Aug 10, 2025

I found a major problem ... may you guys have an idea ...

Currently de.codecentric.boot.admin.client.config.SpringBootAdminClientRegistrationClientAutoConfigurationTest is failing for some of the defined "what kind of RegistrationClient is used" tests ... because de.codecentric.boot.admin.client.config.SpringBootAdminClientAutoConfiguration is mixing restclient and webclient and the bean-conditions which working with sb 3.5.x are no longer working because of the new splitting of the springboot dependencies ... may I'm wrong ...

image image

…re.MailSenderAutoConfiguration]

- some tests failing because of missing MailSenderAutoConfiguration
- adding spring-boot-starter-mail in test-scope
@ahoehma ahoehma marked this pull request as draft August 10, 2025 10:17
@ahoehma
Copy link
Author

ahoehma commented Aug 10, 2025

jolokia-support-spring is also not ready yet for spring-boot 4: jolokia/jolokia#866
I started a MR there as well: jolokia/jolokia#867

Andreas Höhmann added 9 commits August 11, 2025 07:24
- wee need both starters because of import of RestTemplateAutoConfiguration,
  RestClientAutoConfiguration and WebClientAutoConfiguration in SpringBootAdminClientAutoConfiguration
- locally (win 11, gitbash, eclipse) ClientServletApplicationTest not working,
  the --spring.jackson.property-naming-strategy=SNAKE_CASE is not used (no glue why)
  - in github build its working - why is this?
- locally SpringBootAdminClientRegistrationClientAutoConfigurationTest is also not
  working 100% - a lot of "wrong" RegistrationClient are created, doesn't matter
  which "builders" are in the spring context
  - in github build its working - why is this?
- I did "mvn install -DskipTests=true -nsu -Dcheckstyle.skip=false -pl '!spring-boot-admin-docs'"
- spring-boot-admin-docs doesn't build successfully no glue why
…re.MailSenderAutoConfiguration]

- some tests failing because of missing MailSenderAutoConfiguration
- adding spring-boot-starter-mail in test-scope
- the CountDownLatch.await should have a timeout too
…re.MailSenderAutoConfiguration]

- some tests failing because of missing MailSenderAutoConfiguration
- adding spring-boot-starter-mail in test-scope
- wee need both starters because of import of RestTemplateAutoConfiguration,
  RestClientAutoConfiguration and WebClientAutoConfiguration in SpringBootAdminClientAutoConfiguration
- locally (win 11, gitbash, eclipse) ClientServletApplicationTest not working,
  the --spring.jackson.property-naming-strategy=SNAKE_CASE is not used (no glue why)
  - in github build its working - why is this?
- locally SpringBootAdminClientRegistrationClientAutoConfigurationTest is also not
  working 100% - a lot of "wrong" RegistrationClient are created, doesn't matter
  which "builders" are in the spring context
  - in github build its working - why is this?
@SteKoe SteKoe force-pushed the feat/migrate-to-springboot-4 branch from c6403c2 to 13573fd Compare August 11, 2025 07:38
@SteKoe SteKoe changed the title Migration to springboot 4 feat(deps): Upgrade to Spring Boot 4 Aug 15, 2025
@SteKoe SteKoe changed the title feat(deps): Upgrade to Spring Boot 4 feat(deps): upgrade to Spring Boot 4 Aug 15, 2025
@SteKoe SteKoe mentioned this pull request Aug 20, 2025
…ing-boot-admin into feat/migrate-to-springboot-4
@ahoehma
Copy link
Author

ahoehma commented Oct 3, 2025

I update the fork to M3 ... now some more new "incompatibilities" present ... all about jackson.

For example in de.codecentric.boot.admin.server.utils.jackson.EndpointMixinTest

image

The new org.springframework.boot.test.json.JacksonTester.initFields(Object, JsonMapper)

accept only tools.jackson.databind.json.JsonMapper from jackson-databind-3.0.0-rc9.

I can't find anything in spring's migration guide for this changes :-(

https://github.com/spring-projects/spring-boot/wiki/Spring-Boot-4.0-Migration-Guide

Andreas Höhmann added 2 commits October 3, 2025 12:53
- cleaned also all "invalid" code-formattings from previous commits

- one big problem in spring-boot-admin-server

jackson's databind module 3.0.0-rc9 was included into spring which
conflicts here and there with the old jackson stuff?!
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.

1 participant