incubator-cloudstack-dev mailing list archives

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

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

Ship it!


Ship It!

- Brian Federle


On July 10, 2012, 10:18 a.m., Pranav Saxena wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/5881/
> -----------------------------------------------------------
> 
> (Updated July 10, 2012, 10:18 a.m.)
> 
> 
> 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