Skip to content

Conversation

@shwstppr
Copy link
Contributor

@shwstppr shwstppr commented Jun 19, 2025

Description

Some changes from PR #10814 are missing in DeployVM and DeployVnf wizards causing warnings in teh browser console.

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
  • test (unit or integration test code)

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

  • Major
  • Minor

Bug Severity

  • BLOCKER
  • Critical
  • Major
  • Minor
  • Trivial

Screenshots (if appropriate):

Before:

Screenshot from 2025-06-19 14-16-37

After:

Launch button shows menu,
image

How Has This Been Tested?

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

Some changes from PR apache#10814 are missing in DeployVM and DeployVnf
wizards causing warnings in teh browser console.

Signed-off-by: Abhishek Kumar <[email protected]>
@shwstppr
Copy link
Contributor Author

@blueorangutan ui

@blueorangutan
Copy link

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

@codecov
Copy link

codecov bot commented Jun 19, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 16.57%. Comparing base (5b7c3b4) to head (4c7ae7b).
Report is 1 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main   #11060      +/-   ##
============================================
- Coverage     16.57%   16.57%   -0.01%     
- Complexity    13967    13968       +1     
============================================
  Files          5743     5743              
  Lines        510468   510470       +2     
  Branches      62073    62074       +1     
============================================
- Hits          84616    84615       -1     
- Misses       416390   416392       +2     
- Partials       9462     9463       +1     
Flag Coverage Δ
uitests 3.90% <ø> (-0.01%) ⬇️
unittests 17.47% <ø> (-0.01%) ⬇️

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.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@blueorangutan
Copy link

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

Copy link
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.

clgtm

@DaanHoogland
Copy link
Contributor

merge forward errors become more frequesnt :( not sure I was the culprit but sorry anyway.

Copy link
Contributor

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

verified in qa

@DaanHoogland DaanHoogland merged commit 0bd35a5 into apache:main Jun 23, 2025
25 checks passed
@DaanHoogland DaanHoogland deleted the ui-fix-deployform-fwdmergeissue branch June 23, 2025 15:48
@DaanHoogland DaanHoogland added this to the 4.21.0 milestone Jul 7, 2025
dhslove pushed a commit to ablecloud-team/ablestack-cloud that referenced this pull request Jul 16, 2025
Some changes from PR apache#10814 are missing in DeployVM and DeployVnf
wizards causing warnings in teh browser console.

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

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants