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

filter does not work when switching from pod to ingress view #308

Closed
mosheavni opened this issue Sep 10, 2024 · 11 comments · Fixed by #319
Closed

filter does not work when switching from pod to ingress view #308

mosheavni opened this issue Sep 10, 2024 · 11 comments · Fixed by #319

Comments

@mosheavni
Copy link
Contributor

mosheavni commented Sep 10, 2024

Describe the bug
ok this is weird, I started experiencing it a while ago but disregarded it.
now it's bugging me
when opening straight to pods view, then applying a filter, then switching to ingress, even though there are resources that should match the filter, I see 0 ingresses!
I ran a git bisect and found that this: b637e87 is the first bad commit:

b637e870ba5942697314afa1aa65547a3ba35e95 is the first bad commit

which makes this the last good commit: a076125

a07612541e5b4fe05f5309460def2e1a0ced5f75

Bad commit reproduction

  1. make sure we're aligned to origin/maon
  2. open k8s on pods view
  3. filter for a value
  4. switch to ingress view
  5. filter is applied, ingress exists, yet I see 0!
    2024-09-10_16-56-18 (1)

Good commit reproduction

  1. reset to this commit:
git checkout a07612541e5b4fe05f5309460def2e1a0ced5f75
  1. open k8s on pods view
  2. filter for a value
  3. switch to ingress view
  4. filter is applied, ingress exists, i see the ingress

2024-09-10_17-06-04 (1)

Expected behavior

Screenshots

Environment (please complete the following information):

  • OS: [e.g. OSX]
  • Neovim version [e.g. 0.10]

Additional context

@Ramilito
Copy link
Owner

This is super weird, that commit isn't even touching the code relevant to the filter.
I'm not able to reproduce even though I see it happening in the gif!

debug_ingress

Could it be something with the state files? Could you check if it looks ok?

@mosheavni
Copy link
Contributor Author

This is super weird, that commit isn't even touching the code relevant to the filter. I'm not able to reproduce even though I see it happening in the gif!

debug_ingress debug_ingress

Could it be something with the state files? Could you check if it looks ok?

checked and state file looks fine. i doubled that and removed the state file completely, but the issue stills seem to happen

@mosheavni
Copy link
Contributor Author

this is bad. I have to close and reopen hundred times a day..

@Ramilito
Copy link
Owner

oh man, that's not acceptable, so hard to debug something I can't recreate.
I'll try more tonight, the overview is ready to be checked in, not completed but it gives us something to build on at least. After that, I'll try to fix this.

@mosheavni
Copy link
Contributor Author

Owner

i appreciate it.

@Ramilito
Copy link
Owner

I could not for the life of me replicate that behaviour, found your dotfiles and tried with that setup as well but still nothing.
I'm guessing there is something with the fallback view, will try to look at it again but for now I've added an ingress view, could you check if that fixes it?

@mosheavni
Copy link
Contributor Author

that definitely looks better now, thanks! problem is, I experience it with every fallback resource, so yes it looks like the problem is on the fallback view somehow. maybe we're not clearing stuff when changing a fallbackview? unlike any other view, fallback can be for multiple resources so a proper cleanup is in order?

@Ramilito
Copy link
Owner

yeah something seems to be off there, just not sure why I can't replicate it.
Did a guess fix in d798b5a and merged to main, but not sure if it solves it, could you test it out?

@mosheavni
Copy link
Contributor Author

doesn't seem to solve it.
I'm trying to debug it for the last hour, it seems that the problem lies within find.is_in_table

@mosheavni
Copy link
Contributor Author

for some reason this prevents the problem from happening:
#319

@Ramilito
Copy link
Owner

I love it, mystery programming 😅

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.

2 participants