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

Add the Google Cloud Instance ID to Sentry, so we can search logs #2220

Closed
teor2345 opened this issue May 27, 2021 · 1 comment
Closed

Add the Google Cloud Instance ID to Sentry, so we can search logs #2220

teor2345 opened this issue May 27, 2021 · 1 comment
Labels
A-infrastructure Area: Infrastructure changes A-rust Area: Updates to Rust code C-bug Category: This is a bug I-usability Zebra is hard to understand or use S-needs-investigation Status: Needs further investigation

Comments

@teor2345
Copy link
Contributor

Is your feature request related to a problem? Please describe.

When our deployed Zebra instances panic or crash, they send a report to Sentry.io.

But this report only contains the Google Cloud instance name, so we can't search Google Cloud Logs for the instance's logs. This makes it hard to diagnose issues.

Describe the solution you'd like

  • Add the Google Cloud Instance ID to the Sentry metadata
  • Check that those instance IDs can be used to search Google Cloud Logs
    • maybe we'll need to trigger a deliberate crash

Describe alternatives you've considered

If the instance is still running, we can look up the logs from the Google Cloud VPC interface. But that doesn't work for deleted instances.

Additional context

This is blocking bugs like #1678: #2209, #2055

@teor2345 teor2345 added C-bug Category: This is a bug A-rust Area: Updates to Rust code A-infrastructure Area: Infrastructure changes S-needs-triage Status: A bug report needs triage S-needs-investigation Status: Needs further investigation P-Medium I-usability Zebra is hard to understand or use labels May 27, 2021
@mpguerra mpguerra removed the S-needs-triage Status: A bug report needs triage label Jun 14, 2021
@gustavovalverde gustavovalverde self-assigned this Apr 28, 2022
@gustavovalverde gustavovalverde removed their assignment May 13, 2022
@teor2345
Copy link
Contributor Author

This doesn't seem to have been a problem recently.

@teor2345 teor2345 closed this as not planned Won't fix, can't repro, duplicate, stale May 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-infrastructure Area: Infrastructure changes A-rust Area: Updates to Rust code C-bug Category: This is a bug I-usability Zebra is hard to understand or use S-needs-investigation Status: Needs further investigation
Projects
None yet
Development

No branches or pull requests

3 participants