Revert "Enable event log for qualification & profiling tools testing …

[gerashegalov]

This reverts #4217 commit b18492e.

#4217 was turning on event logging which was already on by default. #4217 overrode xdist-aware setting of event log locations, and causes the hang

Fixes #12096

Downstream steps in pipelines need to collect event logs under all eventlog_gw[0-9] paths

$ find ./integration_tests/target/run_dir-20250211100159-C4VO -name \*zstd
./integration_tests/target/run_dir-20250211100159-C4VO/eventlog_gw3/eventlog_v2_local-1739268134946/events_1_local-1739268134946.zstd
./integration_tests/target/run_dir-20250211100159-C4VO/eventlog_gw0/eventlog_v2_local-1739268134579/events_1_local-1739268134579.zstd
./integration_tests/target/run_dir-20250211100159-C4VO/eventlog_gw2/eventlog_v2_local-1739268134806/events_1_local-1739268134806.zstd
./integration_tests/target/run_dir-20250211100159-C4VO/eventlog_gw1/eventlog_v2_local-1739268134420/events_1_local-1739268134420.zstd

[gerashegalov]

build

[tgravescs]

Please add a description as to why this is being reverted. I get that it caused some hang, but why did it cause it.

Then is there a followup issue to turn it back on because i assume it was turned on for a reason.

[gerashegalov]

@tgravescs

The issue linked as being fixed #12096 provides the description of how the hang is triggered by the PR being reverted

#4217 was turning on event logging which was already on by default. #4217 overrode xdist-aware setting of event log locations, and causes the hang

I'll repeat this in the PR description.

[mythrocks]

This is a great catch, @gerashegalov. LGTM, FWIW.

[tgravescs]

thanks, I missed the description of solution/root cause in #12096 since it was under the steps to reproduce. I generally try to separate that out into a next comment but we don't have a convention for that. I just missed it.

@zhanga5 to see why original pr went in if eventlogging already enabled? Maybe configuration missing