Skip to content

feat: log details to message log#268

Merged
tsundvoll merged 1 commit into
mainfrom
feat/log-details-to-message-log
Jun 8, 2026
Merged

feat: log details to message log#268
tsundvoll merged 1 commit into
mainfrom
feat/log-details-to-message-log

Conversation

@tsundvoll

@tsundvoll tsundvoll commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

to be used for example from the fmu-sumo-uplader
to include a summary of the upload with the logs

Issue

#fmu-sumo-uploader/issues/219

Description

Necessary change to make use of details provided when logging.

How to test

From a library using the LogHandlerSumo, such as fmu-sumo-uploader, make a logging, such as

details = {
    "caseuuid": "1234-5678",
    "ensamblename": "my_ensamble",
    "numbers_of_objects_uploaded": "42",
    "start_time": "2026-06-04T12:00:00Z",
    "end_time": "2026-06-04T12:30:00Z",
}

logger.info("Upload finished",  extra={"details": details})

And observe that data is stored in the message log.

Checklist

  • No redundant `print()` statements, commented-out code, or other remnants from the development 👀
  • New/refactored code is following same conventions as the rest of the code base 🧬
  • New/refactored code is tested ⚙
  • Documentation has been updated 🧾
  • Commits are semantic ✅

to be used for example from the fmu-sumo-uplader
to include a summary of the upload with the logs
@tsundvoll tsundvoll requested a review from a team June 8, 2026 06:17

@rwiker rwiker left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@tsundvoll tsundvoll merged commit 73244ae into main Jun 8, 2026
11 checks passed
@tsundvoll tsundvoll deleted the feat/log-details-to-message-log branch June 8, 2026 07:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants