Subclassed SlackMessenger for the four different notification channels (DE, DS, SE, PS), with success() and failure() methods that post "Success"/"Failure" to the Slack channel without the full error message text. Replaced all uses of old Slack clients in dataset_ingress_v1.py and s3_tar_received.py. Tests amended.
What
Subclassed
SlackMessengerfor the four different notification channels (DE, DS, SE, PS), withsuccess()andfailure()methods that post "Success"/"Failure" to the Slack channel without the full error message text. Replaced all uses of old Slack clients indataset_ingress_v1.pyands3_tar_received.py. Tests amended.How to review
Sanity check.
Who can review
Anyone.