Skip to content

Fix tests fails on Windows - revert checking log level before lock mutex

Pavel Kisliak requested to merge epam/fix_tests_fails_on_win into main

Type of change

  • Bug Fix
  • Feature

Does this introduce a change in the core logic?

  • [No]

Does this introduce a change in the cloud provider implementation, if so which cloud?

  • AWS
  • Azure
  • GCP
  • IBM

Updates description

Fix tests fails on Windows (SegFault). It's related to last logging refactoring, the issue that mutexes lock was on the upper scope.

std::scoped_lock lock(state.recv_mt_, state.send_mt_, state.proc_mt_);

Actually there is will be enough to just open scope { }, but I've also reverted condition that checks logging level, I think that it's help to avoid lose performance with logging level that is higher that 'verbose'.

Edited by Pavel Kisliak

Merge request reports