-
Notifications
You must be signed in to change notification settings - Fork 33
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
We need to check memory requirements #53
Comments
I think that once we find out values by hand then set of CI tests needs to be added. |
is thet memroy needed including the qemu overhead or is that just the ram allocated to the instnace. i was not expecting to see the guest ram requiremetns change with a change in bootloader in the openstack ci we had been useing 64mb for ram for the older cirros images but we do have space to increase that. im just surpised to see uefi having an effect. any idea why? is this due to uefi runtime service consuming ram or is this deivce memory? |
For CirrOS 0.6 instance memory requirements look like that:
Notes:
|
Just a comment on this. That is because cirros puts all modules listed into initramfs. Other initramfs solutions only put modules in their initramfs that are needed to reach the "real root", and then loads those modules from lib/modules as necessary. So, memory footprint for cirros goes up because:
long term solution for this is:
|
One of solutions could be drop of initramfs (or most of it) and provide disk image as main artifact. BTW - are there any stats on which files are downloaded? |
Artifacts by GitHub download stats:
Looks like fetching kernel/initramfs files is not popular - people mostly want disk images. |
Code used: #!/usr/bin/python3
import requests
r = requests.get('https://api.github.com/repos/cirros-dev/cirros/releases')
for release in r.json():
for artifact in release["assets"]:
print(f"{artifact['download_count']:9}\t{artifact['name']}") |
just using the disk image doesn't solve anything. i use the initramfs for a lot of things. they're way useful for testing. |
This should be suitable for use with CirrOS [1] and other minimal test OSes used by KubeVirt CI. [1] cirros-dev/cirros#53 Signed-off-by: Lee Yarwood <[email protected]>
This should be suitable for use with CirrOS [1] and other minimal test OSes used by KubeVirt CI. [1] cirros-dev/cirros#53 Signed-off-by: Lee Yarwood <[email protected]>
This should be suitable for use with CirrOS [1] and other minimal test OSes used by KubeVirt CI. [1] cirros-dev/cirros#53 Signed-off-by: Lee Yarwood <[email protected]>
This should be suitable for use with CirrOS [1] and other minimal test OSes used by KubeVirt CI. [1] cirros-dev/cirros#53 Signed-off-by: Lee Yarwood <[email protected]> Signed-off-by: Lee Yarwood <[email protected]>
On CI we boot with 512MB of memory so everything works.
I did some minimal testing and got interesting results:
The text was updated successfully, but these errors were encountered: