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

Support contextual logging #40

Merged
merged 1 commit into from
Apr 15, 2024
Merged

Conversation

haijianyang
Copy link
Contributor

Issue

当前 CAPE 的 context 使用不规范,例如:

Change

  • 日志使用 CAPI 社区推荐的日志规范
  • controller 传递 controller-runtime 传递过来的 ctx,并取消原有的自定义 Context,不再继承原生的 Context

Test

E2E

@haijianyang haijianyang changed the title Support Support contextual logging Support contextual logging Apr 12, 2024
@haijianyang haijianyang merged commit 50f8609 into smartxworks:main Apr 15, 2024
1 check passed
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