Repository navigation
feat(runner): Add sentry-server and deprecate the sentry console script - #129100
Closed
sentry-junior[bot] wants to merge 2 commits into
Closed
sentry-junior[bot] wants to merge 2 commits into
sentry-junior[bot] wants to merge 2 commits into
Conversation
The server's `sentry` entry point shadows the Sentry CLI (getsentry/cli) in dev venvs. Rename it to `sentry-server` and stop deleting other `sentry` binaries in .envrc and postinstall so the CLI can win. devenv sync removes the stale `.venv/bin/sentry` that fast_editable leaves behind. The self-hosted image keeps a deprecated `sentry` symlink for existing manifests. Co-Authored-By: David Cramer <david@sentry.io>
Contributor
|
🚨 Warning: This pull request contains Frontend and Backend changes! It's discouraged to make changes to Sentry's Frontend and Backend in a single pull request. The Frontend and Backend are not atomically deployed. If the changes are interdependent of each other, they must be separated into two pull requests and be made forward or backwards compatible, such that the Backend or Frontend can be safely deployed independently. Have questions? Please ask in the |
getsentry builds against sentry master and its image, CI, and options scripts still call `sentry` directly, so a hard cut would break it. Keep both console scripts for now; the alias, and cleanup of stale `.venv/bin/sentry` scripts, go away in a follow-up. Co-Authored-By: Joshua Li <14209781+joshuarli@users.noreply.github.com>
This was referenced Oct 28, 2026
Closed
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
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.
Rollout:
sentry→sentry-server,getsentry→getsentry-serverWhy: the new Sentry CLI ships a binary named
sentry, which the server's console script shadows. Same rollout across all PRs below; keep this section in sync.Phase 1: add new names and keep the old names as deprecated aliases (merge first)
sentry-server, keeps thesentryalias, moves in-repo callersPhase 2: move callers to the new names (after phase 1)
getsentry-server(keepsgetsentryalias) and moves callers. NeedsRequireson the sentry PR for CI; image builds needsentry-versionbumped past the phase 1 mergesentry-serverships in thenightlyimagesentry/getsentryPhase 3: remove the aliases (only after prod manifests and self-hosted are confirmed off the old names)
sentryalias and clean up stale.venv/bin/sentry(TBD)getsentryalias (TBD)command: ["sentry"]) before the sentry alias is removedAdds a
sentry-serverconsole script and keepssentryas a deprecated alias, both pointing atsentry.__main__:main.The new Sentry CLI (getsentry/cli) also installs a
sentrybinary, and the server's console script shadows it. This PR moves every in-repo caller tosentry-server: devservices programs, devserver daemons,devenv/sync.py,scripts/lib.sh, the Makefile, CI workflows, the self-hosted Dockerfile and entrypoint,.vscode/launch.json, the click prog name (Usage: sentry-server …), user-facing messages, and agent/dev docs.It also removes the
.envrcandpackage.jsonpostinstall logic that deletednode_modules/.bin/sentryand died on any othersentrybinary on PATH, so the npm CLI is no longer actively deleted..envrcnow checks for.venv/bin/sentry-server.Compatibility
sentrystays insetup.cfg, so dev venvs and images ship both names. getsentry builds against sentry master and its image, CI, GoCD jobs, andbin/options/*still callsentrydirectly. So do ops manifests and self-hosted.docker_entrypoint.pyprependssentry-serverfor bare subcommands and wraps bothsentryandsentry-serverin tini.#!/usr/bin/env sentry execscripts keep working through the alias, but should move tosentry-server exec.Merge order: this PR lands first. getsentry/getsentry#22824 follows. Alias removal is a later phase 3 PR, after getsentry, ops manifests, and self-hosted are confirmed off
sentry. Dev shadowing is only fully fixed in phase 3: the venv'ssentryalias still ranks ahead of the CLI on PATH until then. Phase 3 also has to delete the stale.venv/bin/sentryscript, becausetools/fast_editable.pynever removes old console scripts.Changelog Entry
sentry-serveras the Sentry server command.sentrystill works as a deprecated alias and will be removed in a future release; update scripts, including#!/usr/bin/env sentry exechashbangs, to usesentry-server.via @joshuarli.
--
View Junior Session [Sentry]