Skip to content

Conversation

sduenas
Copy link
Member

@sduenas sduenas commented Jun 30, 2025

Organizing log data in a structured format makes easier to read and analyze them.

With this PR, log messages will always have a structured format. The default mode is to print the messages to the
console in a plain format, but it can also be configured to print them in JSON format.

We have added structured logs to GrimoireLab using the 'structlog' package.

@sduenas sduenas requested a review from jjmerchante June 30, 2025 21:26
@sduenas sduenas force-pushed the structlog branch 2 times, most recently from 381a613 to 0f63865 Compare July 3, 2025 19:49
@sduenas sduenas force-pushed the structlog branch 2 times, most recently from 35c09b9 to e92e072 Compare October 10, 2025 09:42
Copy link
Contributor

@jjmerchante jjmerchante left a comment

Choose a reason for hiding this comment

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

I include some comments about the handlers and future improvements.

@sduenas sduenas force-pushed the structlog branch 2 times, most recently from 8b14370 to 4428a93 Compare October 16, 2025 11:10
Copy link
Contributor

@jjmerchante jjmerchante left a comment

Choose a reason for hiding this comment

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

Looks good to me, just a small change

Copy link
Contributor

@jjmerchante jjmerchante left a comment

Choose a reason for hiding this comment

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

LGTM

Organizing log data in a structured format makes easier
to read and analyze them.

With this commit, log messages will always have a structured
format. The default mode is to print the messages to the
console in a plain format, but it can also be configured
to print them in JSON format.

We have added structured logs to GrimoireLab using the
'structlog' package.

By default, tests will run silent. If developers want to get
log messages printed to the console, activate the environment
variable 'GRIMOIRELAB_TESTING_VERBOSE'.

Signed-off-by: Santiago Dueñas <[email protected]>
Some messages were updated to follow the structured format.

Signed-off-by: Santiago Dueñas <[email protected]>
@jjmerchante jjmerchante merged commit 3e052cc into chaoss:main Oct 16, 2025
10 checks passed
@sduenas sduenas deleted the structlog branch October 16, 2025 15:37
@sduenas sduenas moved this to 2.0-alpha1 in GrimoireLab Roadmap Oct 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 2.0-alpha1

Development

Successfully merging this pull request may close these issues.

2 participants