java-driver icon indicating copy to clipboard operation
java-driver copied to clipboard

New info log print during cluster shutdown, is breaking dtest usages of c-s

Open fruch opened this issue 10 months ago • 4 comments

seems like recently this log line was added during shutdown of the cluster:

INFO  [main] 2025-01-29 02:03:18,734 Metadata.java:164 - Unregistered tablet map's event listener.

dtest has logic that is looking for the END print in c-s output, and this line "stands" in the way and lots of the dtest gating tests failed cause of this silly thing

Ref: https://github.com/scylladb/scylla-dtest/pull/5538

fruch avatar Jan 29 '25 20:01 fruch

Its not really something to fix on the driver side, right? Printing a log is not a bug.

Lorak-mmk avatar Jan 30 '25 14:01 Lorak-mmk

Its not really something to fix on the driver side, right? Printing a log is not a bug.

+1, dtest should not be that fragile, driver does lot's ot async stuff after cluster is being closed, it can throw in anything. Probably what we should do, we should separate logs from output, throwing logs into stderr so that dtest and sct could have a way to separate between them. But it should be done on c-s side.

dkropachev avatar Jan 30 '25 15:01 dkropachev

Also this print is meaningless and doesn't need to be info, seems like it's a leftover from some debugging session

fruch avatar Jan 30 '25 18:01 fruch

Also this print is meaningless and doesn't need to be info, seems like it's a leftover from some debugging session

True, I have created pr to lower it down to debug

dkropachev avatar Jan 30 '25 19:01 dkropachev