-
Notifications
You must be signed in to change notification settings - Fork 4
DTOSS-11768 Remove message status queue #838
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
dba5ba1 to
a22c433
Compare
|
The review app at this URL has been deleted: |
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. The coupling between queues and collecting metrics has proven to be a bit brittle but I think this covers the issue.
Beginning to wonder if there is any value in preserving the collect metrics job as it only measures queues we are no longer using and given Django has a new background task feature in v 6 we will likely not use Azure Storage Queues in the forseeable.
We no longer need this queue and will be deleting it
a22c433 to
cd37c0d
Compare
|
@steventux Oh that's interesting about the background task feature! |



Description
Now that we have removed the
smsjob 251bf4f we can remove the associated queue and its monitoring.Jira link
Review notes
Review checklist