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

feat(fs-gen, vmm): disable compression #50

Merged
merged 1 commit into from
May 30, 2024

Conversation

Mathias-Boulay
Copy link
Contributor

What's in it?

This PR brings the ability to disable compression on the final image, via the --no-compression arg.

Additionally, it enables the usage of this argument by vmm.

Why this PR?

Compression significantly increases startup times. This isn't desired behavior.

@mfernd
Copy link
Contributor

mfernd commented May 30, 2024

Example when using normal compression level:

...
[    2.477661] IP-Config: Complete:
[    2.479763]      device=eth0, hwaddr=b6:93:57:01:b9:92, ipaddr=172.29.0.2, mask=255.255.0.0, gw=172.29.0.1
[    2.486307]      host=172.29.0.2, domain=, nis-domain=(none)
[    2.490389]      bootserver=255.255.255.255, rootserver=255.255.255.255, rootpath=
[    2.490391]      nameserver0=1.1.1.1
2024-05-30T16:14:07.505812Z ERROR vmm::grpc::client: Failed to connect to Agent service: transport error    
2024-05-30T16:14:07.505883Z ERROR vmm::grpc::client: Retrying in 2s...    
2024-05-30T16:14:09.507168Z ERROR vmm::grpc::client: Failed to connect to Agent service: transport error    
2024-05-30T16:14:09.507210Z ERROR vmm::grpc::client: Retrying in 2s...    
2024-05-30T16:14:11.509261Z ERROR vmm::grpc::client: Failed to connect to Agent service: transport error    
2024-05-30T16:14:11.509302Z ERROR vmm::grpc::client: Retrying in 2s...    
2024-05-30T16:14:13.511420Z ERROR vmm::grpc::client: Failed to connect to Agent service: transport error    
2024-05-30T16:14:13.511474Z ERROR vmm::grpc::client: Retrying in 2s...
...
[   19.803169] Run /init as init process
listening on http://0.0.0.0:50051/
2024-05-30T16:14:25.525841Z  INFO vmm::grpc::server: Successfully connected to Agent service
...

And we have almost no waiting time now, but we have very large images now (150MB to ~1GB).

Copy link
Contributor

@mfernd mfernd left a comment

Choose a reason for hiding this comment

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

lgtm 👍

@mfernd mfernd merged commit 9ab2b92 into virt-do:main May 30, 2024
2 checks passed
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.

2 participants