Move to structured logging, with logback, for docker image, testing, …#1796
Merged
Conversation
…and compose files.
krowvin
reviewed
Jun 22, 2026
| @@ -0,0 +1,29 @@ | |||
| <?xml version="1.0" encoding="UTF-8" ?> | |||
| <!DOCTYPE configuration> | |||
| <configuration debug="true"> | |||
Collaborator
There was a problem hiding this comment.
Will this push all the way to prod and make the prod logs noisy?
Contributor
Author
There was a problem hiding this comment.
oops, thanks that should not be in there.
Contributor
Author
There was a problem hiding this comment.
Though that particular debug is about the initial log configuration, so it wouldn't be super noisy, just some initial text at the start, still doesn't need to be in there though.
Co-authored-by: Charles Graham <charles.r.graham@usace.army.mil>
adamkorynta
approved these changes
Jun 23, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
…and compose files.
Summary
Changes logging setup in preparation for w3c trace context.
Intentionally made no additional changes to logging.
Related Issue
Closes #1794
Validation
Manually, change is to logging, verified expected logs were created in the desired format.
Checklist