digitalocean / go-openvswitch

Go packages which enable interacting with Open vSwitch and related tools. Apache 2.0 Licensed.
Other
295 stars 95 forks source link

ovsnl: only fail if no known families are found #88

Closed armando-migliaccio closed 4 years ago

armando-migliaccio commented 5 years ago

Before this commit the ovnl client logic was such that it needs to be kept in lock-step synchronization with a particular version of the netlink interface. This is too fragile, as each new update requires touching this logic, which in turn makes it non-backward compatible.

This commit flips the logic on its head and instead makes the client complains only if no 'known' families are found. This makes the init logic future proof and backward compatible.

stevenjm commented 4 years ago

Hi there,

We're also affected by this issue. Is there anything we can do to help it along?

armando-migliaccio commented 4 years ago

let me nudge along to see if I can get this merged, thanks @stevenjm

nicboul commented 4 years ago

Hi there,

We're also affected by this issue. Is there anything we can do to help it along?

@stevenjm please give it a try !