Skip to content

Conversation

@aldy505
Copy link
Collaborator

@aldy505 aldy505 commented Dec 21, 2024

DESCRIBE YOUR PR

Self-hosted troubleshooting section is getting all over the place. I think it'd be better if we separate it per component.

IS YOUR CHANGE URGENT?

Help us prioritize incoming PRs by letting us know when the change needs to go live.

  • Urgent deadline (GA date, etc.):
  • Other deadline:
  • None: Not urgent, can wait up to 1 week+

SLA

  • Teamwork makes the dream work, so please add a reviewer to your PRs.
  • Please give the docs team up to 1 week to review your PR unless you've added an urgent due date to it.
    Thanks in advance for your help!

PRE-MERGE CHECKLIST

Make sure you've checked the following before merging your changes:

  • Checked Vercel preview for correctness, including links
  • PR was reviewed and approved by any necessary SMEs (subject matter experts)
  • PR was reviewed and approved by a member of the Sentry docs team

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.

EXTRA RESOURCES

@vercel
Copy link

vercel bot commented Dec 21, 2024

@aldy505 is attempting to deploy a commit to the Sentry Team on Vercel.

A member of the Team first needs to authorize it.

@aldy505 aldy505 marked this pull request as ready for review December 27, 2024 10:35
@aldy505
Copy link
Collaborator Author

aldy505 commented Dec 27, 2024

Can I have this reviewed? Thanks! @bc-sentry @BYK @hubertdeng123

@aldy505 aldy505 requested a review from bc-sentry January 5, 2025 13:39
@aldy505
Copy link
Collaborator Author

aldy505 commented Jan 8, 2025

cc @hubertdeng123 @BYK


## Container Healthcheck

There may be some circumstances which you may want to increase or decrease healthcheck interval, timeout or retries for your custom needs. This can be achieved by editing `HEALTHCHECK_INTERVAL`, `HEALTHCHECK_TIMEOUT`, `HEALTHCHECK_RETRIES` variables' values in `.env`.
Copy link
Member

Choose a reason for hiding this comment

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

You may want to be more specific about how to add these.

docker network prune
```

Append `-f` flag for no confirmation on deletions.
Copy link
Member

Choose a reason for hiding this comment

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

Wouldn't really advertise this :)

```shell
docker compose up -d
```
## Reducing disk usage
Copy link
Member

Choose a reason for hiding this comment

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

May be add a reference to "external kafka" doc/patch when it is done?

Copy link
Member

@BYK BYK left a comment

Choose a reason for hiding this comment

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

Overall looks good to me. Would be great if you can add in the suggestions I made.

Co-authored-by: Burak Yigit Kaya <ben@byk.im>
@BYK BYK enabled auto-merge (squash) January 11, 2025 21:52
@vercel
Copy link

vercel bot commented Jan 11, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
develop-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 11, 2025 10:04pm
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
sentry-docs ⬜️ Ignored (Inspect) Visit Preview Jan 11, 2025 10:04pm

@BYK BYK merged commit 68e447a into getsentry:master Jan 11, 2025
9 of 10 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Jan 27, 2025
@aldy505 aldy505 deleted the docs/self-hosted/cleanup-troubleshooting-section branch June 6, 2025 12:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants