Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

config: Add max_concurrent_download for pull operation #153

Merged
merged 1 commit into from
May 25, 2023
Merged

config: Add max_concurrent_download for pull operation #153

merged 1 commit into from
May 25, 2023

Conversation

arronwy
Copy link
Member

@arronwy arronwy commented May 25, 2023

std::thread::available_parallelism() will get all available cpu num in host side for enclave-cc, export the config for enclave/kata agent to decide.

Thanks @mythi for point this out.

DEFAULT_MAX_CONCURRENT_DOWNLOAD is set to 3 which is the same default config for dockerd.

std::thread::available_parallelism() will get available cpu num
in host side for enclave-cc, export the config for enclave/kata
agent to decide.

DEFAULT_MAX_CONCURRENT_DOWNLOAD is set to 3 which is the same
default config for dockerd.

Signed-off-by: Wang, Arron <arron.wang@intel.com>
Copy link
Contributor

@mythi mythi left a comment

Choose a reason for hiding this comment

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

Looks good to me and tested with:

-image-rs = { git = "https://github.com/confidential-containers/image-rs.git", default-features = false, rev = "44f27c2" }
+image-rs = { git = "https://github.com/arronwy/image-rs.git", default-features = false, branch = "config" }

in enclave-cc and it helps to fix an image pull that was failing earlier.

@arronwy arronwy merged commit a485b48 into confidential-containers:main May 25, 2023
5 checks passed
dcmiddle pushed a commit to dcmiddle/guest-components that referenced this pull request Jun 5, 2023
This commit fixes KEK registration for keyprovider to
KBS. Including address parse, key generation and indexing

Close confidential-containers#153

Signed-off-by: Xynnn007 <xynnn@linux.alibaba.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants