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

[10-10EZ] Update Monitor Documentation #100061

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

coope93
Copy link
Contributor

@coope93 coope93 commented Jan 6, 2025

Update 10-10EZ Monitor Documentation for a new monitor for VANotify failures.
Relevant Ticket

- No Action needed, this alert is letting us know that the error rate is back to normal

## Slack & Email Alert: "1010EZ Silent Failure on sending submission failure email"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Everything above this was autoformatting. This is the new monitor documentation. We can update this as we need to.


- There should be a corresponding rails log named `Error notification to user failed to deliver` and with metadata containing the notification_id and form id (`{ notification_record_id: notification_record.id, form_number: metadata['form_number'] }`). The form id is `10-10EZ`.
- Query the database with `VANotify::Notification.find_by(notification_id: notification_id)` to find the failed email address. This email should correspond to a Personal Information Log from roughly 24 hours prior that is created when a 10-10EZ Form fails submission.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So if the email address is wrong, we should follow the process to pull Veteran data (name & phone number) that should be sent to HEC.

  • Obtain Veteran contact information (Name, Phone number)
  • Send the Veteran contact information via secure, encrypted email (from va.gov email address) to [email protected]
  • Explain what has happened (submission failure) and provide the Veteran contact info.
  • Copy PO (Patrick Bateman) and PMs (Alex Seelig and Heather Justice - VA emails only)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added this to the steps 👍

## Slack & Email Alert: "1010EZ Silent Failure on sending submission failure email"

- A 10-10EZ Form submission has failed submission, and the email that is sent to the Veteran has failed sending through VANotify.
- We should have already seen a `1010EZ submission job has failed with no retries left` failure, and then this monitor would trigger roughly 24 hours later if the email also fails (The email send operation is in a sidekiq job that has 14 retries).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Didn't this get updated to 16 retries?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The VANotify EmailJob does 14 retries, and our 10-10EZ Submission job does 16 retries.

@coope93 coope93 self-assigned this Jan 7, 2025
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

Successfully merging this pull request may close these issues.

2 participants