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

kata-env: Fix display of debug options #738

Conversation

jodh-intel
Copy link
Contributor

The runtime and hypervisor Debug options were always showing as
false (although all debug options in configuration.toml were
correctly honoured).

Note: Also moved location of FactoryConfig in RuntimeConfig as the
malign linter was complaining:

virtcontainers/pkg/oci/utils.go:102:20:warning: struct of size 408 could be 400 (maligned)

Original-master-commit: https://github.com/kata-containers/runtime/commits/23a35c84c92e6ce73366edc575ae875ba4d77473

Fixes #724.

Signed-off-by: James O. D. Hunt [email protected]

The runtime and hypervisor `Debug` options were always showing as
`false` (although all debug options in `configuration.toml` were
correctly honoured).

Note: Also moved location of `FactoryConfig` in `RuntimeConfig` as the
`malign` linter was complaining:

```
virtcontainers/pkg/oci/utils.go:102:20:warning: struct of size 408 could be 400 (maligned)
```

Original-master-commit: https://github.com/kata-containers/runtime/commits/23a35c84c92e6ce73366edc575ae875ba4d77473

Fixes kata-containers#724.

Signed-off-by: James O. D. Hunt <[email protected]>
@WeiZhang555
Copy link
Member

WeiZhang555 commented Sep 17, 2018

LGTM
CI leaves again

Approved with PullApprove

@katacontainersbot
Copy link
Contributor

PSS Measurement:
Qemu: 165632 KB
Proxy: 4048 KB
Shim: 8905 KB

Memory inside container:
Total Memory: 2043460 KB
Free Memory: 2003876 KB

@codecov
Copy link

codecov bot commented Sep 17, 2018

Codecov Report

❗ No coverage uploaded for pull request base (stable-1.2@3cdc501). Click here to learn what that means.
The diff coverage is 100%.

@@              Coverage Diff              @@
##             stable-1.2     #738   +/-   ##
=============================================
  Coverage              ?   66.94%           
=============================================
  Files                 ?       85           
  Lines                 ?     9166           
  Branches              ?        0           
=============================================
  Hits                  ?     6136           
  Misses                ?     2393           
  Partials              ?      637

Copy link
Contributor

@grahamwhaley grahamwhaley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@grahamwhaley grahamwhaley merged commit 626fb77 into kata-containers:stable-1.2 Sep 18, 2018
egernst pushed a commit to egernst/runtime that referenced this pull request Feb 9, 2021
mount: Use "virtiofs" instead of "virtio_fs"
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants