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

namespace: do not create path using rand. #279

Merged
merged 1 commit into from
Jun 27, 2018

Conversation

jcvenegas
Copy link
Member

@jcvenegas jcvenegas commented Jun 26, 2018

Using random char generation at early boot may slow the process due to
low entropy.

Also from kernel 4.14.36 rand call hangs due to low-entropy. Lets
remove dependency on rand at early sandbox creation.

Fixes: #278

Signed-off-by: Jose Carlos Venegas Munoz jose.carlos.venegas.munoz@intel.com

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

@nitkon
Copy link
Contributor

nitkon commented Jun 26, 2018

@jcvenegas: Thanks for reporting this issue. I have faced this issue multiple times in the past few days but thought it was some local config issue. Tested your patch on ppc64le. Worked fine.

@jcvenegas
Copy link
Member Author

hit error: kata-containers/tests#361

@codecov
Copy link

codecov bot commented Jun 26, 2018

Codecov Report

Merging #279 into master will decrease coverage by 0.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #279      +/-   ##
==========================================
- Coverage   43.27%   43.25%   -0.02%     
==========================================
  Files          14       14              
  Lines        2244     2240       -4     
==========================================
- Hits          971      969       -2     
+ Misses       1141     1140       -1     
+ Partials      132      131       -1
Impacted Files Coverage Δ
namespace.go 57.14% <100%> (+0.73%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f06eb20...57bf4e6. Read the comment docs.

@jcvenegas jcvenegas force-pushed the dont-call-rand branch 3 times, most recently from 5d56420 to 65a1677 Compare June 26, 2018 20:22
Using random char generation at early boot may slow the process due to
low entropy.

Also from kernel 4.14.36 rand call hangs due to low-entropy. Lets
remove dependency on rand at early sandbox creation.

Fixes: kata-containers#278

Signed-off-by: Jose Carlos Venegas Munoz <jose.carlos.venegas.munoz@intel.com>
@jodh-intel
Copy link
Contributor

jodh-intel commented Jun 27, 2018

Good catch @jcvenegas!

lgtm

I am curious to know why we originally created a randomly-named ns though (which seems perfectly reasonable).

Approved with PullApprove

@sboeuf
Copy link

sboeuf commented Jun 27, 2018

I have retriggered the build again, I really want to merge this asap so that we can move forward with the latest kernel 4.14.51. In the meantime, we have to figure out how to give more entropy to the VM.

@jodh-intel
Copy link
Contributor

@sboeuf sboeuf merged commit 994ac9c into kata-containers:master Jun 27, 2018
caoruidong pushed a commit to caoruidong/runtime that referenced this pull request Sep 14, 2018
Update to kernel to get latest fixes.

Depends-on: github.com/kata-containers/agent#279

Fixes: kata-containers#420

Signed-off-by: Jose Carlos Venegas Munoz <jose.carlos.venegas.munoz@intel.com>
@jcvenegas jcvenegas deleted the dont-call-rand branch January 29, 2019 21:37
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.

5 participants