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

revert: agent: sandbox_pause should get arguments from proc #621

Merged
merged 1 commit into from
Aug 2, 2019

Conversation

devimc
Copy link

@devimc devimc commented Aug 1, 2019

When agent runs as init, it is not able to read /proc/self/cmdline because
of /proc has not been mounted when the C constructor is called.

This reverts commit cfbd8c9.

fixes #620

Signed-off-by: Julio Montes [email protected]

@devimc
Copy link
Author

devimc commented Aug 1, 2019

/test

When agent runs as init, it is not able to read `/proc/self/cmdline` because
of `/proc` has not been mounted when the C constructor is called.

This reverts commit cfbd8c9.

fixes kata-containers#620

Signed-off-by: Julio Montes <[email protected]>
@devimc
Copy link
Author

devimc commented Aug 1, 2019

/test

@amshinde
Copy link
Member

amshinde commented Aug 1, 2019

Lets wait for @bergwolf 's input before this is merged.

@codecov
Copy link

codecov bot commented Aug 1, 2019

Codecov Report

Merging #621 into master will increase coverage by 0.08%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #621      +/-   ##
==========================================
+ Coverage   61.14%   61.22%   +0.08%     
==========================================
  Files          17       17              
  Lines        2486     2486              
==========================================
+ Hits         1520     1522       +2     
+ Misses        820      819       -1     
+ Partials      146      145       -1

@raravena80
Copy link
Member

Looks ready to merge

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.

agent: failed to open proc: No such file or directory
5 participants