go-ipam icon indicating copy to clipboard operation
go-ipam copied to clipboard

Dump and Load only consider defaultNamespace

Open majst01 opened this issue 1 year ago • 1 comments

Both calls must consider all existing Namespaces

majst01 avatar Apr 18 '23 05:04 majst01

Im not sure this was made clear but, the Ipamer interface doesn't get 'namespace' from the context rather, it is hard coded to cal ipamer.NamespcedDump using the defaultNamespace constant.

There is a FIXME in Dump that really should be taking the namespace from the context as do all other calls to the interface (if you want to be consistent anyway).

Same is true for load.

dawenga avatar Jan 12 '24 14:01 dawenga