Change _load logging to lowercase #450
Merged
Conversation
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
liustve
changed the title
thpierce
approved these changes
Aug 12, 2025
jj22ee
pushed a commit
that referenced
this pull request
Aug 13, 2025
*Description of changes:* `OTEL_PYTHON_LOG_LEVEL` uses lowercase for log levels. Changing it to fix the uppercase only from this PR: #449 **Testing this on Lambda (with default Lambda config setup)**: <img width="1302" height="200" alt="image" src="https://github.com/user-attachments/assets/4b4070bf-90e6-48ae-a787-1dfee1072512" /> ``` START RequestId: 0157a3c0-c584-49bc-b5ba-fd2008335b83 Version: $LATEST END RequestId: 0157a3c0-c584-49bc-b5ba-fd2008335b83 REPORT RequestId: 0157a3c0-c584-49bc-b5ba-fd2008335b83 Duration: 230.96 ms Billed Duration: 231 ms Memory Size: 512 MB Max Memory Used: 113 MB Init Duration: 1416.80 ms XRAY TraceId: 1-689b663c-01c68093613ed19523fb9c6f SegmentId: 76d15c82545c2530 Sampled: true ``` **Testing this on Lambda (with modified Lambda config setup)**: <img width="1378" height="264" alt="image" src="https://github.com/user-attachments/assets/66a75fb5-77af-49eb-bf23-ad65a4dbdb65" /> ``` Configuration of configurator not loaded, aws_configurator already loaded START RequestId: 16d4902c-527b-4edd-86ef-abd423da365d Version: $LATEST END RequestId: 16d4902c-527b-4edd-86ef-abd423da365d REPORT RequestId: 16d4902c-527b-4edd-86ef-abd423da365d Duration: 166.12 ms Billed Duration: 167 ms Memory Size: 512 MB Max Memory Used: 113 MB Init Duration: 1467.68 ms XRAY TraceId: 1-689b6692-1ea1293937b97276187b0a71 SegmentId: d2ed420726a62b67 Sampled: true ``` See: https://opentelemetry.io/docs/zero-code/python/configuration/#logging For valid logging values. By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Description of changes:
OTEL_PYTHON_LOG_LEVELuses lowercase for log levels. Changing it to fix the uppercase only from this PR:#449
Testing this on Lambda (with default Lambda config setup):
Testing this on Lambda (with modified Lambda config setup):
See: https://opentelemetry.io/docs/zero-code/python/configuration/#logging
For valid logging values.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.