Skip to content

move log config to struct, support disabling stdout for builds#3203

Merged
GuillaumeGomez merged 1 commit intorust-lang:mainfrom
syphar:configu-logging
Feb 12, 2026
Merged

move log config to struct, support disabling stdout for builds#3203
GuillaumeGomez merged 1 commit intorust-lang:mainfrom
syphar:configu-logging

Conversation

@syphar
Copy link
Member

@syphar syphar commented Feb 12, 2026

this is triggered by @marcoieni asking us for logging less to syslog for cost reasons.

Especially the build logs inside rustwide::logging::capture are also stored on S3, so we don't need to additionally log them to stdout.

While I'm on it, move the logging config to a config struct.

@syphar syphar self-assigned this Feb 12, 2026
@syphar syphar requested a review from a team as a code owner February 12, 2026 20:27
@github-actions github-actions bot added the S-waiting-on-review Status: This pull request has been implemented and needs to be reviewed label Feb 12, 2026
@GuillaumeGomez
Copy link
Member

Thanks!

@GuillaumeGomez GuillaumeGomez merged commit bea03fb into rust-lang:main Feb 12, 2026
13 checks passed
@GuillaumeGomez GuillaumeGomez deleted the configu-logging branch February 12, 2026 20:53
@github-actions github-actions bot added S-waiting-on-deploy This PR is ready to be merged, but is waiting for an admin to have time to deploy it and removed S-waiting-on-review Status: This pull request has been implemented and needs to be reviewed labels Feb 12, 2026
@syphar syphar removed the S-waiting-on-deploy This PR is ready to be merged, but is waiting for an admin to have time to deploy it label Feb 13, 2026
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