Skip to content

[Timesketch] Implement direct static file serving and tune Nginx proxy buffers#245

Draft
jaegeral wants to merge 4 commits intogoogle:mainfrom
jaegeral:2026-02-09_nginx
Draft

[Timesketch] Implement direct static file serving and tune Nginx proxy buffers#245
jaegeral wants to merge 4 commits intogoogle:mainfrom
jaegeral:2026-02-09_nginx

Conversation

@jaegeral
Copy link

@jaegeral jaegeral commented Feb 9, 2026

This change optimizes how Timesketch frontend assets are served to resolve Nginx warnings regarding
upstream responses being buffered to disk. By offloading static file delivery from the Python
application server (Gunicorn) to Nginx, we reduce latency and CPU overhead on the backend.

Note: This PR received lots of help by Gemini

@jaegeral
Copy link
Author

jaegeral commented Feb 9, 2026

------------------------------------------------------------------------------------------------------------------------
 Charts to be processed:
------------------------------------------------------------------------------------------------------------------------
 osdfir-infrastructure => (version: "2.7.7", path: "charts/osdfir-infrastructure")
------------------------------------------------------------------------------------------------------------------------

"osdfir-charts" has been added to your repositories
Hang tight while we grab the latest from your chart repositories...
...Successfully got an update from the "osdfir-charts" chart repository
Update Complete. ⎈Happy Helming!⎈
Saving 5 charts
Deleting outdated charts
Linting chart "osdfir-infrastructure => (version: \"2.7.7\", path: \"charts/osdfir-infrastructure\")"
Checking chart "osdfir-infrastructure => (version: \"2.7.7\", path: \"charts/osdfir-infrastructure\")" for a version bump...
Old chart version: 2.7.7
Error: failed linting charts: failed processing charts
New chart version: 2.7.7
------------------------------------------------------------------------------------------------------------------------
 ✖︎ osdfir-infrastructure => (version: "2.7.7", path: "charts/osdfir-infrastructure") > chart version not ok. Needs a version bump! 
------------------------------------------------------------------------------------------------------------------------
failed linting charts: failed processing charts

Do I need to bump the version here?

@wajihyassine
Copy link
Member

Thank you for this @jaegeral !

This looks pretty good, to update the version can you

That will fix the failing test above. There will be a final GH action that runs when thats fixed to make sure the helm chart can install properly.

Did you by chance run these changes locally and were able to upload a Timesketch timeline w/o the issues you were seeing?

@jaegeral
Copy link
Author

jaegeral commented Feb 9, 2026

No I was not testing it locally so far, but I will test it and update the things you mentioned 👍

@wajihyassine wajihyassine self-requested a review February 9, 2026 19:29
@wajihyassine
Copy link
Member

Sweet thank you. To help you with any troubleshooting, here are some commands that might be useful:

  • To exec into nginx pod kubectl exec -it <nginx-pod-name> -- /bin/sh
  • To check init container logs kubectl logs <nginx-pod-name> -c init-static or main pod logs is just kubectl logs <nginx-pod-name>

@jaegeral jaegeral marked this pull request as draft February 10, 2026 09:45
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 this pull request may close these issues.

2 participants