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

feat: Slack alerting #12

Merged
merged 4 commits into from
Jan 9, 2025

Conversation

anaselmhamdi
Copy link
Collaborator

  • Added an abstract interface for alerting
  • Added a Slack alerting config

@anaselmhamdi anaselmhamdi self-assigned this Jan 9, 2025
Copy link

linear bot commented Jan 9, 2025

@anaselmhamdi anaselmhamdi requested a review from aballiet January 9, 2025 17:35
@anaselmhamdi anaselmhamdi merged commit 401c25b into gorgias Jan 9, 2025
0 of 2 checks passed
@anaselmhamdi anaselmhamdi deleted the anas/daae-4623-implement-alerting-interface branch January 9, 2025 17:35
aballiet added a commit that referenced this pull request Jan 23, 2025
* fix: add infer_schema_length=None to polars transfo (#9)

* fix: raise warning when MSG_SIZE_TOO_LARGE and skip (#10)

* fix: raise warning when MSG_SIZE_TOO_LARGE and skip

* fix: update pagination to only existing messages

* format

* dummy

* fix increment

* feat: Slack alerting (#12)

* chore: update gitignore

* chore: update gitignore

* chore: lint

* chore: add log levels and e2e testing

* feat: commit offsets on successfull read (#11)

* chore: add reusable address (#13)

* chore: change log to info when no records to write

* chore: change log to info when total records mismatch

* fea: add datadog monitoring (#14)

* fea: add datadog monitoring

* chore: remove prometheus and otel

* chore: handle no datadog

* chore: lint

---------

Co-authored-by: Anas El Mhamdi <[email protected]>
anaselmhamdi added a commit that referenced this pull request Jan 23, 2025
* feat: monitoring (datadog) and alerting (slack) (bizon-data#20)

* fix: add infer_schema_length=None to polars transfo (#9)

* fix: raise warning when MSG_SIZE_TOO_LARGE and skip (#10)

* fix: raise warning when MSG_SIZE_TOO_LARGE and skip

* fix: update pagination to only existing messages

* format

* dummy

* fix increment

* feat: Slack alerting (#12)

* chore: update gitignore

* chore: update gitignore

* chore: lint

* chore: add log levels and e2e testing

* feat: commit offsets on successfull read (#11)

* chore: add reusable address (#13)

* chore: change log to info when no records to write

* chore: change log to info when total records mismatch

* fea: add datadog monitoring (#14)

* fea: add datadog monitoring

* chore: remove prometheus and otel

* chore: handle no datadog

* chore: lint

---------

Co-authored-by: Anas El Mhamdi <[email protected]>

* chore: add extra poetry

---------

Co-authored-by: Antoine Balliet <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant