8000 Fix delete service check ratelimit by chuntaojun · Pull Request #587 · polarismesh/polaris · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fix delete service check ratelimit #587

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related ema 8000 ils.

Already on GitHub? Sign in to your account

Conversation

chuntaojun
Copy link
Member
@chuntaojun chuntaojun commented Aug 18, 2022

Please provide issue(s) of this PR:
Fixes #

To help us figure out who should review this PR, please put an X in all the areas that this PR affects.

  • Configuration
  • Docs
  • Installation
  • Performance and Scalability
  • Naming
  • HealthCheck
  • Test and Release

Please check any characteristics that apply to this pull request.

  • Does not have any user-facing changes. This may include API changes, behavior changes, performance improvements, etc.

@codecov
Copy link
codecov bot commented Aug 18, 2022

Codecov Report

Merging #587 (bfe032d) into release-v1.11.0 (9c6e725) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@                 Coverage Diff                 @@
##           release-v1.11.0     #587      +/-   ##
===================================================
- Coverage            54.74%   54.72%   -0.03%     
===================================================
  Files                  155      155              
  Lines                22159    22160       +1     
===================================================
- Hits                 12130    12126       -4     
- Misses                8049     8055       +6     
+ Partials              1980     1979       -1     
Impacted Files Coverage Δ
store/boltdb/default.go 64.17% <ø> (ø)
service/service.go 66.43% <100.00%> (ø)
service/ratelimit_config.go 62.12% <0.00%> (-1.41%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@chuntaojun chuntaojun merged commit 12de24c into polarismesh:release-v1.11.0 Aug 19, 2022
chuntaojun added a commit that referenced this pull request Sep 5, 2022
* feat: 合入增量的限流功能修改 (#567)

* feat: 返回所有字段,避免前端出现问题 (#563)

* fix:官网文档页面错误

* feat: 限流规则优化调整

* fix: invalid default value for etime

* fix: 单机版用例失败问题

* fix:部分单测用例依赖了无效字段

* feat: 修复鉴权用例失败问题

* fix:修复用例失败

* fix: 修复限流规则过滤条件的问题

* fix: 修复用例失败问题

* 修复codecov的单测问题

* fix:调整激活时间的实现,使用真实时间戳

* feat: 修复数据库时间戳问题

* feat: 增加feat的覆盖测试

* feat:INCLUDE改成IN,贴近集合匹配的语义

* feat:接口字段为空的时候不返回 & 补齐action字段

* feat:增加启用限流规则接口

* feat:增加最大排队时长

* feat:提交新变化接口的mock

* feat:返回全部字段,否则前端无法适配

* feat:返回所有的字段,否则前端适配会出问题

* feat: 支持限流规则的最大排队时间字段 (#564)

* fix:官网文档页面错误

* feat: 限流规则优化调整

* fix: invalid default value for etime

* fix: 单机版用例失败问题

* fix:部分单测用例依赖了无效字段

* feat: 修复鉴权用例失败问题

* fix:修复用例失败

* fix: 修复限流规则过滤条件的问题

* fix: 修复用例失败问题

* 修复codecov的单测问题

* fix:调整激活时间的实现,使用真实时间戳

* feat: 修复数据库时间戳问题

* feat: 增加feat的覆盖测试

* feat:INCLUDE改成IN,贴近集合匹配的语义

* feat:接口字段为空的时候不返回 & 补齐action字段

* feat:增加启用限流规则接口

* feat:增加最大排队时长

* feat:提交新变化接口的mock

* feat:返回全部字段,否则前端无法适配

* feat:返回所有的字段,否则前端适配会出问题

* feat:增加MaxQueueDelay值的设置及返回

* feat:支持修改实例时修改地理位置信息 (#578)

* Update docker.yml

* fix: fix docker prometheus (#582)

* Fix delete service check ratelimit (#587)

* fix: fix docker prometheus

* fix: 修复删除服务时无法检查到关联的限流规则

* feat:支持注册polaris.discover以及polaris.healthcheck

* fix: 修复单机版本无法自注册问题

* test: 修复单元测试

* fix: fix standalone install can't modify port

Co-authored-by: andrew shan <45474304+andrewshan@users.noreply.github.com>
Co-authored-by: heige <zhuwei313@hotmail.com>
@chuntaojun chuntaojun deleted the fix_delete_service_check_ratelimit branch October 21, 2022 08:55
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
0