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

packages/nixos: write Kata agent output to console and journal #1087

Merged
merged 1 commit into from
Dec 19, 2024

Conversation

msanft
Copy link
Contributor

@msanft msanft commented Dec 19, 2024

Previously, we only wrote Kata agent output to the console / TTY (which is also what it does upstream 1). This makes the output appear in the Kata runtime logs on the Host via the vmconsole= field. However, when debugging from within the pod VM, it's helpful to also have the logs in there so that they can be searched in with journalctl. This makes it so the output of the agent is written to both the console and the journal.

Footnotes

  1. https://github.com/kata-containers/kata-containers/blob/3.10.1/src/agent/kata-agent.service.in

Previously, we only wrote Kata agent output to the console / TTY (which is also what it does upstream [^1]). This makes the output appear in the Kata runtime logs on the Host via the `vmconsole=` field. However, when debugging from within the pod VM, it's helpful to also have the logs in there so that they can be searched in with `journalctl`. This makes it so the output of the agent is written to both the console _and_ the journal.

[^1]: https://github.com/kata-containers/kata-containers/blob/3.10.1/src/agent/kata-agent.service.in
@msanft msanft requested a review from burgerdev December 19, 2024 09:03
@msanft msanft requested a review from katexochen as a code owner December 19, 2024 09:03
@msanft msanft added the no changelog PRs not listed in the release notes label Dec 19, 2024
@katexochen
Copy link
Member

what does the [1/x] mean? How is this related to the other PRs you just opened?

@msanft
Copy link
Contributor Author

msanft commented Dec 19, 2024

what does the [1/x] mean? How is this related to the other PRs you just opened?

My idea was to mark each PR that originates from the burgerdev/gpu branch, so that e.g. @burgerdev, who created (part of) the patches can match against that.

@katexochen
Copy link
Member

If things depend on each other, please base dependent branches on the PR they depend on. Otherwise I don't see how this is helpful.

@msanft
Copy link
Contributor Author

msanft commented Dec 19, 2024

If things depend on each other, please base dependent branches on the PR they depend on. Otherwise I don't see how this is helpful.

They don't depend on each other, so I've just removed the mark.

@msanft msanft changed the title [1/x] packages/nixos: write Kata agent output to console and journal packages/nixos: write Kata agent output to console and journal Dec 19, 2024
@msanft msanft merged commit 21e6d0d into main Dec 19, 2024
14 checks passed
@msanft msanft deleted the msanft/kata-agent-output branch December 19, 2024 10:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no changelog PRs not listed in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants