Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix loggers not respecting json config (#10808) #11655

Merged
merged 2 commits into from
Apr 1, 2022
Merged

fix loggers not respecting json config (#10808) #11655

merged 2 commits into from
Apr 1, 2022

Conversation

NajiObeid
Copy link
Contributor

@NajiObeid NajiObeid commented Apr 1, 2022 •
edited
Loading

backport #10808 to v9

@NajiObeid NajiObeid self-assigned this Apr 1, 2022
@NajiObeid NajiObeid enabled auto-merge (squash) April 1, 2022 04:55
@github-actions github-actions bot requested review from jakule and russjones April 1, 2022 04:55
smallinsky
smallinsky approved these changes Apr 1, 2022
View reviewed changes
Copy link
Contributor

@smallinsky smallinsky left a comment

Choose a reason for hiding this comment

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

Please mark in the PR that his is a backport.

@NajiObeid NajiObeid merged commit b12de24 into branch/v9 Apr 1, 2022
@NajiObeid NajiObeid deleted the naji/cloud-8120-json-logging-v9-backport branch April 1, 2022 15:09
@webvictim webvictim mentioned this pull request Apr 19, 2022
Closed
@webvictim webvictim mentioned this pull request Jun 8, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@smallinsky smallinsky smallinsky approved these changes

@jimbishopp jimbishopp jimbishopp approved these changes

@jakule jakule Awaiting requested review from jakule

@russjones russjones Awaiting requested review from russjones

Assignees

@NajiObeid NajiObeid

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@NajiObeid @jimbishopp @smallinsky