nuagenetworks / nuage-metroae

Nuage Networks Metro Automation Engine
http://devops.nuagenetworks.net
Apache License 2.0
44 stars 17 forks source link

Separate mgmt-bridge for VSD (in clustered environment) #348

Closed madhukark closed 6 years ago

madhukark commented 7 years ago

The currently build_vars.yml assumes that all VSDs in a clustered environment will be connected to the same management bridge. While this is true in most cases, I have come across customer production environments where each VSD is being deployed on a separate (routable) network. It would be flexible if build_vars had the following:

jonasvermeulen commented 7 years ago

Is this something that was already started ? Apart from VSD, this would be useful as well for VSC, Elastic, etc.

jbemmel commented 7 years ago

At TotalPlay we had a case where different hosts used different bridge names, because they were in different dcs (different teams?)

This would require bridgenames to be associated with individual vms, may not be the same for all vsds for example

Sent from my iPhone

On Sep 12, 2017, at 03:32, Jonas Vermeulen notifications@github.com wrote:

Is this something that was already started ? Apart from VSD, this would be useful as well for VSC, Elastic, etc.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

jbemmel commented 7 years ago

PR submitted: https://github.com/nuagenetworks/nuage-metro/pull/374

It's a fairly trivial change, providing much flexibility

On Tue, Sep 12, 2017 at 8:25 AM, Jeroen Van Bemmel jvb127@gmail.com wrote:

At TotalPlay we had a case where different hosts used different bridge names, because they were in different dcs (different teams?)

This would require bridgenames to be associated with individual vms, may not be the same for all vsds for example

Sent from my iPhone

On Sep 12, 2017, at 03:32, Jonas Vermeulen notifications@github.com wrote:

Is this something that was already started ? Apart from VSD, this would be useful as well for VSC, Elastic, etc.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/nuagenetworks/nuage-metro/issues/348#issuecomment-328782191, or mute the thread https://github.com/notifications/unsubscribe-auth/AB8AC4aDMeyIIwnB-4XfYlg_zl2UtfLaks5shkG2gaJpZM4OuGL0 .

ghost commented 6 years ago

Fixed.