Skip to content
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.

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

devops: re-integrate Sentry to our CD and Release nodes #8862

Closed
gustavovalverde opened this issue Sep 11, 2024 · 1 comment
Closed

devops: re-integrate Sentry to our CD and Release nodes #8862

gustavovalverde opened this issue Sep 11, 2024 · 1 comment
Assignees
Labels
A-devops Area: Pipelines, CI/CD and Dockerfiles A-infrastructure Area: Infrastructure changes C-bug Category: This is a bug C-design Category: Software design work C-tech-debt Category: Code maintainability issues P-High 🔥 S-needs-triage Status: A bug report needs triage

Comments

@gustavovalverde
Copy link
Member

Describe the issue or request

For multiple previous issues we've not been using Sentry for a while, but there are no longer any more roadblocks

Expected Behavior

We should be notified about panics, race conditions or other non-expected behaviors in Zebra after we deploy our main branch nodes, and our releases nodes.

This will allow us to get notified on hard to identify errors happening on an ongoing basis in those "unattended" instances.

Current Behavior

We don't have, nor plan to do, further integrations with Google Cloud Logging to classify errors in the application, making it hard to classify error information/logs.

Possible Solution

As most the error handling and classification is easier to do with existing Sentry integrations, and considering Sentry is agnostic to cloud providers; this is a more feasible and general solution we should keep using and maintaining.

This might require:

  • Updating existing code
  • Enabling sentry in our nodes
  • Configuring our Sentry account correctly

Additional Information/Context

No response

Is this happening on PRs?

No

Is this happening on the main branch?

Yes

@gustavovalverde gustavovalverde added C-bug Category: This is a bug C-design Category: Software design work A-infrastructure Area: Infrastructure changes A-devops Area: Pipelines, CI/CD and Dockerfiles S-needs-triage Status: A bug report needs triage C-tech-debt Category: Code maintainability issues P-High 🔥 labels Sep 11, 2024
@mpguerra
Copy link
Contributor

mpguerra commented Oct 1, 2024

this is now done

@mpguerra mpguerra closed this as completed Oct 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-devops Area: Pipelines, CI/CD and Dockerfiles A-infrastructure Area: Infrastructure changes C-bug Category: This is a bug C-design Category: Software design work C-tech-debt Category: Code maintainability issues P-High 🔥 S-needs-triage Status: A bug report needs triage
Projects
Archived in project
Development

No branches or pull requests

2 participants