Skip to content

docs: provide information for SENTRY_AIR_GAP flag on Django config file #3935

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.

Sign up for GitHub

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

Merged
merged 4 commits into from
Sep 15, 2025
Merged

docs: provide information for SENTRY_AIR_GAP flag on Django config file #3935

Changes from 3 commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
fca60b1
docs: provide information for SENTRY_AIR_GAP flag on Django config file
aldy505 Sep 13, 2025
846c0c4
Merge branch 'master' into aldy505/docs/sentry-air-gap
aldy505 Sep 13, 2025
6106d1b
Apply suggestions from code review
aldy505 Sep 13, 2025
9d9e1a4
Apply suggestions from code review
aldy505 Sep 13, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 8 additions & 0 deletions sentry/sentry.conf.example.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -95,6 +95,14 @@ def get_internal_network():
# See https://develop.sentry.dev/self-hosted/experimental/errors-only/
SENTRY_SELF_HOSTED_ERRORS_ONLY = env("COMPOSE_PROFILES") != "feature-complete"

# When running in an air-gapped environment, set this to True to entirely disable
# external network calls and features that require internet connectivity.
#
# Setting the value to False while running in an air-gapped environment will
# cause some containers to raise an exception. One known example is fetching
# AI model prices from various public APIs.
SENTRY_AIR_GAP = False
Copy link
Member

Choose a reason for hiding this comment

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

I feel like this should come from an env variable?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Too much environment variables.

Copy link
Member

Choose a reason for hiding this comment

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

But this airgap setting may need to be a global setting for the entire deploy (like retention days)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I believe no. Since the current settings is strictly just for acquiring AI model prices.

Copy link
Member

Choose a reason for hiding this comment

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

Okay I'll let you deal with this when it happens in the future 🤣


################
# Node Storage #
################
Expand Down
Loading