Skip to content
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 emails.

Already on GitHub? Sign in to your account

feat: 分离客户端检查与服务实例检查的机制,以免互相影响 #930

Merged
merged 23 commits into from
Feb 20, 2023
Merged

Conversation

andrewshan
Copy link
Member

@andrewshan andrewshan commented Feb 19, 2023

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

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

  • ApiServer
  • Auth
  • Configuration
  • Naming
  • HealthCheck
  • Metrics
  • Docs
  • Installation
  • Performance and Scalability
  • 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.

andrewshan and others added 21 commits January 17, 2023 10:33
feat: optimize route rule v2->v1 subset name conversion (#901)
fix: 单机版过滤条件不生效问题 (#905)
fix: 无法拉取统配规则 & mysql接入时规则信息更新无效 (#906)
fix: fix the faultdetect config db write error (#908)
@polaris-test-bot
Copy link

polaris-test-bot commented Feb 19, 2023

2023-02-19 15:38:56: =========TEST START==========
2023-02-19 15:39:24: Pre-Merge check success! Now we will perform a full automated test which is expected to take 2 hours, thank you for your patience.
2023-02-19 15:52:41: [STANDALONE VERSION]PR autotest PASSED! We executed 20 test cases and 20 were passed. You can see test report in https://polaris-opentest-1306863770.cos.ap-guangzhou.myqcloud.com/pr1246179039/sta/qta-report.html
2023-02-19 15:52:44: [CLUSTER VERSION] PR autotest PASSED! We executed 20 test cases and 20 were passed. You can see test report in https://polaris-opentest-1306863770.cos.ap-guangzhou.myqcloud.com/pr1246179039/clu/qta-report.html
2023-02-19 15:52:49: [KUBENETES VERSION]PR autotest PASSED! We executed 20 test cases and 20 were passed. You can see test report in https://polaris-opentest-1306863770.cos.ap-guangzhou.myqcloud.com/pr1246179039/k8s/qta-report.html
2023-02-19 22:05:22: =========TEST START==========
2023-02-19 22:05:49: Pre-Merge check success! Now we will perform a full automated test which is expected to take 2 hours, thank you for your patience.
2023-02-19 22:18:38: [STANDALONE VERSION]PR autotest PASSED! We executed 20 test cases and 20 were passed. You can see test report in https://polaris-opentest-1306863770.cos.ap-guangzhou.myqcloud.com/pr1246179039/sta/qta-report.html
2023-02-19 22:18:41: [CLUSTER VERSION] PR autotest PASSED! We executed 20 test cases and 20 were passed. You can see test report in https://polaris-opentest-1306863770.cos.ap-guangzhou.myqcloud.com/pr1246179039/clu/qta-report.html
2023-02-19 22:18:45: [KUBENETES VERSION]PR autotest PASSED! We executed 20 test cases and 20 were passed. You can see test report in https://polaris-opentest-1306863770.cos.ap-guangzhou.myqcloud.com/pr1246179039/k8s/qta-report.html
2023-02-19 22:41:12: =========TEST START==========
2023-02-19 22:41:39: Pre-Merge check success! Now we will perform a full automated test which is expected to take 2 hours, thank you for your patience.
2023-02-19 22:55:07: [STANDALONE VERSION]PR autotest PASSED! We executed 20 test cases and 20 were passed. You can see test report in https://polaris-opentest-1306863770.cos.ap-guangzhou.myqcloud.com/pr1246179039/sta/qta-report.html
2023-02-19 22:55:11: [CLUSTER VERSION] PR autotest PASSED! We executed 20 test cases and 20 were passed. You can see test report in https://polaris-opentest-1306863770.cos.ap-guangzhou.myqcloud.com/pr1246179039/clu/qta-report.html
2023-02-19 22:55:14: [KUBENETES VERSION]PR autotest PASSED! We executed 20 test cases and 20 were passed. You can see test report in https://polaris-opentest-1306863770.cos.ap-guangzhou.myqcloud.com/pr1246179039/k8s/qta-report.html

@codecov
Copy link

codecov bot commented Feb 19, 2023

Codecov Report

Merging #930 (844ce33) into main (248b222) will increase coverage by 1.66%.
The diff coverage is 50.66%.

@@            Coverage Diff             @@
##             main     #930      +/-   ##
==========================================
+ Coverage   55.30%   56.97%   +1.66%     
==========================================
  Files         183      184       +1     
  Lines       33549    33586      +37     
==========================================
+ Hits        18555    19136     +581     
+ Misses      12521    11938     -583     
- Partials     2473     2512      +39     
Impacted Files Coverage Δ
service/healthcheck/leader.go 0.00% <0.00%> (ø)
service/healthcheck/time_adjust.go 62.85% <0.00%> (+0.35%) ⬆️
service/healthcheck/config.go 30.43% <50.00%> (+10.43%) ⬆️
service/healthcheck/server.go 13.81% <50.00%> (+4.41%) ⬆️
service/healthcheck/check.go 66.74% <86.72%> (+59.51%) ⬆️
service/healthcheck/cache.go 74.20% <100.00%> (+33.33%) ⬆️
service/healthcheck/test_export.go 69.04% <100.00%> (+0.75%) ⬆️
store/boltdb/client.go 70.83% <100.00%> (+9.37%) ⬆️
cache/user.go 81.42% <0.00%> (-5.47%) ⬇️
store/mysql/maintain.go 51.38% <0.00%> (-3.96%) ⬇️
... and 19 more

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

@chuntaojun chuntaojun merged commit bdbbdb6 into polarismesh:main Feb 20, 2023
@chuntaojun chuntaojun linked an issue Feb 20, 2023 that may be closed by this pull request
andrewshan added a commit that referenced this pull request Mar 1, 2023
* feat: 分离客户端检查与服务实例检查的机制,以免互相影响 (#930)

* test: 补充健康检查相关的单测 (#933)

* feat: upgrade specification to 1.1.0 stable release

* feat:适配V1规则的分组名,V1规则分组名=V2规则名.分组名

* fix: boltdb load int value should be int64

* fix:单机版规则过滤标签不生效

* fix:修复静态列数据更新问题以及统配规则新加入缓存不生效问题

* feat:数据库字段修改为下划线

* fix: 解决mysql数据库写入失败问题

* fix: fix golint erros

* feat:合入1.13.x改动的eureka兼容相关的问题

* fix: golint-ci issues

* feat: 客户端健康检查与服务实例健康检查机制分离

* fix: sqldb healthcheck config

* fix: 修复客户端管理时间单位不一致的问题

* feat:支持存量熔断规则的迁移

* feat:补充心跳上报单测

* fix: eureka updateStatus鉴权失败

* fix:db integrate test run failed

* fix: database deadlock issues

* fix: all write db operation should be in transaction

* fix: group not in tx

* feat:增加串行创建实例的覆盖

* feat:remove ListNamespaces refer

* fix: 解决不在同一个事务的问题

* fix: compile fail variables

* feat: eureka创建和删除实例使用异步处理
andrewshan added a commit that referenced this pull request Mar 3, 2023
* feat: 分离客户端检查与服务实例检查的机制,以免互相影响 (#930)

* test: 补充健康检查相关的单测 (#933)

* feat: upgrade specification to 1.1.0 stable release

* feat:适配V1规则的分组名,V1规则分组名=V2规则名.分组名

* fix: boltdb load int value should be int64

* fix:单机版规则过滤标签不生效

* fix:修复静态列数据更新问题以及统配规则新加入缓存不生效问题

* feat:数据库字段修改为下划线

* fix: 解决mysql数据库写入失败问题

* fix: fix golint erros

* feat:合入1.13.x改动的eureka兼容相关的问题

* fix: golint-ci issues

* feat: 客户端健康检查与服务实例健康检查机制分离

* fix: sqldb healthcheck config

* fix: 修复客户端管理时间单位不一致的问题

* feat:支持存量熔断规则的迁移

* feat:补充心跳上报单测

* fix: eureka updateStatus鉴权失败

* fix:db integrate test run failed

* fix: database deadlock issues

* fix: all write db operation should be in transaction

* fix: group not in tx

* feat:增加串行创建实例的覆盖

* feat:remove ListNamespaces refer

* fix: 解决不在同一个事务的问题

* fix: compile fail variables

* feat: eureka创建和删除实例使用异步处理

* fix: 简化心跳事件的推送逻辑,去掉二次channel传递 (#975)

* fix: eureka updateStatus鉴权失败

* fix:db integrate test run failed

* fix: database deadlock issues

* fix: all write db operation should be in transaction

* fix: group not in tx

* feat:增加串行创建实例的覆盖

* feat:remove ListNamespaces refer

* fix: 解决不在同一个事务的问题

* feat: 迁移工具支持环境变量传参,以及打印日志

* feat: eureka创建和删除实例使用异步处理

* feat:解决首次注册后心跳上报失败的问题 & 针对复制的实例可以允许关闭健康状态的转换

* feat:修复实例多次后404的问题

* fix: 修复用例失败问题

* fix: 简化心跳事件的推送逻辑,去掉二次channel传递

* fix: 将服务名转换提取成公共逻辑

* feat:event用例调整

* feat: update go version

* fix:ut fail issues

* fix: 用例失败问题

* update go.mod version to 1.19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants