incubator-cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Pranav Saxena" <pranav.sax...@citrix.com>
Subject Review Request: CS-15475:Not showing "Add GuestNetwork" Tab in Network in multi zone setup(basic/advance)
Date Tue, 10 Jul 2012 10:18:56 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/5881/
-----------------------------------------------------------

Review request for cloudstack, Brian Federle and Jessica Wang.


Description
-------

when we configured /enabled "basic zone" on existing setup which as already configured with
Advanced Zone.UI not showing the AddGuestNetwork" in the -Network page
Steps:
1.Build up advance Zone setup
2.configured multiple networks and assign tags
3.navigate to Network- and click on "Add guestnetwork" button to create isolatedGuestNetwork
4.deploy a VM
5.Add another zone "Basic Zone"
6.Navigate to Network- and click on "Add guest-network" button to create isolatedGuestNetwork

Actual result:
"Add GuestNetwork" tab was missing when we add/configred basic zone in the existing setup(which
has advance zone already)

SOLUTION - Modified the logic behind displaying the "Add network tab" . Now any occurence
of advanced zone set up , in any combination with a Basic zone will lead to displaying the
"Add network tab" since we can have as many guest networks in an advanced zone set up. Following
cases will be taken care of - 
1) Only advanced zone - add network tab enabled
2) Only basic zone - Add network tab disabled
3) advanced zone with Basic zone on top - Add network tab enabled
4) Basic Zone with advanced zone on top - Add network tab enabled


This addresses bug CS-15475.


Diffs
-----

  ui/scripts/network.js 150ade2 

Diff: https://reviews.apache.org/r/5881/diff/


Testing
-------

Verified on my local cloudstack set up .Tested for following cases - 
1)Added two zones one with Advanced networking set up and second with Basic . 
2) Only advanced networking zone
3) Only basic zone 
Attaching the screenshot for it.


Thanks,

Pranav Saxena


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message