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

cli: fix the issue of using wrong path to get version #869

Merged
merged 1 commit into from
Oct 30, 2018

Conversation

lifupan
Copy link
Member

@lifupan lifupan commented Oct 30, 2018

Both of the netmon and proxy should use the right path
figured out from the configure instead of the default settings
to get their versions.

Fixes: #868

Signed-off-by: Fupan Li [email protected]

lifupan added a commit to lifupan/kata-runtime that referenced this pull request Oct 30, 2018
Refactor the config related codes into a separated
package which can be shared with other cli programs
such as kata's shimv2.

Depends-on: github.com/kata-containers#869

Fixes: kata-containers#787
Fixes: kata-containers#714

Signed-off-by: fupan <[email protected]>
@marcov
Copy link
Contributor

marcov commented Oct 30, 2018

lgtm
/test

lifupan added a commit to lifupan/kata-runtime that referenced this pull request Oct 30, 2018
Refactor the config related codes into a separated
package which can be shared with other cli programs
such as kata's shimv2.

Fixes: kata-containers#787
Fixes: kata-containers#714

Depends-on:github.com/kata-containers#869

Signed-off-by: fupan <[email protected]>
Both of the netmon and proxy should use the right path
figured out from the configure instead of the default settings
to get their versions.

Fixes: kata-containers#868

Signed-off-by: Fupan Li <[email protected]>
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
/cc @jodh-intel

lifupan added a commit to lifupan/kata-runtime that referenced this pull request Oct 30, 2018
Refactor the config related codes into a separated
package which can be shared with other cli programs
such as kata's shimv2.

Fixes: kata-containers#787
Fixes: kata-containers#714

Depends-on:github.com/kata-containers#869

Signed-off-by: fupan <[email protected]>
@lifupan
Copy link
Member Author

lifupan commented Oct 30, 2018

/test

Copy link
Member

@amshinde amshinde left a comment

Choose a reason for hiding this comment

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

lgtm

@amshinde
Copy link
Member

/retest

@sboeuf sboeuf merged commit 2cae9c3 into kata-containers:master Oct 30, 2018
@lifupan lifupan deleted the fix_path branch October 31, 2018 01:52
egernst pushed a commit to egernst/runtime that referenced this pull request Feb 9, 2021
This PR enables the github actions for agent repository.

Fixes kata-containers#869

Signed-off-by: Gabriela Cervantes <[email protected]>
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.

6 participants