Skip to content

Commit

Permalink
ENH: Remove question UUID from log record message body
Browse files Browse the repository at this point in the history
BREAKING CHANGE: See others.
  • Loading branch information
cortadocodes committed Jul 31, 2023
1 parent d4ac4d3 commit f752390
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
1 change: 0 additions & 1 deletion octue/cloud/pub_sub/logging.py
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,6 @@ def emit(self, record):
{
"type": "log_record",
"log_record": self._convert_log_record_to_primitives(record),
"question_uuid": self.question_uuid,
},
topic=self.topic,
attributes={"question_uuid": self.question_uuid, "is_question": False},
Expand Down
2 changes: 1 addition & 1 deletion octue/cloud/pub_sub/message_handler.py
Original file line number Diff line number Diff line change
Expand Up @@ -364,7 +364,7 @@ def _handle_log_message(self, message):
# Add information about the immediate child sending the message and colour it with the first colour in the
# colour palette.
immediate_child_analysis_section = colourise(
f"[{self.service_name} | analysis-{message['question_uuid']}]",
f"[{self.service_name} | analysis-{self.question_uuid}]",
text_colour=self._log_message_colours[0],
)

Expand Down

0 comments on commit f752390

Please sign in to comment.