Make common MX bluesky logging

[olliesilvester]

Fixes #574

I will either put the below instructions in the docs or as a helper function:

A beamline should use these logs by:

1. Creating a new global logger, eg LOGGER = logging.getLogger("Hyperion")
2. Importing the dodal logger and setting this as the parent
3. Running do_default_logging_setup

eg:

from dodal.log import LOGGER as dodal_logger
from mx_bluesky.common.utils.log import do_default_logging_setup
LOGGER = logging.getLogger("Hyperion")
LOGGER.parent = dodal_logger
do_default_logging_setup(...)

[codecov]

Codecov Report

Attention: Patch coverage is 88.67925% with 6 lines in your changes missing coverage. Please review.

Project coverage is 78.24%. Comparing base (2957b1e) to head (026e3ce).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #580      +/-   ##
==========================================
+ Coverage   78.21%   78.24%   +0.03%     
==========================================
  Files          92       93       +1     
  Lines        6797     6809      +12     
==========================================
+ Hits         5316     5328      +12     
  Misses       1481     1481              

Components	Coverage Δ
i24 SSX	57.17% <ø> (ø)
hyperion	96.37% <100.00%> (+0.13%)	⬆️
other	95.52% <85.71%> (-4.48%)	⬇️

[noemifrisina]

I will either put the below instructions in the docs or as a helper function

Something in the developer docs about how the logs work and how to set them up would definitely be helpful - thinking mainly in terms of having a reference for the future.

[olliesilvester]

@noemifrisina has requested an option to toggle whether or not this function runs integrate_bluesky_and_ophyd_logging, so I'll add that

[DominicOram]

Why? Whats the usecase for dropping all the ophyd/bluesky logs?

[olliesilvester]

From what @noemifrisina was saying (but correct me if I'm wrong), it's useful to see the sys.sterr logs on i24, and it gets cluttered with ophyd+bluesky+ophyd-async logs. Also, she was seeing an issue where ophyd debug logs were appearing even when it wasn't set to debug level

[noemifrisina]

Yes, mainly to have the choice of getting everything or just setting up the ones we're interested in (in i24 case we don't need the debug logs from ophyd, just ophyd_async).

[DominicOram]

Can we replace some of the code in i24/serial/log.py with calls to this common one to make it consistent?

[olliesilvester]

Can we replace some of the code in i24/serial/log.py with calls to this common one to make it consistent?

Yes, I was thinking this could be done in separate PR (maybe #572). Happy to do it in here instead though

[noemifrisina]

We could do this here, or I can pull the changes from this branch into #572 and do that bit in there. I'm fine either way, if we're going to have that in this PR I'll just close the other one.