add sentry function #9
Conversation
Signed-off-by: aryan lamba <aryanlamba54@gmail.com>
|
@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 <aryanlamba54@gmail.com>
|
@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 |
|
@peterdudfield I think now all things are done that you told me, so look at this ? |
There was a problem hiding this comment.
I think this is good, but ill get @devsjc to double check it
|
Hi @devsjc , Just following up on my PR. Could you please review it when you get a chance? Let me know if any changes are needed |
lambaaryan011
force-pushed
the
add-sentry
branch
from
20deca4 to
7171873
Compare
I don’t see an issue, but for better clarity and performance, I optimized the code. |
| sentry_sdk.set_tag("app_name", "satellite_consumer") | ||
| sentry_sdk.set_tag("version", __version__) | ||
|
|
||
| def _consume_command(command_opts: ArchiveCommandOptions | ConsumeCommandOptions) -> None: |
There was a problem hiding this comment.
can you reset all changes below this, if you want to do this, please put in a different PR and explain what they are for and why
There was a problem hiding this comment.
if there is green lines below here, then it means new lines have been added
lambaaryan011
force-pushed
the
add-sentry
branch
from
912d12b to
fd8b9fe
Compare
lambaaryan011
force-pushed
the
add-sentry
branch
from
eb8ab2f to
fd8b9fe
Compare
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: