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

runtime : delete redundant code in CreateContainer#1741

Merged
caoruidong merged 1 commit intokata-containers:masterfrom
jshachm:delete-code
May 27, 2019
Merged

runtime : delete redundant code in CreateContainer#1741
caoruidong merged 1 commit intokata-containers:masterfrom
jshachm:delete-code

Conversation

@jshachm
Copy link
Copy Markdown
Member

@jshachm jshachm commented May 26, 2019

Here we have done with logger and container ID map
Just delete these code.
fixes #1740

Signed-off-by: Haomin Tsai caihaomin@huawei.com

Here we have done with logger and container ID map
Just delete these code.
fixes kata-containers#1740

Signed-off-by: Haomin Tsai <caihaomin@huawei.com>
@jshachm
Copy link
Copy Markdown
Member Author

jshachm commented May 27, 2019

/test

@jshachm jshachm requested review from jodh-intel and lifupan and removed request for lifupan May 27, 2019 02:06
@jshachm
Copy link
Copy Markdown
Member Author

jshachm commented May 27, 2019

ping @lifupan @jodh-intel

@codecov
Copy link
Copy Markdown

codecov bot commented May 27, 2019

Codecov Report

Merging #1741 into master will decrease coverage by 1.51%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1741      +/-   ##
==========================================
- Coverage   56.09%   54.57%   -1.52%     
==========================================
  Files          43      106      +63     
  Lines        4200    12767    +8567     
==========================================
+ Hits         2356     6968    +4612     
- Misses       1661     4967    +3306     
- Partials      183      832     +649

Copy link
Copy Markdown
Member

@bergwolf bergwolf left a comment

Choose a reason for hiding this comment

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

Looks like some merge fuzz when applying patches by hand. Nice catch!

@caoruidong caoruidong merged commit 43c2796 into kata-containers:master May 27, 2019
@jshachm jshachm deleted the delete-code branch May 27, 2019 08:21
This was referenced Jun 3, 2019
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.

Some redundant code in CreateContainer

5 participants