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

logging: configure logging on SR #1020

Merged

Conversation

nosahama
Copy link
Contributor

@nosahama nosahama commented Jan 9, 2025

About this change - What it does

We configure the logging for the FastAPI SR project.

@nosahama nosahama requested review from a team as code owners January 9, 2025 10:50
@nosahama nosahama force-pushed the nosahama/EC-629/add-logging-to-sr branch from 799c067 to 2064bea Compare January 9, 2025 11:00
@nosahama nosahama force-pushed the nosahama/EC-629/add-logging-to-sr branch from 2064bea to 96d0b06 Compare January 9, 2025 11:02
Copy link

github-actions bot commented Jan 9, 2025

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  src/karapace
  config.py 32
  logging_setup.py 22
  src/schema_registry
  __main__.py 66
Project Total  

This report was generated by python-coverage-comment-action

@jjaakola-aiven jjaakola-aiven merged commit 8cd588e into jjaakola-aiven-fastapi Jan 9, 2025
10 checks passed
@jjaakola-aiven jjaakola-aiven deleted the nosahama/EC-629/add-logging-to-sr branch January 9, 2025 11:35
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