Skip to content

ui: Refresh page on deployvm result#5393

Merged
yadvr merged 1 commit into
apache:4.15from
shapeblue:fix-deployvm
Sep 1, 2021
Merged

ui: Refresh page on deployvm result#5393
yadvr merged 1 commit into
apache:4.15from
shapeblue:fix-deployvm

Conversation

@davidjumani
Copy link
Copy Markdown
Contributor

@davidjumani davidjumani commented Aug 31, 2021

Description

Fixes #5389

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)

Bug Severity

  • BLOCKER
  • Critical
  • Major
  • Minor
  • Trivial

@davidjumani
Copy link
Copy Markdown
Contributor Author

@blueorangutan ui

@blueorangutan
Copy link
Copy Markdown

@davidjumani 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/5393 (SL-JID-581)

@davidjumani davidjumani force-pushed the fix-deployvm branch 2 times, most recently from 2e50195 to 1354959 Compare August 31, 2021 11:01
@davidjumani davidjumani changed the base branch from main to 4.15 August 31, 2021 11:02
@davidjumani
Copy link
Copy Markdown
Contributor Author

@blueorangutan ui

@blueorangutan
Copy link
Copy Markdown

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

@davidjumani davidjumani added this to the 4.15.2.0 milestone Aug 31, 2021
@blueorangutan
Copy link
Copy Markdown

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

@davidjumani
Copy link
Copy Markdown
Contributor Author

@blueorangutan ui

@blueorangutan
Copy link
Copy Markdown

@davidjumani 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/5393 (SL-JID-583)

Copy link
Copy Markdown
Contributor

@nvazquez nvazquez 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 fine

@nvazquez
Copy link
Copy Markdown
Contributor

@davidjumani is there anything else remaining on this PR?

@davidjumani davidjumani marked this pull request as ready for review September 1, 2021 04:20
@davidjumani
Copy link
Copy Markdown
Contributor Author

It's ready @nvazquez

Copy link
Copy Markdown
Contributor

@shwstppr shwstppr left a comment

Choose a reason for hiding this comment

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

LGTM. Works fine

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!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

VM machine state transition from 'Starting' to 'Running' is not auto-updated in the Instances screen

6 participants