Skip to content

UI: Prevent passing boottype/bootmode when template is deploy-as-is#6151

Merged
nvazquez merged 1 commit into
apache:mainfrom
shapeblue:fix-bootparams-deployasis
Mar 22, 2022
Merged

UI: Prevent passing boottype/bootmode when template is deploy-as-is#6151
nvazquez merged 1 commit into
apache:mainfrom
shapeblue:fix-bootparams-deployasis

Conversation

@Pearl1594
Copy link
Copy Markdown
Contributor

Description

This PR prevents boot type & boot mode being passed for a template is registered as deploy-as-is
image

Types of changes

  • Breaking change (fix or feature that would cause existing functionality to change)
  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Enhancement (improves an existing feature and functionality)
  • Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity

Bug Severity

  • BLOCKER
  • Critical
  • Major
  • Minor
  • Trivial

Screenshots (if appropriate):

How Has This Been Tested?

Successfully deploys a VM from a template registered as deploy-as-is without passing boot mode & type

@Pearl1594
Copy link
Copy Markdown
Contributor Author

@blueorangutan ui

@blueorangutan
Copy link
Copy Markdown

@Pearl1594 a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

@blueorangutan
Copy link
Copy Markdown

UI build: ✔️
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/6151 (SL-JID-1285)

@Pearl1594 Pearl1594 requested a review from utchoang March 22, 2022 07:01
Copy link
Copy Markdown

@utchoang utchoang left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Copy Markdown
Member

@yadvr yadvr left a comment

Choose a reason for hiding this comment

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

LGTM - if the intention is not allow boot type/mode override for deploy-as-is templates for vmware.

@nvazquez nvazquez merged commit f8aca04 into apache:main Mar 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants