Skip to content
This repository has been archived by the owner on Dec 7, 2023. It is now read-only.

Save the contents of the FC fifos to files #88

Open
luxas opened this issue Jul 1, 2019 · 1 comment
Open

Save the contents of the FC fifos to files #88

luxas opened this issue Jul 1, 2019 · 1 comment
Assignees
Labels
good first issue Good for newcomers hacktoberfest help wanted Extra attention is needed kind/enhancement Categorizes issue or PR as related to improving an existing feature. priority/backlog Higher priority than priority/awaiting-more-evidence.
Milestone

Comments

@luxas
Copy link
Contributor

luxas commented Jul 1, 2019

Firecracker emits one fifo for metrics and one for VMM logs. Both of these are by nature very ephemeral. ignite-spawn should read the fifos periodically (or better, stream) and write the contents to persistent log files in /var/lib/firecracker/vm/$id/{vmm,metrics}.log

@luxas luxas self-assigned this Jul 1, 2019
@luxas luxas modified the milestones: v0.4.0, v0.5.0 Jul 1, 2019
@luxas
Copy link
Contributor Author

luxas commented Jul 17, 2019

Hi @patrobinson would you like to help with this issue?
Basically, it's about opening the two FIFO files Firecracker emits, reading them using https://github.com/containerd/fifo; and streaming the read content to a persistent file, e.g. a file with the same name, but ending with .log and not .fifo.

What do you think? 😄

@luxas luxas added help wanted Extra attention is needed kind/enhancement Categorizes issue or PR as related to improving an existing feature. priority/backlog Higher priority than priority/awaiting-more-evidence. labels Jul 17, 2019
@luxas luxas modified the milestones: v0.5.0, Next Aug 2, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
good first issue Good for newcomers hacktoberfest help wanted Extra attention is needed kind/enhancement Categorizes issue or PR as related to improving an existing feature. priority/backlog Higher priority than priority/awaiting-more-evidence.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants