The main objective is to have a nicira based private network guru to use for vpsgateways. I want to abstract common code with the 'generic' vlan based private network but also abstract out commonalities that might be shared with the guest networks. canHandle could be generalized and called by all classes, though it has another footprint now in the guest networks then it has in PrivateNetworkGuru.
Alternatively I will copy code from NiciraNvpGuestNetworkGuru and PrivateNetworkGuru to a new class and later refactor it, which is not my favorite way to go. I must admit that including network gurus that do not support any extensions in the hierarchy is an esthetic touch if no code is shared. I will refrain if maintainability issues can be expected. Regards, -----Original Message----- From: Murali Reddy [mailto:murali.re...@citrix.com] Sent: dinsdag 7 mei 2013 15:17 To: dev@cloudstack.apache.org Subject: Re: network guru refactor proposal On 07/05/13 5:23 PM, "Daan Hoogland" <dhoogl...@schubergphilis.com> wrote: >LS, > >I want to refactor the network guru hierarchy to put som functionality >in abstract base classes. This will come down to extending the >hierarchy for guest networks to include all gurus. Are there any >thoughts or gotchas to share? GuestNetworkGuru in some sense already acting as abstract base class, except for the fact that it is tied to 'Vlan' isolation. We can generalise the 'GuestNetworkGuru' and let the isolation type specific network design aspects to concrete classes. Other gurus (direct, pod based) for guest networks does not have any extensions at this point and does not overlap much with GuestNetworkGuru, so they may remain as is. Are there any specify observations that you think refactor will address? > >This would be the second part of a three stage strategy I have to >support creating a nicira hosted private gateway for vpcs. >The first one is making sure vlans are specified as uri throughout the >system. I will be submitting a patch for review for this part soon. >The last part will be creating a guru based on Hugo's >NiciraNvpGuestNetworkGuru. > >Any comment is appreciated, >Daan Hoogland >