Skip to content

UI - Add Network: shows "Offering for Isolated networks with no Source Nat service" on Network Offering for normal users#5904

Merged
sureshanaparti merged 4 commits into
apache:4.16from
EWERK-DIGITAL:feature/add-network-with-normal-user
Feb 9, 2022
Merged

UI - Add Network: shows "Offering for Isolated networks with no Source Nat service" on Network Offering for normal users#5904
sureshanaparti merged 4 commits into
apache:4.16from
EWERK-DIGITAL:feature/add-network-with-normal-user

Conversation

@utchoang
Copy link
Copy Markdown

@utchoang utchoang commented Jan 27, 2022

Description

This PR fixes #5897

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

How Has This Been Tested?

@sureshanaparti
Copy link
Copy Markdown
Contributor

@blueorangutan ui

@blueorangutan
Copy link
Copy Markdown

@sureshanaparti 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: http://qa.cloudstack.cloud:8080/client/pr/5904 (SL-JID-1067)

Comment thread ui/src/views/network/CreateIsolatedNetworkForm.vue Outdated
@weizhouapache
Copy link
Copy Markdown
Member

this depends on #5920 . when #5920 is approved and merged into 4.16/main, I will review this pr.

@DaanHoogland
Copy link
Copy Markdown
Contributor

this depends on #5920 . when #5920 is approved and merged into 4.16/main, I will review this pr.

it just was @weizhouapache

@sureshanaparti
Copy link
Copy Markdown
Contributor

this depends on #5920 . when #5920 is approved and merged into 4.16/main, I will review this pr.

it just was @weizhouapache

@DaanHoogland @weizhouapache work in this PR is still in progress. don't review now.

@utchoang utchoang marked this pull request as ready for review February 7, 2022 01:03
@sureshanaparti
Copy link
Copy Markdown
Contributor

Hi @utchoang Is this PR ready for review / testing.

@sureshanaparti
Copy link
Copy Markdown
Contributor

@blueorangutan ui

@blueorangutan
Copy link
Copy Markdown

@sureshanaparti 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: http://qa.cloudstack.cloud:8080/client/pr/5904 (SL-JID-1107)

@sureshanaparti
Copy link
Copy Markdown
Contributor

Verified with admin and user accounts, able to view the network offerings without source nat. LGTM.

@weizhouapache
Copy link
Copy Markdown
Member

I have tested it just now.

root admin is able to see network offerings for isolated networks, vpc networks (with specifyvlan=true or false). good.
normal user is able to see network offerings for isolated networks, vpc networks (with specifyvlan=false). good
However...
domain admin can only see network offerings for isolated networks, but not vpc networks.

@sureshanaparti
Copy link
Copy Markdown
Contributor

@blueorangutan ui

@blueorangutan
Copy link
Copy Markdown

@sureshanaparti 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: http://qa.cloudstack.cloud:8080/client/pr/5904 (SL-JID-1112)

@sureshanaparti
Copy link
Copy Markdown
Contributor

I have tested it just now.

root admin is able to see network offerings for isolated networks, vpc networks (with specifyvlan=true or false). good. normal user is able to see network offerings for isolated networks, vpc networks (with specifyvlan=false). good However... domain admin can only see network offerings for isolated networks, but not vpc networks.

Hi @weizhouapache can you confirm the domain admin case with the latest changes.

@weizhouapache
Copy link
Copy Markdown
Member

@utchoang @sureshanaparti
I am still not able to see offering for vpc networks when login as domain admin.

see screenshot below
image

for normal users, it looks ok. see below
image

@weizhouapache
Copy link
Copy Markdown
Member

after discussion with @sureshanaparti , I checked my environment and found out it is because regular user has 1 vpc but domain admin does not. it makes sense.

@sureshanaparti
Copy link
Copy Markdown
Contributor

thanks for testing this @weizhouapache

@sureshanaparti sureshanaparti merged commit b275c29 into apache:4.16 Feb 9, 2022
Pearl1594 pushed a commit to shapeblue/cloudstack that referenced this pull request Feb 14, 2022
…e Nat service" on Network Offering for normal users (apache#5904)

* shows "Offering for Isolated networks with no Source Nat service" for normal users

* fixes roles

* fix selected tabs
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.

Add Network : Network Offering Drop down is not showing "Offering for Isolated networks with no Source Nat service "

6 participants