-
Notifications
You must be signed in to change notification settings - Fork 4
DTOSS-11923: remove retry queue #865
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
Conversation
42f8fdf to
041c161
Compare
|
The review app at this URL has been deleted: |
56af6df to
76b6389
Compare
steventux
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is OK, we can reinstate the collect_metrics job if we need it. Better to document and remove than have useless resources like storage queues hanging around.
We no longer need this because we will not be sending messages to Notify
As we are no longer reporting any metrics from notifications.
This includes the alert on queue sizes because there are no more queues
To leave as examples in case code gets deleted/forgotten.
76b6389 to
0d9a65d
Compare
|



Description
Removes some of the work originally introduced to collect and alert on metrics such as: #617
As we do not have any queues for the Notifications code anymore
Jira link
https://nhsd-jira.digital.nhs.uk/browse/DTOSS-11923
Review notes
Review checklist