Skip to content

Conversation

@nvazquez
Copy link
Member

@nvazquez nvazquez commented Mar 13, 2019

Description

Problem: Users can register ISOs from URL but cannot upload local ISOs.

Root cause: CloudStack provides browser-based upload support for volumes and templates, but ISOs are not supported.

Solution:
The existing browser-based upload from local functionality for templates and volumes (https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=39620237) is extended to support uploading local ISOs.

  • Extend the UI: A new button is created under the ISOs view: 'Upload from Local'. A new dialog form is displayed in which the user must select the ISO to upload from its local file system.
  • Extend the API: New 'GetUploadParamsForIso' API command is created to handle the ISO upload.

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):

image
image

How Has This Been Tested?

Templates tab -> ISOs -> Upload from local -> Select file on local file system

@nvazquez nvazquez self-assigned this Mar 13, 2019
@yadvr
Copy link
Member

yadvr commented Mar 29, 2019

@nvazquez please send this to apache/cloudstack when you've time. Thanks.

@nvazquez nvazquez force-pushed the addlocalisoupload branch from eff96dd to 24750aa Compare April 1, 2019 11:39
@nvazquez
Copy link
Member Author

nvazquez commented Apr 1, 2019

Closing PR, moved to apache/master: apache/#3251

@nvazquez nvazquez closed this Apr 1, 2019
yadvr added a commit that referenced this pull request Jan 20, 2021
This fixes #35
This fixes #51
This closes #78

This adds the following:
- Download iso, template, volume handler to show link
- Show VM password on VM start and upon reset password action

Signed-off-by: Rohit Yadav <[email protected]>
shwstppr added a commit that referenced this pull request Aug 16, 2021
Signed-off-by: Abhishek Kumar <[email protected]>

ui: fix UI label bug #28 (#34)

Signed-off-by: Abhishek Kumar <[email protected]>

fix system vm migration with zone wide storage (#35)

* fix system vm migration with zone wide storage

Signed-off-by: Abhishek Kumar <[email protected]>

* fix

Signed-off-by: Abhishek Kumar <[email protected]>

* fix spacing

Signed-off-by: Abhishek Kumar <[email protected]>

* add empty volumes check

Signed-off-by: Abhishek Kumar <[email protected]>

fix fr568 issue #28 (#36)

Fixes systemvm migration with zone-wide storage
Check only for usable volumes

Signed-off-by: Abhishek Kumar <[email protected]>

vmware: fix error msg for vmotion without shared storage (#37)

Fixes error message which wrongly mentions worker VM

Signed-off-by: Abhishek Kumar <[email protected]>

refactor error message for interpod systemvm migration (#38)

Fixes error message introduced in #28

Signed-off-by: Abhishek Kumar <[email protected]>
shwstppr added a commit that referenced this pull request Nov 23, 2021
Signed-off-by: Abhishek Kumar <[email protected]>

ui: fix UI label bug #28 (#34)

Signed-off-by: Abhishek Kumar <[email protected]>

fix system vm migration with zone wide storage (#35)

* fix system vm migration with zone wide storage

Signed-off-by: Abhishek Kumar <[email protected]>

* fix

Signed-off-by: Abhishek Kumar <[email protected]>

* fix spacing

Signed-off-by: Abhishek Kumar <[email protected]>

* add empty volumes check

Signed-off-by: Abhishek Kumar <[email protected]>

fix fr568 issue #28 (#36)

Fixes systemvm migration with zone-wide storage
Check only for usable volumes

Signed-off-by: Abhishek Kumar <[email protected]>

vmware: fix error msg for vmotion without shared storage (#37)

Fixes error message which wrongly mentions worker VM

Signed-off-by: Abhishek Kumar <[email protected]>

refactor error message for interpod systemvm migration (#38)

Fixes error message introduced in #28

Signed-off-by: Abhishek Kumar <[email protected]>
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.

3 participants