Skip to content
This repository has been archived by the owner on Sep 24, 2021. It is now read-only.

Enable unikernel runtime log #226

Merged
merged 2 commits into from Sep 12, 2017

Conversation

Crazykev
Copy link
Contributor

@Crazykev Crazykev commented Sep 4, 2017

  1. enable basic unikernel log, we still need vm wrapper to reformat log from qemu.
  2. enable vm ACPI, if not, it will hang at start time.
  3. fix domain copy in ListDomains
  4. enable basic libvirt bridge network for debug purpose.

/cc @feiskyer @resouer PTAL

Signed-off-by: Crazykev <crazykev@harmonycloud.cn>
Signed-off-by: Crazykev <crazykev@harmonycloud.cn>
@k8s-ci-robot
Copy link

@Crazykev: GitHub didn't allow me to request PR reviews from the following users: PTAL.

Note that only kubernetes members can review this PR, and authors cannot review their own PRs.

In response to this:

  1. enable basic unikernel log, we still need vm wrapper to reformat log from qemu.
  2. enable vm ACPI, if not, it will hang at start time.
  3. fix domain copy in ListDomains
  4. enable basic libvirt bridge network for debug purpose.

/cc @feiskyer @resouer PTAL

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Sep 4, 2017
@k8s-reviewable
Copy link

This change is Reviewable

@k8s-ci-robot k8s-ci-robot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Sep 4, 2017
@feiskyer feiskyer merged commit d676192 into kubernetes-retired:master Sep 12, 2017
@Crazykev Crazykev deleted the unikernel-log branch September 12, 2017 09:12
@feiskyer feiskyer mentioned this pull request Sep 27, 2017
3 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants