add sentry function #9
Conversation
Signed-off-by: aryan lamba <[email protected]>
|
@peterdudfield can you take a look |
What do you want to discuss? About this issue? |
may be if i get issue so for that |
|
@peterdudfield have a look i thank at this time all are good . |
Not currently. Please ask on the relevant discussion in https://github.com/orgs/openclimatefix/discussions/categories/project-ideas-2025 |
|
@peterdudfield have a look |
lambaaryan011
force-pushed
the
add-sentry
branch
from
ee89c4a to
0b26b0c
Compare
Signed-off-by: aryan lamba <[email protected]>
|
@devsjc have a look |
|
@peterdudfield Is our sentry environment "production" regardless of where the code is deployed? Or should that be an environment variable? |
|
It should be an env var |
|
@devsjc I update the environment so can you look at this |
Pull Request
Description
This PR adds Sentry integration to the satellite-consumer project. The integration helps in error tracking and performance monitoring by capturing exceptions and logs throughout the data processing pipeline.
Fixes #5
How Has This Been Tested?
⇾ Verified that errors are properly captured in Sentry by intentionally triggering an exception.
⇾ Ran the full data processing pipeline to ensure no breaking changes were introduced.
⇾ Logged relevant metadata (satellite name, time window, etc.) in Sentry for better debugging.
If your changes affect data processing, have you plotted any changes? i.e. have you done a quick sanity check?
Checklist: