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

Cameronbaird/hyp loglevel #303

Open
wants to merge 1 commit into
base: msft-main
Choose a base branch
from
Open

Conversation

Camelron
Copy link

@Camelron Camelron commented Jan 29, 2025

Merge Checklist
  • Followed patch format from upstream recommendation: https://github.com/kata-containers/community/blob/main/CONTRIBUTING.md#patch-format
    • Included a single commit in a given PR - at least unless there are related commits and each makes sense as a change on its own.
  • Aware about the PR to be merged using "create a merge commit" rather than "squash and merge" (or similar)
  • The upstream/missing label (or upstream/not-needed) has been set on the PR.
Summary

Introduce new config, hypervisor_debug_level, which allows the user to specify the verbosity of the hypervisor.

Test Methodology

Built and deployed on an AKS kata cluster. Set the new config param to 3 and saw that the verbosity increased for clh:
image

@Camelron Camelron requested review from a team as code owners January 29, 2025 22:27
@Camelron Camelron force-pushed the cameronbaird/hyp-loglevel branch 2 times, most recently from 73ace7f to da33aba Compare January 29, 2025 22:34
Implement HypervisorLoglevel config option for clh.

Signed-off-by: Cameron Baird <[email protected]>
@Camelron Camelron force-pushed the cameronbaird/hyp-loglevel branch from da33aba to b785624 Compare January 30, 2025 18:49
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.

1 participant