Skip to content

ui: Show diskoffering for create volume from ROOT volume snaps#4912

Merged
yadvr merged 1 commit into
apache:4.15from
shapeblue:fix-snap-vol
Apr 16, 2021
Merged

ui: Show diskoffering for create volume from ROOT volume snaps#4912
yadvr merged 1 commit into
apache:4.15from
shapeblue:fix-snap-vol

Conversation

@davidjumani

Copy link
Copy Markdown
Contributor

Description

Fixes the issue of not being able to select the diskofferingid while creating volumes from snapshots of ROOT volumes

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

Screenshots (if appropriate):

Screenshot from 2021-04-12 16-44-30

@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.

dataView: true,
show: (record) => { return record.state === 'BackedUp' },
args: ['snapshotid', 'name'],
args: (record, store) => {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

do we need store on the definition?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Probably not necessary

@blueorangutan

Copy link
Copy Markdown

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

@yadvr

yadvr commented Apr 12, 2021

Copy link
Copy Markdown
Member

@davidjumani is that for master branch only, or is it a bugfix that 4.15 can have as well?

@shwstppr shwstppr changed the base branch from master to 4.15 April 13, 2021 12:35
@shwstppr shwstppr added this to the 4.15.1.0 milestone Apr 13, 2021
@shwstppr

Copy link
Copy Markdown
Contributor

@davidjumani changed the base branch. Please update if wrong cc @rhtyd

@yadvr yadvr requested review from Pearl1594 and shwstppr April 15, 2021 07:27

@Pearl1594 Pearl1594 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

@shwstppr shwstppr 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

@yadvr yadvr merged commit 90fc5ac into apache:4.15 Apr 16, 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.

7 participants