Update failure notifications to have details in a structured format

[namitad]

Currrently the email and slack notification sent out on pipeline failure contains a text content with the error message as a plain string.

update both functions to send failure notification in the below format:

database id: <identifier name>
snapshot id: <snapshot identifier name>
failed step: <task name>
cause of failure: <error message>

[WallysFerreira]

I would like to work on this.

[namitad]

@WallysFerreira have assigned it to you.

[WallysFerreira]

This is my first contribution and I want to make sure I'm on the right track to implement this.

I need to change the expected_message in the 'test_lambda_handler_Error' test to match the new format then in the lambda_handler I need to get the database id, snapshot id, task name and error message from the event and add it to the message.

Is this right or did I not understand it correctly?

[namitad]

@WallysFerreira yes thats correct. Please ensure you are making this change for both the email and slack notification lambdas

[stationeros]

@namitad Can we add the executionId from the context object in the email as well, it would help pinpoint the exact execution ARN which failed , if not in this probably we can add an issue later this is merged

[namitad]

yes @stationeros that would be good addition and since it might involve changes in the overall state machine, i would prefer to have that as a independent issue.

[github-actions[bot]]

This pull request has been automatically closed because it has been inactive for more than 14 days. Please reopen if you still intend to submit this pull request.