Skip to content

Conversation

@xuxiaowei-com-cn
Copy link
Contributor

@xuxiaowei-com-cn xuxiaowei-com-cn commented Dec 20, 2025

Describe what this PR does / why we need it

Does this pull request fix one issue?

#4178
#4180

Describe how you did it

Describe how to verify it

run https://github.com/xuxiaowei-com-cn/spring-cloud-alibaba-example/blob/main/spring-cloud-2025.1.x/user-actuator/src/test/java/cn/com/xuxiaowei/user/Actuator_2025_1_x_Tests.java

Special notes for reviews

…spring.config.import configuration (alibaba#4156)"

This reverts commit d45607e.
@xuxiaowei-com-cn
Copy link
Contributor Author

@uuuyuqi

  1. 我发现 在 2025/12/16 20:24 之前的 2025.1.x 分支 的几次提交记录质量堪忧
  2. 比如:不支持配置动态刷新、读取配置不支持 List、强行要求必须填写 spring.config.import、不支持 spring-boot-starter-actuator
  3. 后来我定位到好像是因为 issues/4147 缺失spring.config.import,NacosConfigManager不进行spring bean注册 #4156 的原因,所以我的本次 PR 撤回了 d45607e

测试方式

  1. 我把 alibaba/spring-cloud-alibaba 在 2025/12/16 20:24 之前的几次提交撤回了,仅保留 ✨ support spring-boot 4.0.0、spring-cloud 2025.1.0 #4068
  2. 代码推送到 https://gitlab.xuxiaowei.com.cn/xuxiaowei-com-cn/spring-cloud-alibaba-refresh
  3. 发布到 https://gitlab.xuxiaowei.com.cn/xuxiaowei-com-cn/spring-cloud-alibaba-refresh/-/packages
  4. 测试程序 https://github.com/xuxiaowei-com-cn/spring-cloud-alibaba-examplexuxiaowei/user-actuator 分支,提交记录:xuxiaowei-com-cn/spring-cloud-alibaba-example@067eacb
  5. Actuator_2025_1_x_Tests.java 测试 spring-boot-starter-actuator
  6. UserActuatorController_2025_1_x_Tests.java 测试刷新配置
  7. 注意看一下 https://github.com/xuxiaowei-com-cn/spring-cloud-alibaba-example 中的 README.md

@xuxiaowei-com-cn xuxiaowei-com-cn changed the title ✨ support spring-boot-starter-actuator ✨ support spring-boot-starter-actuator, config refresh Dec 20, 2025
@forestwanglin
Copy link

辛苦,正好在做springboot4+nacos的尝试。

@yuhuangbin
Copy link
Collaborator

close by #4167

@yuhuangbin yuhuangbin closed this Dec 23, 2025
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.

3 participants