Skip to content

ui: fix user create featured template from volume#9253

Merged
DaanHoogland merged 2 commits intoapache:4.19from
shapeblue:ui-fix-createtemplatevol
Jun 19, 2024
Merged

ui: fix user create featured template from volume#9253
DaanHoogland merged 2 commits intoapache:4.19from
shapeblue:ui-fix-createtemplatevol

Conversation

@shwstppr
Copy link
Copy Markdown
Contributor

Description

Fixes #9241

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)
  • build/CI

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?

How did you try to break this feature and the system with this change?

Fixes apache#9241

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
@shwstppr shwstppr linked an issue Jun 14, 2024 that may be closed by this pull request
@codecov
Copy link
Copy Markdown

codecov bot commented Jun 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 4.29%. Comparing base (8d02e5f) to head (fa35b8c).
Report is 7 commits behind head on 4.19.

Additional details and impacted files
@@             Coverage Diff              @@
##               4.19   #9253       +/-   ##
============================================
- Coverage     14.95%   4.29%   -10.67%     
============================================
  Files          5378     363     -5015     
  Lines        469930   29393   -440537     
  Branches      59422    5138    -54284     
============================================
- Hits          70301    1261    -69040     
+ Misses       391842   27989   -363853     
+ Partials       7787     143     -7644     
Flag Coverage Δ
uitests 4.29% <ø> (-0.01%) ⬇️
unittests ?

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@DaanHoogland
Copy link
Copy Markdown
Contributor

@blueorangutan ui

@blueorangutan
Copy link
Copy Markdown

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

@blueorangutan
Copy link
Copy Markdown

UI build: ✔️
Live QA URL: https://qa.cloudstack.cloud/simulator/pr/9253 (QA-JID-381)

Copy link
Copy Markdown
Contributor

@sureshanaparti sureshanaparti left a comment

Choose a reason for hiding this comment

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

clgtm

@DaanHoogland
Copy link
Copy Markdown
Contributor

@shwstppr , I tried in qa but see no featured field:

with change it looks good but without (qa without PR uri) it looks the same. Any extra condition I am missing maybe?
image

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
@shwstppr
Copy link
Copy Markdown
Contributor Author

@DaanHoogland I was able to see featured selection option in UI without fix in QA server.
Go to ROOT volume of a stopped VM and create template option.

Before:
Screenshot from 2024-06-18 13-36-10

After:
image

@DaanHoogland
Copy link
Copy Markdown
Contributor

ok @shwstppr , probably I'm creating the wrong template. I'll try again.

@DaanHoogland
Copy link
Copy Markdown
Contributor

indeed I think I was at the wrong location. however the create template from volume works in the before situation and not in the PR specific code.
image
pressing it gives no popup. I tried with clear cache and in private window. Also I tried edge and chrome.

@shwstppr
Copy link
Copy Markdown
Contributor Author

@DaanHoogland there was a js error earlier. I forgot to kick UI build

@blueorangutan ui

@blueorangutan
Copy link
Copy Markdown

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

@blueorangutan
Copy link
Copy Markdown

UI build: ✔️
Live QA URL: https://qa.cloudstack.cloud/simulator/pr/9253 (QA-JID-384)

Copy link
Copy Markdown
Contributor

@DaanHoogland DaanHoogland left a comment

Choose a reason for hiding this comment

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

verified to work

@DaanHoogland DaanHoogland merged commit 7a0cd55 into apache:4.19 Jun 19, 2024
@DaanHoogland DaanHoogland deleted the ui-fix-createtemplatevol branch June 19, 2024 06:52
dhslove pushed a commit to ablecloud-team/ablestack-cloud that referenced this pull request Jun 24, 2024
Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Abhishek's Bugfix List
Status: Done

Development

Successfully merging this pull request may close these issues.

Users have the permission to create featured templates

5 participants