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: add url check before download #877

Conversation

burnsjared0415
Copy link
Contributor

@burnsjared0415 burnsjared0415 commented Apr 15, 2024

Summary of Pull Request

Added a check to verify a download URL before starting the download of an ISO.

Type of Pull Request

  • This is a bugfix. type/bug
  • This is an enhancement or feature. type/feature or type/enhancement
  • This is a documentation update. type/docs
  • This is a refactoring update. type/refactor
  • This is a chore. type/chore
  • This is something else.
    Please describe:

Related to Existing Issues

Issue Number: N/A

Test and Documentation Coverage

  • Tests have been completed.
  • Documentation has been added or updated.

Breaking Changes?

  • Yes, there are breaking changes.
  • No, there are no breaking changes.

@burnsjared0415 burnsjared0415 requested a review from a team as a code owner April 15, 2024 15:08
@vmwclabot vmwclabot added the cla-not-required Contributor License Agreement Not Required label Apr 15, 2024
@tenthirtyam tenthirtyam requested review from tenthirtyam and removed request for GaryJBlake April 15, 2024 15:11
@tenthirtyam tenthirtyam changed the title Feat/add url for file download feat: add url check Apr 15, 2024
@tenthirtyam tenthirtyam added this to the v0.20.0 milestone Apr 15, 2024
@tenthirtyam tenthirtyam changed the title feat: add url check feat: add url check before download Apr 15, 2024
@tenthirtyam tenthirtyam force-pushed the feat/add-url-for-file-download branch from f607707 to 2529969 Compare April 15, 2024 15:14
Copy link
Contributor

@tenthirtyam tenthirtyam left a comment

Choose a reason for hiding this comment

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

Minor nit and suggestion.

@tenthirtyam tenthirtyam added the type/enhancement Enhancement label Apr 15, 2024
@tenthirtyam tenthirtyam force-pushed the feat/add-url-for-file-download branch 2 times, most recently from f8ede74 to c783656 Compare April 15, 2024 15:57
tenthirtyam and others added 4 commits April 15, 2024 12:09
Removes the PowerShell provisioner for Windows 11 and 10 as it's not required currently after the transition to Ansible.

Signed-off-by: Ryan Johnson <ryan.johnson@broadcom.com>
Adds a script (`./download.sh`) and a configuration (`project.json`) to assist in the download of the guest operating systems.

Signed-off-by: Jared Burns <jared.burns@broadcom.com>
Co-authored-by: Ryan Johnson <ryan.johnson@broadcom.com>
added ISO check before starting download. this will prevent download from starting if iso exist.

Signed-off-by: Jared Burns <jared.burns@broadcom.com>
Added a check to verify a download URL before starting the download of an ISO.

Signed-off-by: Jared Burns <jared.burns@broadcom.com>
@tenthirtyam tenthirtyam force-pushed the feat/add-url-for-file-download branch from c783656 to d4bb321 Compare April 15, 2024 16:19
Signed-off-by: Ryan Johnson <ryan.johnson@broadcom.com>
@tenthirtyam tenthirtyam merged commit 669a4a5 into vmware-samples:develop Apr 15, 2024
Copy link

I'm going to lock this pull request because it has been closed for 30 days. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 16, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla-not-required Contributor License Agreement Not Required type/enhancement Enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants