Exceptions should use logger.exception, not logger.error

[spwoodcock]

Describe the bug

• We use log.error in most places, even when exceptions are raised.
• This is fine when viewing logs in the terminal.
• However to send via OTEL to tools like Sentry, the traceback is only included if log.exception is used.
  • This is essentially the same as log.error(exc_info=True).
• If we want the full traceback stack to be included, then we need log.exception("some info", stack_info=True)