search

eBay / go-ovn

A Go library for OVN Northbound/Southbound DB access using native OVSDB protocol
Apache License 2.0
108 stars 59 forks source link

Add GetSchema() API #134

Closed Pardhakeswar closed 3 years ago

Pardhakeswar commented 3 years ago

Signed-off-by: Pardhakeswar Pacha ppacha@nvidia.com

girishmg commented 3 years ago

/lgtm

@hzhou8 @noah8713 @vtolstov the background behind this is that in ovn-k8s we want to make sure that the OVN backend supports a table before we call APIs. Having this API gives the client a chance to make such decisions.

noah8713 commented 3 years ago

Thanks @Pardhakeswar @girishmg . Looks neat. /lgtm. Seems https://github.com/eBay/go-ovn/pull/133 can re-use this as it validates current table with getschema.

Will wait a bit if @hzhou8 @vtolstov have any other suggestions. Can be merged accordingly.