-
Notifications
You must be signed in to change notification settings - Fork 0
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
remove duplicated entries from response #112
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I do not really know. Technically this is wrong. It is not correct for one provider to return also results for the other. Provider deployment should be changed not to do this. In addition to that: once one provider returns results of another one there is no guarantee you get proper values (since due to the interface change there might be some fields missing or populated wrongly).
can we maybe filter out those, that came with provider property not matching which provider did actually returned it? |
Yes, I agree that this behavior is wrong. So i decided just to pop out the duplicated ids from list |
In case when two types of loadbalancers in one region, the entries from lister is duplicated
Closes #110