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

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

Merged
merged 1 commit into from
May 24, 2023

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 [email protected]

@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.

docker/fedora/keylime_rust.Dockerfile Outdated Show resolved Hide resolved
@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 <[email protected]>
@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