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

Truncated log message in the event #8291

Closed
xtrey opened this issue Aug 8, 2024 · 3 comments · Fixed by #8403
Closed

Truncated log message in the event #8291

xtrey opened this issue Aug 8, 2024 · 3 comments · Fixed by #8403

Comments

@xtrey
Copy link

xtrey commented Aug 8, 2024

SCT truncates the log messages, so the full message is not visible in the events.
Job example: https://argus.scylladb.com/test/4051b1c9-c265-4cdb-bf19-e24748ed3302/runs?additionalRuns[]=fdaae0bd-f82c-4726-a498-7edbd36aa7ef

Actual event message:

Stress command completed with bad status 1: com.datastax.driver.core.exceptions.InvalidQueryException: Cannot create CDC log for a table cdc_tes

But the full message from the cassandra-stress log is actualy reports the root cause of the problem: #16317 :

Stress command completed with bad status 1: com.datastax.driver.core.exceptions.InvalidQueryException: Cannot create CDC log for a table cdc_test.test_table, because keyspace uses tablets. See issue #16317.

so, we should print the full message instead of truncating it to speed up Agrus investigations

@temichus
Copy link
Contributor

temichus commented Aug 8, 2024

cc @roydahan

@fruch
Copy link
Contributor

fruch commented Aug 8, 2024

I don't think it's classified as tier1 issue.

It's a cosmetic issue

We can change to show two first lines (and not 100 characters or something like that, that is now)

Even that can really help when you have dozens of lines in stderr of the stress command.

And one would need to open the stress log to analyze the issue

@fruch
Copy link
Contributor

fruch commented Aug 8, 2024

This is the place to fix this one:

return f"Stress command completed with bad status {exc.result.exited}: {exc.result.stderr:.100}"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants