cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Matthew Smart <msm...@smartsoftwareinc.com>
Subject Re: [apache/cloudstack] Vyos router plugin (#2105)
Date Sat, 03 Jun 2017 20:52:40 GMT
Hey Paul,

I appreciate the feedback. Building the Vyos integration to the point 
where it can be deployed by ACS in place of the VR is the ultimate goal. 
I plan to get there iteratively though with releases that are functional 
for a subset of the VR functionality. That way it gets the code out in 
the wild and some feedback to me as I work. It is quite an undertaking 
to do the full replacement and it seems prudent to release often 
especially since this is my first ACS contribution. I would certainly 
not refuse any help that is on offer.

That being said, the current PR is fully functional and has feature 
parity with the current Palo Alto Firewall plugin. It is usable in its 
current state. There is one major security issue I am working on right 
now regarding ssh access but other than that it is ready for deployment 
(wouldn't recommend production use yet of course since it has not been 
tested).

Thanks,

Matthew Smart
President
Smart Software Solutions Inc.
108 S Pierre St.
Pierre, SD 57501

Phone: (605) 280-0383
Skype: msmart13
Email: msmart@smartsoftwareinc.com

On 06/03/2017 09:15 AM, Paul Angus wrote:
> Hi Matthew,
>
> I don't know how others feel about this, but I don't think that the core CloudStack code
is the place for Proof-of-Concepts.
>
> I have nothing against your VyOS concept, in fact I have suggested it as an alternative
in the past and I kicked off a Blue Orangutan build and looked at what you had done so far.
>
> My personal feeling is that the we (you) need to have a branch which is kept in sync
with master that is used for this work.  My 2p worth, is that the part that really needs PoC
is refactoring the code such that the VyOS can be deployed in place of a VR, then that CloudStack
can send commands to alternate 'drivers' when configuring what it sees as a user created VR.
>
> I really don't wish to **** on your bonfire, I think that this is a great project I would
love to help out with it...
>
>
> Kind regards,
>
> Paul Angus
>
> paul.angus@shapeblue.com
> www.shapeblue.com
> 53 Chandos Place, Covent Garden, London  WC2N 4HSUK
> @shapeblue
>    
>   
>
>
> -----Original Message-----
> From: Matthew Smart [mailto:msmart@smartsoftwareinc.com]
> Sent: 03 June 2017 12:50
> To: dev@cloudstack.apache.org
> Subject: Re: [apache/cloudstack] Vyos router plugin (#2105)
>
> This didn't go through the first time for some reason...
>
> Hey guys,
>
> I know this is going to take some time to merge. In the meantime, I am preparing my design
for the next iteration of the plugin which will tentatively add DNS, DHCP, VPN, and maybe
VRRP (redundant routing) support. I am not very familiar with GIT or the PR process. Assuming
my design phase completes before my PR is merged how should I go about starting work on the
next iteration of the plugin? Can I create a new branch from the branch that is waiting for
the PR to be merged and make my changes there?
>
> Thanks,
>
> Matthew Smart
> President
> Smart Software Solutions Inc.
> 108 S Pierre St.
> Pierre, SD 57501
>
> Phone: (605) 280-0383
> Skype: msmart13
> Email: msmart@smartsoftwareinc.com
>
>
>
> -------- Forwarded Message --------
> Subject: 	Re: [apache/cloudstack] Vyos router plugin (#2105)
> Date: 	Fri, 2 Jun 2017 15:47:37 -0500
> From: 	Matthew Smart <msmart@smartsoftwareinc.com>
> To:
> reply+0133cbd45f36984313fd3e0860108796fac2fb386eb2d6d492cf00000001153d1d
> reply+b792a169ce0da2cbda@reply.github.com
>
>
>
>
> Hey guys,
>
> I know this is going to take some time to merge. In the meantime, I am preparing my design
for the next iteration of the plugin which will tentatively add DNS, DHCP, VPN, and maybe
VRRP (redundant routing) support. I am not very familiar with GIT or the PR process. Assuming
my design phase completes before my PR is merged how should I go about starting work on the
next iteration of the plugin? Can I create a new branch from the branch that is waiting for
the PR to be merged and make my changes there?
>
> Thanks,
>
> Matthew Smart
> President
> Smart Software Solutions Inc.
> 108 S Pierre St.
> Pierre, SD 57501
>
> Phone: (605) 280-0383
> Skype: msmart13
> Email:msmart@smartsoftwareinc.com
>
> On 05/24/2017 05:08 AM, Blue Orangutan Build Things wrote:
>> Packaging result: ✔centos6 ✔centos7 ✔debian. JID-741
>>
>> —
>> You are receiving this because you were mentioned.
>> Reply to this email directly, view it on GitHub
>> <https://github.com/apache/cloudstack/pull/2105#issuecomment-303679093
>>> ,
>> or mute the thread
>> <https://github.com/notifications/unsubscribe-auth/ATPL1JdWfN4FiJb3qjC
>> 15mP-6zmdwUneks5r9AG3gaJpZM4NbYdv>.Web
>> Bug from
>> https://github.com/notifications/beacon/ATPL1CU8Qdww8m5wKxzpzxmhtOcv9O
>> KMks5r9AG3gaJpZM4NbYdv.gif
>>


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