feat: Sync fork #14
Open
feat: Sync fork #14
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
…nce module views in ui (#2993) Adds spans-first-ui flag to enable displaying starfish/performance modules in ui
run tests every 5 minutes
* fix for tests? * fix tests on master
edit file name
#skip-changelog
* bump kafka and zookeeper versions
* add upgrade test
* fix: use nginx realip module * use Docker default address pools
* upgrade clickhouse
add clickhouse healthchecks
make the script a bit more robust in finding compose peojct name
#skip-changelog
Update config.example.yml typo
add comment explaining the one liner in clickhouse config
* ref(spans): Add new feature flags needed * Add new UI flags * Remove addons ingest flag as we're not releasing addons yet * Remove obsolete flags
* different approach to editing permissions of docker volumes
* Update minimum docker compose requirement docker compose down <service> is now required. * Update docker compose version to new minimum in CI config.
We now process tasks via Kafka consumers instead of celerybeat. This needs to be added to self-hosted as well
update consumer flags
#skip-changelog
#skip-changelog
* remove obsolete SENTRY_RELEASE_HEALTH removed in getsentry/sentry#68226 * remove unused feature flags removed in getsentry/sentry#32010 * remove session-replay-enable-canvas removed in getsentry/sentry#87762
Sentry Admin Script always fail because of missing import of lib script. ### Legal Boilerplate Look, I get it. The entity doing business as "Sentry" was incorporated in the State of Delaware in 2015 as Functional Software, Inc. and is gonna need some rights from me in order to utilize my contributions in this here PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Sentry can use, modify, copy, and redistribute my contributions, under Sentry's choice of terms.
#skip-changelog
Fixes problems that appear when Python SDK 3.0 will be released.
a hard stop is in place with this so it can be removed now
* add shellcheck action to lint bash scripts * fix some shellcheck warnings --------- Co-authored-by: ds <ds@local> Co-authored-by: Burak Yigit Kaya <byk@sentry.io>
* Introduce patches with external kafka * Fix pre-commit hooks * Patch relay config file * Documentation for patches stuff * Provide more helpful information for Docker Compose Override file * Fix grep command * ref: rename to 'optional-modifications' * chore(pre-commit): exclude .patch extension * chore(pre-commit): escape backslash * chore(pre-commit): put exclude field on hooks * chore(pre-commit): put exclude field on top level Based on https://pre-commit.com/#top_level-exclude * chore(pre-commit): move to even more top level
PROFILES_DIR was defaulting to `/var/lib/sentry-profiles` which requires root access. When Vroom image decided to go with non-root default user, this started causing permission issues. Now the image is being refactored and it will not use `/var/lib/sentry-profiles` as the default path so we need to explicitly pass it.
…3760) * Revert "fix(vroom): Explicitly set PROFILES_DIR for upcoming change (#3759)" This reverts commit e07445d. It also very importantly changes where we mount the profiles volume which fixes the issue. Our theory is as follows: 1. Vroom Dockerfile had a line doing `mkdirp /var/lib/sentry-profiles` at image build time. This makes the directory owned by `root` 2. When we mount over that directory, and change permissions we can store the permissions changes _in_ the directory but not the directory itself 3. So when we start the vroom image with the new mount, the contents are owned by `vroom` but the main directory is still owned by `root`. This is also why [this approach](https://github.com/getsentry/vroom/pull/601/files/a23a4e395269ca39fd9bd93ecf902cb42530b5cd) worked as the entrypoint script did this at the start of every container instance. --------- Co-authored-by: Burak Yigit Kaya <byk@sentry.io>
#skip-changelog
feat: enable customization sentry DSN endpoint Update sentry/sentry.conf.example.py
The num-brokers option generates broker host names that don't exist in self-hosted.
#skip-changelog
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.
No description provided.