Skip to content

Conversation

@hustliyilin
Copy link
Contributor

@hustliyilin hustliyilin commented May 11, 2023

Obviously, the codes of the agent are located in the rust-keylime repository. However, there is no logic for cloning and compiling rust-keylime in the corresponding rust dockerfile. So we need to add it.

Fixes: #585
Signed-off-by: YiLin.Li YiLin.Li@linux.alibaba.com

@hustliyilin
Copy link
Contributor Author

hustliyilin commented May 11, 2023

Thanks @ansasaki . After adding the cargo clean command, the docker image size reduces about 1.6GB.

@codecov
Copy link

codecov bot commented May 22, 2023

Codecov Report

Merging #584 (fe20d57) into master (4c8d575) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files
Flag Coverage Δ
e2e-testsuite 58.90% <ø> (ø)
upstream-unit-tests 58.56% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

…ompiling rust-keylime

Obviously, the codes of the agent are located in the rust-keylime repository. However, there
is no logic for cloning and compiling rust-keylime in the corresponding rust dockerfile. So
we need to add it.

Signed-off-by: YiLin.Li <YiLin.Li@linux.alibaba.com>
@hustliyilin
Copy link
Contributor Author

Thanks @ansasaki . I fixed it and force-pushed the codes. Could you review it again?

@ansasaki ansasaki merged commit 1f32de1 into keylime:master May 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docker/fedora/keylime_rust.Dockerfile: missing the logic of cloning and compiling rust-keylime

5 participants