Skip to content

UI bug fix: Cannot deploy VM from ISO - #3995

Merged
andrijapanicsb merged 1 commit into
apache: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 into
apache: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

yadvr commented Mar 30, 2020

Copy link
Copy Markdown
Member

@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

nvazquez commented Mar 30, 2020

Copy link
Copy Markdown
Contributor

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

@andrijapanicsb andrijapanicsb left a comment

Copy link
Copy Markdown
Contributor

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