Skip to content

UI bug fix: Cannot deploy VM from ISO#3995

Merged
andrijapanicsb merged 1 commit intoapache:4.13from
ustcweizhou:4.13-ui-fix-deploy-from-iso
Mar 30, 2020
Merged

UI bug fix: Cannot deploy VM from ISO#3995
andrijapanicsb merged 1 commit intoapache:4.13from
ustcweizhou:4.13-ui-fix-deploy-from-iso

Conversation

@ustcweizhou
Copy link
Copy Markdown
Contributor

Description

While deploy a vm from ISO on UI, the instance wizard is not working in vm overview step.

this is regression issue of #3271

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)

Screenshots (if appropriate):

How Has This Been Tested?

@yadvr
Copy link
Copy Markdown
Member

yadvr commented Mar 30, 2020

@andrijapanicsb
Copy link
Copy Markdown
Contributor

image

indeed... wow

@andrijapanicsb
Copy link
Copy Markdown
Contributor

@nvazquez can you also take a look - the issue is valid, and I would like to make sure the OVF stuff is not broken = then merge it. thx

@nvazquez
Copy link
Copy Markdown
Contributor

nvazquez commented Mar 30, 2020

The workaround is clicking on Next once this appears. I'll work on a fix

@andrijapanicsb
Copy link
Copy Markdown
Contributor

the fix is there @nvazquez ?

@andrijapanicsb
Copy link
Copy Markdown
Contributor

@blueorangutan package

@andrijapanicsb
Copy link
Copy Markdown
Contributor

(having some nasty JS overrides in my local lab, so need to build from packages - less effort)

@andrijapanicsb
Copy link
Copy Markdown
Contributor

@blueorangutan package

@blueorangutan
Copy link
Copy Markdown

@andrijapanicsb a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

@nvazquez
Copy link
Copy Markdown
Contributor

Yes @andrijapanicsb, thanks @ustcweizhou I haven't seen the fix sorry

@blueorangutan
Copy link
Copy Markdown

Packaging result: ✔centos7 ✔debian. JID-1107

@andrijapanicsb andrijapanicsb self-requested a review March 30, 2020 18:26
Copy link
Copy Markdown
Contributor

@andrijapanicsb andrijapanicsb left a comment

Choose a reason for hiding this comment

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

LGTM.

Tested by upgrading 4.13 to the PR code (4.13.1) and also verified OVF template properties are passed properly to vCenter.

@andrijapanicsb
Copy link
Copy Markdown
Contributor

2 x LGTM, manual testing. Regression testing not needed since a simple JS change/not applicable.

Merging.

@andrijapanicsb
Copy link
Copy Markdown
Contributor

Thanks for noticing this @weizhouapache

@andrijapanicsb andrijapanicsb merged commit 95f58db into apache:4.13 Mar 30, 2020
DaanHoogland pushed a commit that referenced this pull request Mar 30, 2020
* 4.13:
  UI bug fix: Cannot deploy VM from ISO (#3995)
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.

6 participants