You're correct...I'll remove the zone column as it is unnecessary here. -Brian
-----Original Message----- From: Chiradeep Vittal [mailto:chiradeep.vit...@citrix.com] Sent: Monday, March 11, 2013 6:06 PM To: cloudstack-dev@incubator.apache.org Cc: Animesh Chaturvedi; Jessica Wang; Pranav Saxena; Sonny Chhen Subject: Re: [MERGE] Multiple IPs per NIC UI Not sure if the images attached to the bug are still accurate. If they are, it strikes me that the 'zone' column is extremely redundant. On 3/11/13 10:56 AM, "Chip Childers" <chip.child...@sungard.com> wrote: >On Mon, Mar 11, 2013 at 10:34:15AM -0700, Brian Federle wrote: >> Hi all, >> >> I would like to merge in the UI support for the multiple IPs per NIC >>feature -- https://issues.apache.org/jira/browse/CLOUDSTACK-1628 >> >> This adds UI support for functionality to A) acquire NIC IPs via the >>instances NICs tab, and B) Use the secondary NICs for port forwarding, >>load balancing, and static NAT. >> >> The UI code currently sits in ui-mulitple-nics. >> >> I will also attach screenshots of the UI to the subtask. >> >> Thanks, >> Brian > >+1 from me. > >Looks like a clean diff with master (as of a few moments ago). > >git diff --stat ui-multiple-nics..master > ui/css/cloudstack3.css | 50 +--- > ui/scripts/instances.js | 9 - > ui/scripts/network.js | 622 >++++++++++++++++++++----------- > ui/scripts/ui-custom/enableStaticNAT.js | 3 +- > ui/scripts/ui/widgets/detailView.js | 29 +-- > ui/scripts/ui/widgets/listView.js | 58 +--- > ui/scripts/ui/widgets/multiEdit.js | 12 +- > 7 files changed, 406 insertions(+), 377 deletions(-) >
