FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add local ISO upload via UI by nvazquez · Pull Request #35 · shapeblue/cloudstack · GitHub

Repository navigation

Add local ISO upload via UI - #35

Closed
nvazquez wants to merge 1 commit into
masterfrom
addlocalisoupload
Closed

nvazquez wants to merge 1 commit into
masterfrom
addlocalisoupload

Conversation

nvazquez commented Mar 13, 2019 •
edited
Loading

Copy link
Copy Markdown
Member

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


How Has This Been Tested?

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

nvazquez self-assigned this Mar 13, 2019
Comment thread ui/scripts/templates.js Resolved

yadvr commented Mar 29, 2019

Copy link
Copy Markdown

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

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

nvazquez commented Apr 1, 2019

Copy link
Copy Markdown
Member Author

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

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 <rohit.yadav@shapeblue.com>
shwstppr added a commit that referenced this pull request Aug 16, 2021
Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>

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

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>

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

* fix system vm migration with zone wide storage

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>

* fix

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>

* fix spacing

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>

* add empty volumes check

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>

fix fr568 issue #28 (#36)

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

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>

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

Fixes error message which wrongly mentions worker VM

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>

refactor error message for interpod systemvm migration (#38)

Fixes error message introduced in #28

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
shwstppr added a commit that referenced this pull request Nov 23, 2021
Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>

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

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>

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

* fix system vm migration with zone wide storage

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>

* fix

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>

* fix spacing

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>

* add empty volumes check

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>

fix fr568 issue #28 (#36)

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

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>

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

Fixes error message which wrongly mentions worker VM

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>

refactor error message for interpod systemvm migration (#38)

Fixes error message introduced in #28

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

2 participants


Back | FazBrowse Home | New Git URL