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

Hide Uncontained filter in DNS view not hiding uncontained #2170

Closed
abuehrle opened this issue Jan 31, 2017 · 5 comments
Closed

Hide Uncontained filter in DNS view not hiding uncontained #2170

abuehrle opened this issue Jan 31, 2017 · 5 comments
Assignees
Labels
bug Broken end user or developer functionality; not working as the developers intended it component/ui Predominantly a front-end issue; most/all of the work can be completed by a f/e developer
Milestone

Comments

@abuehrle
Copy link
Contributor

Select DNS view, and filter by application containers and then filter out uncontained. You can see that the uncontained are still appearing.

screen shot 2017-01-31 at 8 29 09 am

@2opremio 2opremio added bug Broken end user or developer functionality; not working as the developers intended it component/ui Predominantly a front-end issue; most/all of the work can be completed by a f/e developer labels Jan 31, 2017
@2opremio 2opremio added this to the Next milestone Jan 31, 2017
@2opremio
Copy link
Contributor

Related to #2120 ?

@abuehrle
Copy link
Contributor Author

abuehrle commented Jan 31, 2017

I wasn't private browsing, but I'll try that!

The hide unmanaged containers filter button doesn't seem to work at all when you are in the Containers --> DNS view. It does work when you're filtered outside of that view.

@rade
Copy link
Member

rade commented Feb 1, 2017

just reproduced this on Weave Cloud. @2opremio are you sure this is a UI issue? Is the theory here that the UI is failing to pass the correct filter options to the b/e?

@2opremio
Copy link
Contributor

2opremio commented Feb 1, 2017

Yes, that's the theory, but I may be wrong.

@fbarl fbarl self-assigned this Feb 2, 2017
@fbarl
Copy link
Contributor

fbarl commented Feb 6, 2017

@fons I spent some time looking into this (running Scope locally) and here are my observations:

  • The UI seems to always pass the correct filter options to the backend
  • IsNotPseudo method gets called if and only if the Hide Uncontained option is selected, which is good
  • Output of IsNotPseudo doesn't result in correct filtering, e.g.:
    • making it always return false still shows the internet nodes (but only them)
    • making it return n.Topology != Pseudo doesn't filter out anything, which is weird since the logging inside that method shows that n.Topology == Pseudo when n is the uncontained node

So my guess would be that the bug occurs when the filters are applied or somewhere deeper. I also tried disabling renderCache, but with no results. I'll continue looking into it, but it's taking me some time to really understand what's happening where in the code :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Broken end user or developer functionality; not working as the developers intended it component/ui Predominantly a front-end issue; most/all of the work can be completed by a f/e developer
Projects
None yet
Development

No branches or pull requests

4 participants