Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

chore: Remove support for deprecated images configuration attribute in favor of image_files attribute in the export config block #346

Merged
merged 1 commit into from
Dec 14, 2023

Conversation

tenthirtyam
Copy link
Collaborator

Summary

  • Removes deprecated images option that was noted to be removed over 2 years ago.
  • Updated export configuration descriptions.
  • Simplified manifest validation.
  • Updated to use ui.Say versus ui.Message for output consistency.
  • Added code comments.
  • Improved output and error messaging for output consistency.

Tests

...
==> vsphere-iso.linux-photon: Executing shutdown command...
==> vsphere-iso.linux-photon: Deleting Floppy drives...
==> vsphere-iso.linux-photon: Eject CD-ROM drives...
==> vsphere-iso.linux-photon: Deleting CD-ROM drives...
==> vsphere-iso.linux-photon: Exporting to Open Virtualization Format (OVF)...
==> vsphere-iso.linux-photon: Downloading linux-photon-4.0-develop-disk-0.vmdk...
==> vsphere-iso.linux-photon: Exporting linux-photon-4.0-develop-disk-0.vmdk...
==> vsphere-iso.linux-photon: Writing OVF descriptor linux-photon-4.0-develop.ovf...
==> vsphere-iso.linux-photon: Creating SHA1 manifest linux-photon-4.0-develop.mf...
==> vsphere-iso.linux-photon: Completed export to Open Virtualization Format (OVF).
    vsphere-iso.linux-photon: Closing sessions ....
Build 'vsphere-iso.linux-photon' finished after 7 minutes 28 seconds.
...

- Removes deprecated `images` option that was noted to be removed over 2 years ago.
- Updated export configuration descriptions.
- Simplified `manifest` validation.
- Updated to use `ui.Say` versus `ui.Message` for output consistency.
- Added code comments.
- Improved output and error messaging for output consistency.

Signed-off-by: Ryan Johnson <ryan.johnson@broadcom.com>
@tenthirtyam tenthirtyam force-pushed the chore/update-export-step branch from c83cb90 to 1fd59e3 Compare December 12, 2023 16:18
Copy link
Contributor

@nywilken nywilken left a comment

Choose a reason for hiding this comment

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

This looks good. Thanks for this change.

@nywilken nywilken changed the title chore: update export step chore: Remove support for deprecated images configuration attribute in favor of image_files attribute in the export config block Dec 14, 2023
@nywilken nywilken merged commit 4c8910c into main Dec 14, 2023
11 checks passed
@nywilken nywilken deleted the chore/update-export-step branch December 14, 2023 20:26
@hashicorp hashicorp locked and limited conversation to collaborators Jun 29, 2024
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants