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
Fix date range sent to Slack from reingestion workflows #4093
Fix date range sent to Slack from reingestion workflows #4093
Conversation
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.
These keyword-arguments are being passed to the
def report_completion( |
…ified reporting function to handle reingestion workflows
Alright changes are made in the new commit. |
How can I check the Slack message locally? I ran Airflow locally and triggered the DAGs but is it possible to trigger the Slack message? |
@BaileyMcKelway since the Slack webhooks are configured for our internal workspace, it wouldn't be possible for you to trigger the message and see it for real. But you can make up a pseudo-preview of the message like this.
Here's an example. This should give you an approximate idea of what the message will look like. |
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.
Thanks again for your contribution @BaileyMcKelway! I have a few idiomatic suggestions for the code, and one requested change regarding the message text. I greatly appreciate the new test!
…n_dag Co-authored-by: Madison Swain-Bowden <bowdenm@spu.edu>
…error handling Co-authored-by: Madison Swain-Bowden <bowdenm@spu.edu>
Alrighty committed suggested changes. 🫡 |
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.
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.
We'll I've already reviewed it and it LGTM, so I should approve it too. Thanks for the contribution @BaileyMcKelway!
Fixes
Fixes #3964 by @stacimc
Description
The create_report_load_completion function has been modified to conditionally add date range parameters to the Slack notification based on whether the workflow is a reingestion process. This prevents misleading date range information from being included in reingestion notifications.
Testing Instructions
Checklist
Update index.md
).main
) or a parent feature branch.Developer Certificate of Origin
Developer Certificate of Origin