LianjiaTech / retrofit-spring-boot-starter

A spring-boot starter for retrofit, supports rapid integration and feature enhancements.(适用于retrofit的spring-boot-starter,支持快速集成和功能增强)
Apache License 2.0
1.78k stars 340 forks source link

sentinel-core升级到1.8.6熔断无法生效 #146

Closed XW512 closed 1 year ago

XW512 commented 1 year ago

我把sentinel-core的版本升级到1.8.6版本,发现熔断不能生效,降到1.6.4版本可以生效,貌似是sentinel的问题

chentianming11 commented 1 year ago

无法复现,我用1.8.6版本实测是可以熔断的。

image
XW512 commented 1 year ago

无法复现,我用1.8.6版本实测是可以熔断的。 image

能麻烦你发下你的测试代码不,或者github上的demo也行,感谢感谢

chentianming11 commented 1 year ago

在源码test代码里面,你clone整个仓库下来就能看到了。

------------------ 原始邮件 ------------------ 发件人: "LianjiaTech/retrofit-spring-boot-starter" @.>; 发送时间: 2023年3月16日(星期四) 晚上7:01 @.>; @.**@.>; 主题: Re: [LianjiaTech/retrofit-spring-boot-starter] sentinel-core升级到1.8.6熔断无法生效 (Issue #146)

无法复现,我用1.8.6版本实测是可以熔断的。

能麻烦你发下你的测试代码不,或者github上的demo也行,感谢感谢

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.Message ID: @.***>

chentianming11 commented 1 year ago
image
XW512 commented 1 year ago

在源码test代码里面,你clone整个仓库下来就能看到了。 ------------------ 原始邮件 ------------------ 发件人: "LianjiaTech/retrofit-spring-boot-starter" @.>; 发送时间: 2023年3月16日(星期四) 晚上7:01 @.>; @.**@.>; 主题: Re: [LianjiaTech/retrofit-spring-boot-starter] sentinel-core升级到1.8.6熔断无法生效 (Issue #146) 无法复现,我用1.8.6版本实测是可以熔断的。 能麻烦你发下你的测试代码不,或者github上的demo也行,感谢感谢 — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.Message ID: @.***>

好的,感谢感谢,我去试一下

XW512 commented 1 year ago
image

我想问下,retrofit-spring-boot-starter能和openfeign一起在一个项目中使用吗?因为我在测试过程中,发现如果和openfeign一起使用的话,全局熔断好像是不生效的

chentianming11 commented 1 year ago

你debug看看全局熔断配置是否生效了。 我测试代码里面,也是使用全局熔断的,可以正常工作。

image
XW512 commented 1 year ago

你debug看看全局熔断配置是否生效了。 我测试代码里面,也是使用全局熔断的,可以正常工作。 image

我debug进去看了下,配置的数据是已经注入了的,我这边使用nacos注册中心,内部调用的,我有试了下,采用baseUrl的调用,发现还是不行,方便看看你的测试代码吗?感谢感谢

XW512 commented 1 year ago

你debug看看全局熔断配置是否生效了。 我测试代码里面,也是使用全局熔断的,可以正常工作。 image

我找到原因了,如果openfeign+sentinel中sentinel的熔断配置放到nacos中去,retrofit的熔断就不会生效,如: spring: cloud: sentinel: eager: true datasource: ds1: nacos: username: nacos password: 123456 server-addr: 127.0.0.1:8848 namespace: ${spring.application.name} data-id: ${spring.application.name}-sentinel group-id: blue data-type: json rule-type: degrade