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

Debug logging does not work on DEV #396

Closed
dsmf opened this issue Jan 30, 2024 · 3 comments
Closed

Debug logging does not work on DEV #396

dsmf opened this issue Jan 30, 2024 · 3 comments
Assignees
Labels
bug Something isn't working

Comments

@dsmf
Copy link
Contributor

dsmf commented Jan 30, 2024

Description

During testing #214 on DEV we noticed that log messages with level DEBUG do not appear in the logs (except for the CompositeLog) even though this should be the case.

This story is related to #293.

Current behaviour

Log level DEBUG is not logged.

Expected behavior

Log level DEBUG is logged.

Steps to reproduce the Bug

Run a query that executes code which has some logging with level DEBUG.

@dsmf dsmf added this to IRS Jan 30, 2024
@dsmf dsmf converted this from a draft issue Jan 30, 2024
@ds-alexander-bulgakov ds-alexander-bulgakov added the bug Something isn't working label Jan 30, 2024
@jzbmw jzbmw changed the title Debug logging does work on DEV Debug logging does not work on DEV Jan 31, 2024
@ds-jhartmann
Copy link
Contributor

Might be related to irs-api/src/main/resources/application.yml:

logging:
  pattern:
    console: "%d %-5level %logger : %msg%n"
  level:
    root: WARN
    org.springframework: INFO
    org.eclipse.tractusx.irs: INFO

@jzbmw jzbmw moved this from backlog to next in IRS Feb 6, 2024
@ds-ext-kmassalski ds-ext-kmassalski self-assigned this Feb 7, 2024
@ds-ext-kmassalski ds-ext-kmassalski moved this from next to wip in IRS Feb 7, 2024
@dsmf
Copy link
Contributor Author

dsmf commented Feb 7, 2024

log4j.xml is already set to debug, but still debug isn't logged, only INFO and above. As @ds-jhartmann mentioned in #396 (comment) this is probably overwritten by the values in application.yml.

@ds-ext-kmassalski ds-ext-kmassalski moved this from wip to review in IRS Feb 8, 2024
@ds-alexander-bulgakov
Copy link
Contributor

ds-alexander-bulgakov commented Feb 15, 2024

DEBUG logs are now existing, adjustment of them was additionally done in PR #767.
Ticket is ready for PO-review, FYI @jzbmw .

@jzbmw jzbmw moved this from review to done in IRS Feb 19, 2024
@jzbmw jzbmw closed this as completed Feb 19, 2024
ds-jhartmann pushed a commit that referenced this issue Feb 20, 2024
@mkanal mkanal assigned dsmf and ds-ext-kmassalski and unassigned jzbmw and dsmf Feb 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: done
Development

No branches or pull requests

5 participants