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

[DBX-94127/94128] add 526 backup failure email job #18928

Merged
merged 6 commits into from
Oct 21, 2024

Conversation

SamStuckey
Copy link
Contributor

@SamStuckey SamStuckey commented Oct 17, 2024

Summary

  • This work is behind a feature toggle (flipper): NO (usecase will be)
  • Add service for sending failure emails to veterans. unblocks following work:
    • leverage service in backup submission exhaustion
    • leverage service in backup polling on failure result
  • blocks this PR

Related issue(s)

Testing done

  • New code is covered by unit tests
  • flipper will enable integration testing in staging
  • all objects load and functions tested via local command line

Acceptance criteria

  • I fixed|updated|added unit tests and integration tests for each feature (if applicable).
  • No error nor warning in the console.
  • Events are being sent to the appropriate logging solution
  • Documentation has been updated (link to documentation)
  • No sensitive information (i.e. PII/credentials/internal URLs/etc.) is captured in logging, hardcoded, or specs
  • Feature/bug has a monitor built into Datadog (if applicable)
  • If app impacted requires authentication, did you login to a local build and verify all authenticated routes work as expected
  • I added a screenshot of the developed feature

Copy link

Error: A file (or its parent directories) does not have a CODEOWNERS entry. Please update the .github/CODEOWNERS file and add the entry for the Offending file: app/sidekiq/form526_submission_failure_email_job.rb

Copy link

Error: A file (or its parent directories) does not have a CODEOWNERS entry. Please update the .github/CODEOWNERS file and add the entry for the Offending file: app/sidekiq/form526_submission_failure_email_job.rb

@SamStuckey SamStuckey changed the title [DBX-94127] add 526 failure email job an [DBX-94127/94128] add 526 backup failure email notifications Oct 17, 2024
Copy link

Error: A file (or its parent directories) does not have a CODEOWNERS entry. Please update the .github/CODEOWNERS file and add the entry for the Offending file: app/sidekiq/form526_submission_failure_email_job.rb

Copy link

Error: A file (or its parent directories) does not have a CODEOWNERS entry. Please update the .github/CODEOWNERS file and add the entry for the Offending file: app/sidekiq/form526_submission_failure_email_job.rb

@SamStuckey SamStuckey changed the title [DBX-94127/94128] add 526 backup failure email notifications [DBX-94127/94128] add 526 backup failure email job Oct 17, 2024
Copy link

Error: A file (or its parent directories) does not have a CODEOWNERS entry. Please update the .github/CODEOWNERS file and add the entry for the Offending file: app/sidekiq/form526_submission_failure_email_job.rb

@va-vfs-bot va-vfs-bot temporarily deployed to dbx-94127/add-526-failure-email-job/main/main October 17, 2024 17:47 Inactive
Copy link

Error: A file (or its parent directories) does not have a CODEOWNERS entry. Please update the .github/CODEOWNERS file and add the entry for the Offending file: app/sidekiq/form526_submission_failure_email_job.rb

Copy link

github-actions bot commented Oct 17, 2024

1 Warning
⚠️ This PR changes 272 LoC (not counting whitespace/newlines).

In order to ensure each PR receives the proper attention it deserves, we recommend not exceeding
200. Expect some delays getting reviews.

File Summary

Files

  • .github/CODEOWNERS (+2/-0)

  • app/sidekiq/form526_submission_failure_email_job.rb (+119/-0)

  • config/settings.yml (+1/-0)

  • spec/factories/form526_submissions.rb (+12/-0)

  • spec/sidekiq/form526_submission_failure_email_job_spec.rb (+138/-0)

    Note: We exclude files matching the following when considering PR size:

    *.csv, *.json, *.tsv, *.txt, *.md, Gemfile.lock, app/swagger, modules/mobile/docs, spec/fixtures/, spec/support/vcr_cassettes/, modules/mobile/spec/support/vcr_cassettes/, db/seeds, modules/vaos/app/docs, modules/meb_api/app/docs, modules/appeals_api/app/swagger/, *.bru, *.pdf
    

Big PRs are difficult to review, often become stale, and cause delays.

Generated by 🚫 Danger

@va-vfs-bot va-vfs-bot temporarily deployed to dbx-94127/add-526-failure-email-job/main/main October 17, 2024 18:40 Inactive
@SamStuckey SamStuckey force-pushed the dbx-94127/add-526-failure-email-job branch from ba7f6bf to 957bdf7 Compare October 17, 2024 18:42
Copy link

Error: A file (or its parent directories) does not have a CODEOWNERS entry. Please update the .github/CODEOWNERS file and add the entry for the Offending file: app/sidekiq/form526_submission_failure_email_job.rb

@SamStuckey SamStuckey added disability-experience To manage benefits disability claims experience. disability-compensation Label used for Pull Requests that impact Disability Compensation claims (526EZ) ZSF:Email zero-silent-failures labels Oct 17, 2024
@va-vfs-bot va-vfs-bot temporarily deployed to dbx-94127/add-526-failure-email-job/main/main October 18, 2024 14:58 Inactive
@SamStuckey
Copy link
Contributor Author

Just to confirm, this PR is just putting the class in, it is not putting in anything that actually uses the class yet, right?

If so, just making a note that we may also need to add a silent_failure tag into wherever you are queuing this job, in the case that you fail to queue the job or throw an error when/before queuing it.

correct, subsequent PR leverages this. We can discuss adding that extra tag in that PR : )

@SamStuckey SamStuckey requested a review from kylesoskin October 18, 2024 16:11
@va-vfs-bot va-vfs-bot temporarily deployed to dbx-94127/add-526-failure-email-job/main/main October 18, 2024 18:56 Inactive
Copy link
Contributor

@kylesoskin kylesoskin left a comment

Choose a reason for hiding this comment

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

small spelling nit-pick, but all good!

@SamStuckey SamStuckey marked this pull request as ready for review October 21, 2024 15:10
@SamStuckey SamStuckey requested review from a team as code owners October 21, 2024 15:10
Copy link

Backend-review-group approval confirmed.

@SamStuckey SamStuckey merged commit 815c24b into master Oct 21, 2024
36 of 45 checks passed
@SamStuckey SamStuckey deleted the dbx-94127/add-526-failure-email-job branch October 21, 2024 20:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
disability-compensation Label used for Pull Requests that impact Disability Compensation claims (526EZ) disability-experience To manage benefits disability claims experience. require-backend-approval test-passing zero-silent-failures ZSF:Email
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants