Skip to content

overrider.settingNamespaces(String... ns) #1259

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

Closed
morhidi opened this issue Jun 1, 2022 · 1 comment · Fixed by #1258
Closed

overrider.settingNamespaces(String... ns) #1259

morhidi opened this issue Jun 1, 2022 · 1 comment · Fixed by #1258

Comments

@morhidi
Copy link

morhidi commented Jun 1, 2022

Hi Folks,

I haven't found an easy way to register a controller with a list of namespaces.

Planning to add some workaround:

operator.register(controller, overrider -> {    
String[] watchedNamespaces = new String[]{"ns1","ns2"};
String fakeNs = UUID.randomUUID().toString();    
overrider.settingNamespace(fakeNs); // <-clears the default, adds a fake
overrider.addingNamespaces(watchedNamespaces);    
overrider.removingNamespaces(fakeNs);}); // <-removes the fakse

Basically I'm missing some methods that clear the default then override the namespaces with multiple values:
overrider.settingNamespaces(String... namespaces)
overrider.settingNamespaces(Set<String> namespaces)

Let me know if I'm missing something here.

@morhidi
Copy link
Author

morhidi commented Jun 1, 2022

cc @csviri

@csviri csviri linked a pull request Jun 1, 2022 that will close this issue
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

Successfully merging a pull request may close this issue.

1 participant