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

refactor:优化鉴权相关功能开关以及修复NPE问题 #1155

Merged
merged 12 commits into from Jun 15, 2023

Conversation

chuntaojun
Copy link
Member

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.

  • 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.

@polaris-test-bot
Copy link

polaris-test-bot commented Jun 14, 2023

2023-06-14 21:51:22: =========TEST START==========
2023-06-14 22:21:45: [STANDALONE VERSION]PR autotest FAILED! We executed 47 test cases and 43 were passed. You can see test report in https://polaris-opentest-1306863770.cos.ap-guangzhou.myqcloud.com/pr1392467539/sta/qta-report.html and https://polaris-opentest-1306863770.cos.ap-guangzhou.myqcloud.com/pr1392467539/sta/ratelimit/qta-report.html
2023-06-14 22:21:48: [CLUSTER VERSION] PR autotest FAILED! We executed 39 test cases and 35 were passed. You can see test report in https://polaris-opentest-1306863770.cos.ap-guangzhou.myqcloud.com/pr1392467539/clu/qta-report.html
2023-06-14 22:21:52: [KUBENETES VERSION]PR autotest FAILED! We executed 39 test cases and 38 were passed. You can see test report in https://polaris-opentest-1306863770.cos.ap-guangzhou.myqcloud.com/pr1392467539/k8s/qta-report.html
2023-06-14 22:59:51: =========TEST START==========
2023-06-14 23:01:06: =========TEST START==========
2023-06-14 23:30:37: [STANDALONE VERSION]PR autotest FAILED! We executed 47 test cases and 42 were passed. You can see test report in https://polaris-opentest-1306863770.cos.ap-guangzhou.myqcloud.com/pr1392467539/sta/qta-report.html and https://polaris-opentest-1306863770.cos.ap-guangzhou.myqcloud.com/pr1392467539/sta/ratelimit/qta-report.html
2023-06-14 23:30:41: [CLUSTER VERSION] PR autotest FAILED! We executed 39 test cases and 35 were passed. You can see test report in https://polaris-opentest-1306863770.cos.ap-guangzhou.myqcloud.com/pr1392467539/clu/qta-report.html
2023-06-14 23:30:45: [KUBENETES VERSION]PR autotest FAILED! We executed 39 test cases and 36 were passed. You can see test report in https://polaris-opentest-1306863770.cos.ap-guangzhou.myqcloud.com/pr1392467539/k8s/qta-report.html
2023-06-14 23:43:47: =========TEST START==========
2023-06-15 00:04:48: =========TEST START==========
2023-06-15 00:34:39: [STANDALONE VERSION]PR autotest FAILED! We executed 47 test cases and 44 were passed. You can see test report in https://polaris-opentest-1306863770.cos.ap-guangzhou.myqcloud.com/pr1392467539/sta/qta-report.html and https://polaris-opentest-1306863770.cos.ap-guangzhou.myqcloud.com/pr1392467539/sta/ratelimit/qta-report.html
2023-06-15 00:34:43: [CLUSTER VERSION] PR autotest FAILED! We executed 39 test cases and 35 were passed. You can see test report in https://polaris-opentest-1306863770.cos.ap-guangzhou.myqcloud.com/pr1392467539/clu/qta-report.html
2023-06-15 00:34:47: [KUBENETES VERSION]PR autotest FAILED! We executed 39 test cases and 36 were passed. You can see test report in https://polaris-opentest-1306863770.cos.ap-guangzhou.myqcloud.com/pr1392467539/k8s/qta-report.html
2023-06-15 10:53:41: =========TEST START==========
2023-06-15 11:23:30: [STANDALONE VERSION]PR autotest FAILED! We executed 47 test cases and 42 were passed. You can see test report in https://polaris-opentest-1306863770.cos.ap-guangzhou.myqcloud.com/pr1392467539/sta/qta-report.html and https://polaris-opentest-1306863770.cos.ap-guangzhou.myqcloud.com/pr1392467539/sta/ratelimit/qta-report.html
2023-06-15 11:23:34: [CLUSTER VERSION] PR autotest FAILED! We executed 39 test cases and 35 were passed. You can see test report in https://polaris-opentest-1306863770.cos.ap-guangzhou.myqcloud.com/pr1392467539/clu/qta-report.html
2023-06-15 11:23:38: [KUBENETES VERSION]PR autotest FAILED! We executed 39 test cases and 36 were passed. You can see test report in https://polaris-opentest-1306863770.cos.ap-guangzhou.myqcloud.com/pr1392467539/k8s/qta-report.html
2023-06-15 15:22:19: =========TEST START==========
2023-06-15 15:52:52: [STANDALONE VERSION]PR autotest FAILED! We executed 47 test cases and 43 were passed. You can see test report in https://polaris-opentest-1306863770.cos.ap-guangzhou.myqcloud.com/pr1392467539/sta/qta-report.html and https://polaris-opentest-1306863770.cos.ap-guangzhou.myqcloud.com/pr1392467539/sta/ratelimit/qta-report.html
2023-06-15 15:52:55: [CLUSTER VERSION] PR autotest FAILED! We executed 39 test cases and 35 were passed. You can see test report in https://polaris-opentest-1306863770.cos.ap-guangzhou.myqcloud.com/pr1392467539/clu/qta-report.html
2023-06-15 15:52:59: [KUBENETES VERSION]PR autotest FAILED! We executed 39 test cases and 36 were passed. You can see test report in https://polaris-opentest-1306863770.cos.ap-guangzhou.myqcloud.com/pr1392467539/k8s/qta-report.html
2023-06-15 17:19:07: =========TEST START==========
2023-06-15 17:51:07: [STANDALONE VERSION]PR autotest FAILED! We executed 47 test cases and 42 were passed. You can see test report in https://polaris-opentest-1306863770.cos.ap-guangzhou.myqcloud.com/pr1392467539/sta/qta-report.html and https://polaris-opentest-1306863770.cos.ap-guangzhou.myqcloud.com/pr1392467539/sta/ratelimit/qta-report.html
2023-06-15 17:51:11: [CLUSTER VERSION] PR autotest FAILED! We executed 39 test cases and 38 were passed. You can see test report in https://polaris-opentest-1306863770.cos.ap-guangzhou.myqcloud.com/pr1392467539/clu/qta-report.html
2023-06-15 17:51:16: [KUBENETES VERSION]PR autotest FAILED! We executed 39 test cases and 38 were passed. You can see test report in https://polaris-opentest-1306863770.cos.ap-guangzhou.myqcloud.com/pr1392467539/k8s/qta-report.html

