fix: host no kube-ovn-speaker log

[zbb88888]

Pull Request

• Make sure you have followed Kube-OVN Code Style.

What type of this PR

Examples of user facing changes:

• Bug fixes

Which issue(s) this PR fixes

Fixes #(issue-number)

[gemini-code-assist]

Summary of Changes

Hello @zbb88888, I'm Gemini Code Assist¹! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request addresses a bug where the kube-ovn-speaker component was not generating logs, hindering the ability to diagnose and troubleshoot issues. By introducing specific logging arguments, the change ensures that critical operational data from the speaker is consistently recorded, significantly improving the observability and debuggability of the Kube-OVN network.

Highlights

• Kube-OVN Speaker Logging: Explicitly configured the kube-ovn-speaker component to output logs to a dedicated file (/var/log/kube-ovn/kube-ovn-speaker.log) and also to stderr. This ensures that speaker logs are captured for debugging and operational monitoring.
• Log File Management: Set a maximum size of 200MB for the kube-ovn-speaker log file, preventing it from consuming excessive disk space.

Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature	Command	Description
Code Review	/gemini review	Performs a code review for the current pull request in its current state.
Pull Request Summary	/gemini summary	Provides a summary of the current pull request in its current state.
Comment	@gemini-code-assist	Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help	/gemini help	Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩

[gemini-code-assist]

Code Review

This pull request addresses an issue where kube-ovn-speaker was not producing logs on the host. It achieves this by adding several command-line arguments to the kube-ovn-speaker container in both the Helm chart (charts/kube-ovn-v2/templates/speaker/speaker.yaml) and the static manifest (yamls/speaker.yaml). The added arguments correctly configure klog to write logs to a file that is mounted from the host.

My review focuses on improving the maintainability and configurability of these changes. I've suggested making the hardcoded log file size configurable in the Helm chart and adding a guiding comment in the static manifest. These changes will make the configuration more flexible and user-friendly.

[coveralls]

Pull Request Test Coverage Report for Build 21243324836

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage remained the same at 22.889%

---

Totals
Change from base Build 21238354955:	0.0%
Covered Lines:	12303
Relevant Lines:	53751

---

💛 - Coveralls