PB-380: split get_logger()

[little-dude]

References

https://xainag.atlassian.net/browse/PB-380

Summary

get_logger() does not just create a logger and wrap it into
structlog. It actually also creates an "aimetrics" logger and
reconfigures structlog. This commit split get_logger() so that it
only creates a logger.

---

Reviewer checklist

Reviewer agreement:

• Reviewers assign themselves at the start of the review.
• Reviewers do not commit or merge the merge request.
• Reviewers have to check and mark items in the checklist.

Merge request checklist

• Conforms to the merge request title naming XP-XXX <a description in imperative form>.
• Each commit conforms to the naming convention XP-XXX <a description in imperative form>.
• Linked the ticket in the merge request title or the references section.
• Added an informative merge request summary.

Code checklist

• Conforms to the branch naming XP-XXX-<a_small_stub>.
• Passed scope checks.
• Added or updated tests if needed.
• Added or updated code documentation if needed.
• Conforms to Google docstring style.
• Conforms to XAIN structlog style.

[PanicButtonPressed]

Looks fine for me. Probably @Robert-Steiner can also have a look at it.

[Robert-Steiner]

LGTM