Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dump and Load only consider defaultNamespace #111

Open
majst01 opened this issue Apr 18, 2023 · 1 comment
Open

Dump and Load only consider defaultNamespace #111

majst01 opened this issue Apr 18, 2023 · 1 comment

Comments

@majst01
Copy link
Contributor

majst01 commented Apr 18, 2023

Both calls must consider all existing Namespaces

@dawenga
Copy link

dawenga commented Jan 12, 2024

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants