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

BaseEC2NodeDriver.ex_register_image: support additional parameters #1906

Merged
merged 2 commits into from Aug 1, 2023

Conversation

rvandegrift
Copy link
Contributor

Description

Add support for more EC2 RegisterImage parameters:

  • boot_mode: control bios vs. uefi boot.
  • tpm_support: enable a tpm.
  • uefi_data: provide NV UEFI vars.
  • imds_support: require IMDSv2.

I haven't added tests, as these are just passed through. The parameter names are taken from the ec2 api doc.

I have tested it by creating an image that sets BootMode to a non-default value.

Status

done, ready for review

Checklist (tick everything that applies)

  • Code linting (required, can be done after the PR checks)
    • my change is blackened, but unrelated things in the file are not.
  • Documentation
  • Tests
  • ICLA (required for bigger changes)

Add support for more EC2 RegisterImage parameters:
- boot_mode: control bios vs. uefi boot.
- tpm_support: enable a tpm.
- uefi_data: provide NV UEFI vars.
- imds_support: require IMDSv2.
Copy link
Member

@Kami Kami left a comment

Choose a reason for hiding this comment

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

Change looks good to me, but some unit tests would be good.

asfgit pushed a commit that referenced this pull request Aug 1, 2023
@asfgit asfgit merged commit 0f86c0b into apache:trunk Aug 1, 2023
20 of 21 checks passed
@Kami
Copy link
Member

Kami commented Aug 1, 2023

Merged into trunk. Thanks.

asfgit pushed a commit that referenced this pull request Aug 1, 2023
@Kami Kami added this to the v3.8.0 milestone Aug 6, 2023
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.

None yet

3 participants