@codecov
Copy link

codecov bot commented Jun 15, 2023

Codecov Report

Merging #1155 (952fce5) into main (0145a03) will decrease coverage by 0.01%.
The diff coverage is 71.94%.

❗ Current head 952fce5 differs from pull request most recent head 3bba163. Consider uploading reports for the commit 3bba163 to get more accurate results

@@            Coverage Diff             @@
##             main    #1155      +/-   ##
==========================================
- Coverage   57.68%   57.67%   -0.01%     
==========================================
  Files         213      213              
  Lines       38269    38279      +10     
==========================================
+ Hits        22074    22076       +2     
- Misses      13563    13569       +6     
- Partials     2632     2634       +2     
Impacted Files Coverage Δ
apiserver/eurekaserver/access.go 6.66% <0.00%> (ø)
config/config_file.go 59.41% <ø> (-0.18%) ⬇️
service/healthcheck/report.go 50.96% <0.00%> (ø)
service/healthcheck/server.go 31.07% <0.00%> (-0.18%) ⬇️
auth/defaultauth/auth_checker.go 66.05% <14.28%> (-1.48%) ⬇️
config/client_config_file.go 57.24% <33.33%> (+7.24%) ⬆️
service/circuitbreaker_rule_authability.go 68.08% <36.36%> (-6.39%) ⬇️
cache/cache.go 83.33% <40.00%> (-0.86%) ⬇️
config/connection_manager.go 33.33% <50.00%> (+2.97%) ⬆️
service/healthcheck/cache.go 86.11% <90.90%> (ø)
... and 2 more

... and 2 files with indirect coverage changes

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

@chuntaojun chuntaojun merged commit 6084cfc into polarismesh:main Jun 15, 2023
6 of 7 checks passed
@chuntaojun chuntaojun linked an issue Jun 15, 2023 that may be closed by this pull request
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.

关闭健康检测开关启动报错
2 participants