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.

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

cut invalid OVN_NB_DAEMON to make log more readable #2601

Merged
merged 1 commit into from
Apr 5, 2023
Merged

cut invalid OVN_NB_DAEMON to make log more readable #2601

merged 1 commit into from
Apr 5, 2023

Conversation

zhangzujian
Copy link
Member

What type of this PR

Examples of user facing changes:

  • Features
  • Bug fixes
  • Docs
  • Tests

Which issue(s) this PR fixes:

Fixes #(issue-number)

@zhangzujian zhangzujian marked this pull request as ready for review April 4, 2023 10:03
@zhangzujian zhangzujian requested a review from oilbeater April 4, 2023 10:03
@github-actions
Copy link
Contributor

github-actions bot commented Apr 4, 2023

  • The commit message is missing, it should provide a clear and concise description of the changes made in this patch.
  • There are no potential bugs or format errors in this patch.
  • There are no performance issues in this patch.
  • The code change seems reasonable, but it would be better to add a comment explaining why the OVN_NB_DAEMON variable is being truncated to 64 characters.

@zhangzujian zhangzujian merged commit d2523f4 into kubeovn:master Apr 5, 2023
@zhangzujian zhangzujian deleted the fix-log branch April 5, 2023 00:53
zhangzujian added a commit that referenced this pull request Apr 5, 2023
@zhangzujian
cut invalid OVN_NB_DAEMON to make log more readable (#2601)
fdc2301
zhangzujian added a commit that referenced this pull request Apr 5, 2023
@zhangzujian
cut invalid OVN_NB_DAEMON to make log more readable (#2601)
48e9384
zhangzujian added a commit that referenced this pull request Apr 5, 2023
@zhangzujian
cut invalid OVN_NB_DAEMON to make log more readable (#2601)
e77adbb
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@oilbeater oilbeater oilbeater approved these changes

Assignees
No one assigned
Labels
need backport
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@zhangzujian @oilbeater