Skip to content

Remove Falco sidekick webui, since it uses Redis as a backend and that has issues - #629

Merged
wcarlsen merged 1 commit into
mainfrom
remove/falco/sidekick-webui
Jul 24, 2026
Merged

Remove Falco sidekick webui, since it uses Redis as a backend and that has issues#629
wcarlsen merged 1 commit into
mainfrom
remove/falco/sidekick-webui

Conversation

@wcarlsen

Copy link
Copy Markdown
Contributor

TLDR: We opted for removing the webui component

Okay I found that we have around 300k events stored in Redis around the clock and we do not implement TTL (can be set via the values file) on events, so I think that might be why we see crazy memory usage. Also Redis seems to behave like it has a memory leak. Also in the webui OOM's whenever I query it.

Here is my conclusions from investigations:

  • The webui might have worked upon install, but seems to have run into issues since then
  • Redis has around 300k events, but many of them seems to be really old
  • Falcoside kick sends event to webui that then uses Redis as a backend. We cannot set a minimumpriority here, but we can set TTL on events
  • The webui OOM's upon simple queries going back in time

Solutions possible:

  • Remove webui completely
  • Set TTL on event from the webui and empty Redis for old event and set maxmemory-policy to volatile-ttl (evict keys closest to expiry)
image

…t has issues

Signed-off-by: Willi Carlsen <carlsenwilli@gmail.com>
@wcarlsen
wcarlsen requested a review from a team as a code owner July 24, 2026 08:44
@samidbb

samidbb commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

I can't see where this breaks anything on the consumer's end that you have marked to be as major release?

@wcarlsen

Copy link
Copy Markdown
Contributor Author

I can't see where this breaks anything on the consumer's end that you have marked to be as major release?

I'm fine with lowering it to minor or patch. Which do you prefer?

@samidbb samidbb left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good

@wcarlsen
wcarlsen merged commit 42e55d6 into main Jul 24, 2026
6 checks passed
@wcarlsen
wcarlsen deleted the remove/falco/sidekick-webui branch July 24, 2026 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants