Skip to content

Conversation

fidencio
Copy link
Member

@fidencio fidencio commented Apr 3, 2025

For those not interested in CoCo, let's at least allow them to easily build the agent without the guest-pull feature.

This reduces the binary size (already stripped) from 25M to 18M.

@katacontainersbot katacontainersbot added the size/tiny Smallest and simplest task label Apr 3, 2025
@fidencio fidencio force-pushed the topic/allow-building-the-agent-without-enabling-guest-pull branch from 7d9247a to e1d030f Compare April 4, 2025 07:18
Copy link
Member

@stevenhorsman stevenhorsman left a comment

Choose a reason for hiding this comment

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

LGTM.

@stevenhorsman stevenhorsman self-requested a review April 4, 2025 11:54
Copy link
Member

@stevenhorsman stevenhorsman left a comment

Choose a reason for hiding this comment

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

We can think about whether we want to add the guest-pull option do static checks in a later PR. Thanks.

Copy link
Member

@danmihai1 danmihai1 left a comment

Choose a reason for hiding this comment

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

Thank you @fidencio !

@@ -100,7 +100,7 @@ lto = true

[features]
# The default-pull feature would support all pull types, including sharing images by virtio-fs and pulling images in the guest
Copy link
Contributor

Choose a reason for hiding this comment

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

Need to change this comment as well

Copy link
Member Author

Choose a reason for hiding this comment

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

Done!

For those not interested in CoCo, let's at least allow them to easily
build the agent without the guest-pull feature.

This reduces the binary size (already stripped) from 25M to 18M.

Signed-off-by: Fabiano Fidêncio <fabiano@fidencio.org>
@fidencio fidencio force-pushed the topic/allow-building-the-agent-without-enabling-guest-pull branch from e1d030f to e3c98a5 Compare April 4, 2025 21:59
@fidencio fidencio merged commit c75ea25 into kata-containers:main Apr 6, 2025
1181 of 1220 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ok-to-test size/tiny Smallest and simplest task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants