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

[WIP] Automatic handling of guest customizations larger than AHV limit #408

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

thunderboltsid
Copy link
Contributor

If guest customization size is larger than the limit enforced by AHV (32 KB) we will package the guest customization as ISO file, upload the file as an image to the nutanix prism central image service, and mount that image as a CDROM to the machine.

AHV has a limit of 32KB for cloud-init userdata. In Openshift, the
ignition can be rather large (a magnitude over the limit). In
order to support larger userdata files, we allow mounting the customization
as an image.
If guest customization size is larger than the limit enforced by AHV (32 KB)
we will package the guest customization as ISO file, upload the file as
an image to the nutanix prism central image service, and mount that image
as a CDROM to the machine.
Copy link

codecov bot commented Apr 10, 2024

Codecov Report

Attention: Patch coverage is 0% with 113 lines in your changes missing coverage. Please review.

Project coverage is 23.99%. Comparing base (92800d9) to head (a43caf7).
Report is 19 commits behind head on main.

Files Patch % Lines
controllers/nutanixmachine_controller.go 0.00% 69 Missing ⚠️
controllers/helpers.go 0.00% 44 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #408      +/-   ##
==========================================
- Coverage   25.63%   23.99%   -1.64%     
==========================================
  Files          19       19              
  Lines        1303     1392      +89     
==========================================
  Hits          334      334              
- Misses        969     1058      +89     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant