8 Commits

Author SHA1 Message Date
Chad Whitacre
27c9cc7594
Fix error fingerprinting (#2237) 2023-06-27 02:20:32 +00:00
Benedikt Franke
ec4f416c26
Reference paths relative to project root (#1800)
* Reference paths relative to the current script or project root

Before this PR:
- some scripts change the current working directory and use relative paths
- different approaches are taken to know which directory a script is running in
- paths are sometimes relative, sometimes absolute, sometimes traversing directories

After this PR:
- scripts do neither change nor care much about the current working directory
- a unified approach determines the directory of the current script
- paths are always relative to the project root

This should resolve an issue I already tried to fix with https://github.com/getsentry/self-hosted/pull/1798,
where the contents of `./sentry` were not copied
into the built container image,
thus `enhance-image.sh` did not apply.

Co-authored-by: Amin Vakil <info@aminvakil.com>
2023-02-17 09:59:48 -08:00
Ethan Smith
47ea7a0346
ref: Move jq build to error-handling.sh, and use proxy config (#1895)
This fixes [errors early in the installer run not getting reported correctly](https://github.com/getsentry/self-hosted/issues/1884) and [jq's docker build not using the proxy config correctly](https://github.com/getsentry/self-hosted/issues/1871)
2023-01-06 12:11:58 -08:00
Ethan Smith
433afd9397
Fix jq usage (#1814)
In this PR we fix our usage of jq:
1. Use the container. Before we were accidentally using jq installed in the environment
1. Rewrite generate_breadcrumb_json to read log file all at once
1. Build docker image for jq in unit tests
2022-11-15 13:36:23 -08:00
Hubert Deng
97a4e5cce4
Add context line, error msg to envelope (#1784)
* add context line, error msg to envelope
2022-11-04 08:38:44 -07:00
Ethan Smith
3a07ff4884
Upload end of log as breadcrumbs, use exceptions and stacktrace (#1775)
In this PR, we change the send-event function to write out more information into the envelope we send via sentry-cli, particularly adding breadcrumbs with the end of the log and giving the stack trace in a format Sentry self-hosted can understand.

Fixes getsentry/team-ospo#54 and getsentry/team-ospo#53
Example issue with this formatting: https://self-hosted.getsentry.net/organizations/self-hosted/issues/227/?query=is%3Aunresolved
2022-10-26 14:15:32 -07:00
Ethan Smith
6b9306a17c
Add pre-commit config (#1738) 2022-10-21 13:46:35 -07:00
Hubert Deng
871c5f6092
Reorganize unit test layout (#1729)
* fix failing unit tests

* fix unit test pipeline

* add newline to unit-test file

* preserve semantics from docker volume test

* add error-handling tests

* reorganize file structure of unit tests

* add comment for using _file format in ensure relay credentials test

* add error handling unit tests

* use unit-test directory

* stack trace should be from the unit test directory

* Slightly better fail signal

* Clearer start as well

Co-authored-by: Chad Whitacre <chadwhitacre@sentry.io>
2022-09-29 14:48:48 -07:00