Skip to content

Allow download of System VM templates through the UI#7039

Merged
DaanHoogland merged 1 commit intoapache:mainfrom
scclouds:allow_download_of_system_vms_in_the_UI
Jan 4, 2023
Merged

Allow download of System VM templates through the UI#7039
DaanHoogland merged 1 commit intoapache:mainfrom
scclouds:allow_download_of_system_vms_in_the_UI

Conversation

@GaOrtiga
Copy link
Copy Markdown
Contributor

@GaOrtiga GaOrtiga commented Jan 2, 2023

Description

The PR#6750 allowed Root Admins to download system VM templates through the API. This function was extended to the UI.

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

Before:
image

After:
image

How Has This Been Tested?

Before applying the changes:

  1. I opened the instance menu and checked that I was unable to download the template.

After applying the changes:

  1. I opened the instance menu with a Root Admin account and checked that the download button was available.

  2. I opened the instance menu with a non-Root Admin account and checked that the download button was not available.

@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 2, 2023

Codecov Report

Merging #7039 (19dda7f) into main (4133f0e) will increase coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #7039   +/-   ##
=========================================
  Coverage     11.54%   11.54%           
- Complexity     7519     7520    +1     
=========================================
  Files          2494     2494           
  Lines        246787   246787           
  Branches      38533    38533           
=========================================
+ Hits          28480    28489    +9     
+ Misses       214626   214616   -10     
- Partials       3681     3682    +1     
Impacted Files Coverage Δ
...rg/apache/cloudstack/quota/QuotaStatementImpl.java 40.26% <0.00%> (+3.98%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Jan 2, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@DaanHoogland DaanHoogland added this to the 4.18.0.0 milestone Jan 4, 2023
@DaanHoogland DaanHoogland merged commit 908981e into apache:main Jan 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants