-
Notifications
You must be signed in to change notification settings - Fork 61
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
feat: Add Xen support #1240
base: staging
Are you sure you want to change the base?
feat: Add Xen support #1240
Conversation
for { | ||
if peekAndRead(f, reader, &logs, &errs) { | ||
break | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can get multi-line writes with one event triggered.
IMO something better would be for peekAndRead
to look ahead for the last newline
so we end up with a single write.
f32e678
to
9b5e55a
Compare
xen
support for kraftkitXen
support
4726be4
to
c67465b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some style comments from my side, I did not try it yet
cb8812c
to
96a6886
Compare
029891f
to
08c224c
Compare
python3-wheel \ | ||
uuid-dev; \ | ||
pip3 install python-config --break-system-packages; \ | ||
git clone -b stable-4.18 https://xenbits.xen.org/git-http/xen.git /xen; \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you turn this version into an ARG
?
git clone -b stable-4.18 https://xenbits.xen.org/git-http/xen.git /xen; \ | |
git clone -b stable-${XEN_VERSION} https://xenbits.xen.org/git-http/xen.git /xen; \ |
if opts.Follow && machine.Status.State == machineapi.MachineStateRunning { | ||
|
||
// Sometimes the kernel can boot and exit faster than we can start tailing the logs | ||
if opts.Follow && (machine.Status.State == machineapi.MachineStateRunning || machine.Status.State == machineapi.MachineStateExited) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Break this up on multiple lines
c517ef1
to
77bcc05
Compare
Signed-off-by: Andrei Stan <andreistan2003@gmail.com>
Signed-off-by: Andrei Stan <andreistan2003@gmail.com>
Signed-off-by: Andrei Stan <andreistan2003@gmail.com>
In the case of Xen, on fast applications, the logs would have been called when the machine was already in the exit state. Signed-off-by: Andrei Stan <andreistan2003@gmail.com>
Prerequisite checklist
make fmt
on your commit series before opening this PR;Description of changes
GitHub-Closes: #990