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

Create Pinpoint SMS alarms and dashboard #368

Closed
3 tasks
sastels opened this issue Jun 5, 2024 · 9 comments
Closed
3 tasks

Create Pinpoint SMS alarms and dashboard #368

sastels opened this issue Jun 5, 2024 · 9 comments
Assignees

Comments

@sastels
Copy link

sastels commented Jun 5, 2024

Description

As a Notify developer, I need to be able to evaluate Pinpoint SMS performance so that I can determine system performance and availability.

WHY are we building?

Need to be able to monitor Pinpoint SMS functions

WHAT are we building?

Add alarms and dashboards for Pinpoint SMS

VALUE created by our solution

knowing when system is having issues

Acceptance Criteria

  • SNS SMS alarms duplicated for Pinpoint as needed
  • dashboards created / modified as needed

QA Steps

  • send a few Pinpoint SMS and see them in the dashboard
@sastels
Copy link
Author

sastels commented Jun 11, 2024

PR to add some Pinpoint metrics:
cds-snc/notification-terraform#1354

@sastels
Copy link
Author

sastels commented Jun 11, 2024

alarms merged in. Now one for a Pinpoint dashboard!
cds-snc/notification-terraform#1369

@sastels
Copy link
Author

sastels commented Jun 11, 2024

dashboard merged but not all working. Will play with in staging and make a corresponding PR.

@sastels sastels self-assigned this Jun 11, 2024
@sastels
Copy link
Author

sastels commented Jun 11, 2024

Fix/pinpoint dashboard:
cds-snc/notification-terraform#1374

Approved 🎉 Will merge tomorrow.

@sastels
Copy link
Author

sastels commented Jun 12, 2024

had issues with the terraform release, so going to wait and merge this to staging Thursday

@sastels
Copy link
Author

sastels commented Jun 13, 2024

fix for a link in the system dashboard:
cds-snc/notification-terraform#1380

@jimleroyer jimleroyer assigned jimleroyer and unassigned sastels Jun 17, 2024
@jimleroyer
Copy link
Member

jimleroyer commented Jun 17, 2024

Jimmy will review the board once it has been merged! (it was reviewed)

@jimleroyer
Copy link
Member

I checked in production and some data was missing in the widgets, it looked good though for what was there. I would like to wait a bit more once we have pinpoint in production, so that would also be an extra verification step in prod for pinpoint. =)

@jimleroyer
Copy link
Member

Let's move this to done as we monitored the Pinpoint production enabling through it yesterday and it worked perfect!

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

No branches or pull requests

2 participants