Skip to content

[Feat] Add SENTRY_ENVIRONMENT configuration for Sentry integration - #15760

Merged
5 commits merged into
BerriAI:mainfrom
Thomas-Mildner:SentryLoggerEnvironment
Oct 21, 2025
Merged

[Feat] Add SENTRY_ENVIRONMENT configuration for Sentry integration#15760
5 commits merged into
BerriAI:mainfrom
Thomas-Mildner:SentryLoggerEnvironment

Conversation

@Thomas-Mildner

Copy link
Copy Markdown
Contributor

Add Sentry Environment Configuration Support

Relevant issues

Feature Proposal for #15759

Pre-Submission checklist

Please complete all items before asking a LiteLLM maintainer to review your PR

  • I have Added testing in the tests/litellm/ directory, Adding at least 1 test is a hard requirement - see details
  • I have added a screenshot of my new test passing locally
image
  • My PR passes all unit tests on make test-unit
  • My PR's scope is as isolated as possible, it only solves 1 specific problem

Type

🆕 New Feature
📖 Documentation

Changes

Sentry does now have "development" as Environment Tag
image

@vercel

vercel Bot commented Oct 21, 2025

Copy link
Copy Markdown

@Thomas-Mildner is attempting to deploy a commit to the CLERKIEAI Team on Vercel.

A member of the Team first needs to authorize it.

@ghost
ghost merged commit 1cfc462 into BerriAI:main Oct 21, 2025
4 of 6 checks passed
fzowl pushed a commit to fzowl/litellm that referenced this pull request Jun 24, 2026
…erriAI#15760)

* [Feat] Add SENTRY_ENVIRONMENT configuration for Sentry integration and corresponding tests

* [Refactor] Enhance test_sentry_environment by mocking sentry_sdk and improving environment handling

* [Fix] Update default SENTRY_ENVIRONMENT to 'production' and enhance test for Sentry integration

* [Fix] Update test_sentry_environment to verify correct handling of SENTRY_ENVIRONMENT values

* [Fix] Update test_sentry_environment to assert correct handling of production environment
This pull request was closed.
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.

1 participant