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(alerts): configure alerts for payment wallet ingestion #2608

Merged
merged 12 commits into from
Dec 17, 2024

Conversation

pietro-tota
Copy link
Contributor

@pietro-tota pietro-tota commented Nov 26, 2024

List of changes

Add alerts to monitor:

  • wallet ingestion storage queue write/read event rate
  • wallet ingestion event hub written events in a day

Motivation and context

Those alerts will monitor payment wallet ingestion dedicated queues checking that onboarded wallet events are processed and sent to data-lake dedicated event hub.
Those events are not linked to opsgenie since they have not to be linked to the on-call system and any eventual ingestion problem will be taken into account during working hours

Type of changes

  • Add new resources
  • Update configuration to existing resources
  • Remove existing resources

Does this introduce a change to production resources with possible user impact?

  • Yes, users may be impacted applying this change
  • No

Does this introduce an unwanted change on infrastructure? Check terraform plan execution result

  • Yes
  • No

Other information


If PR is partially applied, why? (reserved to mantainers)

@pietro-tota pietro-tota self-assigned this Nov 26, 2024
@pietro-tota pietro-tota requested review from a team as code owners November 26, 2024 13:44
CianoDanilo
CianoDanilo previously approved these changes Nov 28, 2024
infantesimone
infantesimone previously approved these changes Nov 29, 2024
@pietro-tota pietro-tota marked this pull request as draft December 3, 2024 17:01
@pietro-tota
Copy link
Contributor Author

Apply on hold: alerts will be activated once rollout in production have been completed

@pietro-tota pietro-tota marked this pull request as ready for review December 17, 2024 13:41
@infantesimone infantesimone dismissed stale reviews from CianoDanilo and themself via d5dbbb6 December 17, 2024 13:55
@pietro-tota pietro-tota merged commit 2a83ed1 into main Dec 17, 2024
4 of 7 checks passed
@pietro-tota pietro-tota deleted the CHK-3592-payment-wallet-ingestion-alerts branch December 17, 2024 13:57
Copy link

🎉 This PR is included in version 1.324.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants