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

Callback k8s client hangs when a reflector cannot be initialized #489

Open
1 task done
fabriziosestito opened this issue Apr 30, 2024 · 1 comment · Fixed by #490
Open
1 task done

Callback k8s client hangs when a reflector cannot be initialized #489

fabriziosestito opened this issue Apr 30, 2024 · 1 comment · Fixed by #490
Assignees
Labels
kind/bug Something isn't working

Comments

@fabriziosestito
Copy link
Contributor

fabriziosestito commented Apr 30, 2024

Is there an existing issue for this?

  • I have searched the existing issues

Current Behavior

The first time a request using a reflector is received, we initialize the reflector for the given resource/selectors.
If the fetching of the initial resource list fails, the reflector does not return as ready, therefore the client hangs indefinitely. As a result, the policy waits till the Kubernetes API server timeout is reached.

Expected Behavior

If the reflector initialization fails, the client should return an error.

@fabriziosestito fabriziosestito self-assigned this Apr 30, 2024
@fabriziosestito fabriziosestito moved this to In Progress in Kubewarden Apr 30, 2024
@fabriziosestito fabriziosestito added the kind/bug Something isn't working label Apr 30, 2024
@fabriziosestito fabriziosestito moved this from In Progress to Pending review in Kubewarden Apr 30, 2024
@github-project-automation github-project-automation bot moved this from Pending review to Done in Kubewarden May 2, 2024
@flavio
Copy link
Member

flavio commented May 17, 2024

Reopening, we reverted the fix because it was causing performance issues on the 1st query of context aware resources.

We need to find a way to notify the policy that the data we're feeding into wasm is stale.

@flavio flavio reopened this May 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something isn't working
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants