8000 [ISSUE #846] Refactor/operation log by chuntaojun · Pull Request #845 · polarismesh/polaris · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

[ISSUE #846] Refactor/operation log #845

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 12 commits into from
Dec 7, 2022
Merged

[ISSUE #846] Refactor/operation log #845

merged 12 commits into from
Dec 7, 2022

Conversation

chuntaojun
Copy link
Member
@chuntaojun chuntaojun commented Dec 5, 2022 •
edited
Loading

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

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.

@polaris-test-bot
Copy link
polaris-test-bot commented Dec 5, 2022 •
edited
Loading

2022-12-05 20:22:31: =========TEST START==========
2022-12-05 20:22:59: Pre-Merge check success! Now we will perform a full automated test which is expected to take 2 hours, thank you for your patience.
2022-12-05 20:26:15: =========TEST START==========
2022-12-05 20:26:42: Pre-Merge check success! Now we will perform a full automated test which is expected to take 2 hours, thank you for your patience.
2022-12-05 22:03:22: =========TEST START==========
2022-12-05 22:03:48: Pre-Merge check success! Now we will perform a full automated test which is expected to take 2 hours, thank you for your patience.
2022-12-07 12:30:55: =========TEST START==========
2022-12-07 12:31:22: Pre-Merge check success! Now we will perform a full automated test which is expected to take 2 hours, thank you for your patience.
2022-12-07 14:46:30: =========TEST START==========
2022-12-07 14:46:56: Pre-Merge check success! Now we will perform a full automated test which is expected to take 2 hours, thank you for your patience.
2022-12-07 16:42:22: =========TEST START==========
2022-12-07 16:42:53: Pre-Me 8000 rge check success! Now we will perform a full automated test which is expected to take 2 hours, thank you for your patience.
2022-12-07 17:56:34: =========TEST START==========
2022-12-07 17:57:02: Pre-Merge check success! Now we will perform a full automated test which is expected to take 2 hours, thank you for your patience.

@chuntaojun chuntaojun changed the title Refactor/operation log [ISSUE #846] Refactor/operation log Dec 5, 2022
@chuntaojun
fix issue polarismesh#629 (polarismesh#693)
77a68fd 
* docs:优化错误信息描述

* Update zh.toml

* fix:修复eureka心跳协议错误码不兼容问题

* fix:修复eureka心跳协议错误码不兼容问题

* unit:添加单元测试

* test:调整测试配置文件位置

* fix:issue polarismesh#692

* fix:issue polarismesh#692

* fix:issue polarismesh#692

* fix:issue polarismesh#692
@chuntaojun
docs:add error code desc
59882be
@chuntaojun
fix:调整license-checker的触发
7c2c0ef
@chuntaojun
fix:调整license-checker的触发
c1077bf
@chuntaojun
feat:operation log
7a8a579
@chuntaojun
style:format code style
0883402
@chuntaojun
style:format code style
aac29ab
@chuntaojun
refactor:support only print log content
20d4ad8
@chuntaojun
refactor:support only print log content
a9646da
@chuntaojun chuntaojun force-pushed the refactor/operation_log branch from d93445e to a9646da Compare December 7, 2022 04:30
@codecov
Copy link
codecov bot commented Dec 7, 2022 •
edited
Loading

Codecov Report

Merging #845 (48d7ce5) into main (69aabd6) will increase coverage by 0.14%.
The diff coverage is 88.08%.

@@            Coverage Diff             @@
##             main     #845      +/-   ##
==========================================
+ Coverage   59.79%   59.93%   +0.14%     
==========================================
  Files         135      136       +1     
  Lines       22068    22192     +124     
==========================================
+ Hits        13195    13301     +106     
- Misses       7261     7280      +19     
+ Partials     1612     1611       -1
Impacted Files Coverage Δ
common/model/naming.go 65.43% <ø> (-5.41%) ⬇️
service/client_v1.go 50.15% <0.00%> (-0.31%) ⬇️
service/server.go 38.80% <40.00%> (-0.88%) ⬇️
config/server.go 48.80% <55.55%> (+1.04%) ⬆️
auth/defaultauth/group.go 72.78% <66.66%> (-0.20%) ⬇️
service/routing_config_v1.go 28.67% <90.00%> (+0.50%) ⬆️
config/config_file.go 48.21% <90.32%> (+2.47%) ⬆️
service/instance.go 61.79% <91.66%> (-0.26%) ⬇️
config/config_file_group.go 38.56% <92.59%> (+3.95%) ⬆️
auth/defaultauth/strategy.go 67.14% <100.00%> (+0.43%) ⬆️
... and 13 more

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

@chuntaojun chuntaojun added this to the v1.14.0 milestone Dec 7, 2022
@chuntaojun chuntaojun merged commit a8c2a6c into polarismesh:main Dec 7, 2022
@chuntaojun chuntaojun deleted the refactor/operation_log branch March 17, 2023 08:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lepdou lepdou lepdou approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v1.14.0
Development

Successfully merging this pull request may close these issues.

优化事件/操作记录的日志打印
3 participants
@chuntaojun @polaris-test-bot @lepdou
0