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

Log classifier crashes due to llm min_context_padding assert #5613

Closed
huydhn opened this issue Aug 29, 2024 · 2 comments
Closed

Log classifier crashes due to llm min_context_padding assert #5613

huydhn opened this issue Aug 29, 2024 · 2 comments
Labels

Comments

@huydhn
Copy link
Contributor

huydhn commented Aug 29, 2024

From #5611 (comment)

With the example job id 29441399677, the log classifier crashes with the following error:

thread 'main' panicked at src/engine.rs:103:9:
min_context_padding cannot be greater or equal to half of max_chunk_size
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
ERROR lambda_runtime: Any { .. }

cc @PaliC @izaitsevfb @ZainRizvi

@huydhn huydhn added bug Something isn't working high priority labels Aug 29, 2024
@huydhn
Copy link
Contributor Author

huydhn commented Aug 29, 2024

This is probably a high priority item because of how Dr.CI works. Till date, the log classifier rarely fails because of the lack of development works there (only when there is infra outage), so Dr.CI assumes that a failing log classifier signifies flaky infra issue, i.e. lambda fails to trigger, network outage. Also, without the log classifier, there is not much information for Dr.CI to work with anyway.

PaliC added a commit that referenced this issue Aug 30, 2024
@PaliC
Copy link
Contributor

PaliC commented Aug 30, 2024

#5614 should fix this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

No branches or pull requests

2 participants