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.

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

ES-2058: Add Corda container logging configuration #99

Merged
merged 1 commit into from
Mar 15, 2024

Conversation

anton-subbotin
Copy link
Contributor

@anton-subbotin anton-subbotin commented Mar 15, 2024

  • logs go to both console and files
  • console logs are available via docker logs
  • file logs are at the same location used by CSDE template
  • default configurations of logging level and debug logging are provided explicitly, allowing users to modify as needed

All of the above tested locally.

image

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

PR title failed to match regex -> ^((CORDA|EG|ENT|INFRA|CORE|ES)-\d+)(.*)

@anton-subbotin anton-subbotin changed the title Asubbotin/es 2058/add logging config ES-2058: Add Corda container logging configuration Mar 15, 2024
@github-actions github-actions bot dismissed their stale review March 15, 2024 10:45

All good!

@anton-subbotin anton-subbotin marked this pull request as ready for review March 15, 2024 11:00
@anton-subbotin anton-subbotin changed the base branch from release-V4 to release-V5.2 March 15, 2024 11:00
@anton-subbotin anton-subbotin requested a review from tlawson3 March 15, 2024 11:02
@anton-subbotin anton-subbotin merged commit 0896db8 into release-V5.2 Mar 15, 2024
5 checks passed
@anton-subbotin anton-subbotin deleted the asubbotin/ES-2058/add-logging-config branch March 15, 2024 11:44
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