Skip to content

UI - Zone wizard - Fixes wrong add resource step with localstorageenabled#5303

Merged
yadvr merged 2 commits intoapache:4.15from
utchoang:feature/fix-launch-zone
Aug 18, 2021
Merged

UI - Zone wizard - Fixes wrong add resource step with localstorageenabled#5303
yadvr merged 2 commits intoapache:4.15from
utchoang:feature/fix-launch-zone

Conversation

@utchoang
Copy link

@utchoang utchoang commented Aug 11, 2021

Description

Fixes #5302

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

Feature/Enhancement Scale

  • Major
  • Minor

Bug Severity

  • BLOCKER
  • Critical
  • Major
  • Minor
  • Trivial

Screenshots (if appropriate):

How Has This Been Tested?

@utchoang
Copy link
Author

@blueorangutan ui

@blueorangutan
Copy link

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

@blueorangutan
Copy link

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

Copy link
Contributor

@davidjumani davidjumani left a comment

Choose a reason for hiding this comment

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

@utchoang Some of the steps are the same such as adding hosts / secondary storage. Perhaps there can just be a check on adding primary storage if localstorageenabled and changing the step numbers accordingly
On another note, should the primary storage step be removed or just skippable ? @rhtyd @shwstppr

@utchoang
Copy link
Author

@davidjumani You can refer to https://github.com/apache/cloudstack/blob/4.14/ui/scripts/zoneWizard.js#L4454, In case both localstorageenabled && localstorageenabledforsystemvm are enabled, the step of adding primary storage will be skipped.

@yadvr yadvr added this to the 4.15.2.0 milestone Aug 13, 2021
@davidjumani
Copy link
Contributor

@utchoang That's right, my point is to skip just that step in an if condition rather than adding the redundant code of adding hosts and secondary storage in the if as well as else clause

@utchoang
Copy link
Author

@blueorangutan ui

@blueorangutan
Copy link

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

@utchoang
Copy link
Author

@davidjumani can you please check again?

@blueorangutan
Copy link

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

Copy link
Contributor

@davidjumani davidjumani left a comment

Choose a reason for hiding this comment

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

Verified working! Thanks @utchoang

@yadvr yadvr merged commit 6d98056 into apache:4.15 Aug 18, 2021
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.

4 participants