发布

  • [OPIK-3077][BE][FE] remove dataset upload limit (#4100)

    frostbyte_neo 发布于 2025-11-19 07:28:19 +00:00

    • OPIK-3077 naive approach initial version

    • OPIK-3077 logical fixes

    • OPIK-3077 code style

    • OPIK-3077 config

    • OPIK-3077 toggle frontend

    • OPIK-3077 compilation error

    • OPIK-3077 avoid dataset lookups for each batch

    • OPIK-3077 code cleanup

    • OPIK-3077 improve memory usage

    • OPIK-3077 attempt at first buffer the file to filesystem, them process async

    • OPIK-3077 configurable idle timeout

    • Revision 12: Increase nginx client_max_body_size to 2GB for CSV uploads

    • Revision 13: Add Helm chart config for CSV uploads - increase nginx body size and proxy timeouts

    • Revision 14: Enable CSV upload feature toggle by default

    • Revision 15: Set 2GB body size limit only for CSV upload route

    • Revision 16: Set 2GB body size limit globally for all routes

    • dummy commit

    • Revision 17: Fix client_max_body_size placement in nginx config

    Move client_max_body_size directive inside server block where it actually
    takes effect. Previously it was in the global section which nginx ignores.

    This fixes the 413 Request Entity Too Large error for CSV uploads.

    • Revision 18: Add X-Opik-Nginx header for nginx layer identification

    Add custom response header to identify Opik frontend nginx in multi-proxy
    environments (ALB → Ingress → Opik Nginx). This helps diagnose which nginx
    layer is returning 413 errors when troubleshooting large file uploads.

    • OPIK-3077 coverage

    • OPIK-3077 properly set the bundle

    • OPIK-3077 code style

    • made client_max_body_size configurable

    • remove hardcoded values for nginx configuration

    • OPIK-3077 remove unnecessary changes

    • OPIK-3077 code style

    • OPIK-3077 configurable idleTimeout

    • OPIK-3077 revert hardcoded changes

    • OPIK-3077 PR comments

    • add default storage for backend and frontend pods

    • Update Helm documentation

    • OPIK-3077 document required configuration changes

    • fix for waitForMysql in opik chart

    • minor fix for workflow

    • Update Helm documentation


    Co-authored-by: Nimrod Lahav nimrod@comet.com
    Co-authored-by: Nimrod Lahav 3535799+Nimrod007@users.noreply.github.com
    Co-authored-by: liyaka liyak@comet.com
    Co-authored-by: CometActions github-actions@comet.com

    下载附件