Skip to content

ui: fix create network access in deploy vm wizard - #9117

Merged
shwstppr merged 1 commit into
apache:4.19from
shapeblue:fix-ui-deployvmcreatenwaccess
May 28, 2024
Merged

shwstppr merged 1 commit into
apache:4.19from
shapeblue:fix-ui-deployvmcreatenwaccess

Conversation

@shwstppr

Copy link
Copy Markdown
Contributor

Description

Fixes #9115

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

For account with createNetwork access,
Screenshot from 2024-05-24 11-14-35

For account without createNetwork access,
Screenshot from 2024-05-24 11-14-56

How Has This Been Tested?

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

Fixes apache#9115

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

codecov Bot commented May 24, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 15.66%. Comparing base (87e7c57) to head (3b2fefd).
Report is 6 commits behind head on 4.19.

Additional details and impacted files
@@             Coverage Diff              @@
##               4.19    #9117      +/-   ##
============================================
+ Coverage     14.96%   15.66%   +0.70%     
+ Complexity    10995    10986       -9     
============================================
  Files          5373     5010     -363     
  Lines        469005   439844   -29161     
  Branches      58953    54564    -4389     
============================================
- Hits          70198    68914    -1284     
+ Misses       391036   363305   -27731     
+ Partials       7771     7625     -146     
Flag Coverage Δ
uitests ?
unittests 15.66% <ø> (-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.

@weizhouapache weizhouapache added this to the 4.19.1.0 milestone May 24, 2024
@weizhouapache

Copy link
Copy Markdown
Member

@blueorangutan ui

@blueorangutan

Copy link
Copy Markdown

@weizhouapache 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/9117 (QA-JID-355)

@DaanHoogland DaanHoogland left a comment

Copy link
Copy Markdown
Contributor

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

tested in qa:
image

@weizhouapache weizhouapache left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

code lgtm

@shwstppr
shwstppr merged commit 283a485 into apache:4.19 May 28, 2024
@shwstppr
shwstppr deleted the fix-ui-deployvmcreatenwaccess branch May 28, 2024 06:37
DaanHoogland added a commit that referenced this pull request May 28, 2024
* 4.19:
  engine-schema: add removed columne for cloud.user_data, fix delete op (#9120)
  ui: fix create menu item access (#9104)
  server: honor listall param for listiso api (#9064)
  ui: add move to top/bottom for acl rules list (#9119)
  ui: fix columns for exportacls csv (#9118)
  ui: fix create network access in deploy vm wizard (#9117)
  UI: Add search filters (#9068)
dhslove pushed a commit to ablecloud-team/ablestack-cloud that referenced this pull request Jun 17, 2024
Fixes apache#9115

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
dhslove pushed a commit to ablecloud-team/ablestack-cloud that referenced this pull request Jun 17, 2024
* 4.19:
  engine-schema: add removed columne for cloud.user_data, fix delete op (apache#9120)
  ui: fix create menu item access (apache#9104)
  server: honor listall param for listiso api (apache#9064)
  ui: add move to top/bottom for acl rules list (apache#9119)
  ui: fix columns for exportacls csv (apache#9118)
  ui: fix create network access in deploy vm wizard (apache#9117)
  UI: Add search filters (apache#9068)
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: No status

Development

Successfully merging this pull request may close these issues.

Create new Network button present on Add Instance Form when account has no permission

4 participants