-
Notifications
You must be signed in to change notification settings - Fork 5
feat: tenant tagging pipeline draft #414
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
Co-authored-by: sandrahoang686 <sandrahoang686@gmail.com>
|
@sandrahoang686 Thanks for the review, I believe I addressed all your feedback. I made one additional change to update the DAG to handle newlines for a collections list since this seems to be the convention that airflow uses for list params. |
ividito
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.
Some nits but no blockers
Co-authored-by: Isayah Vidito <isayah@developmentseed.org>
…A-IMPACT/veda-data-airflow into issue-677/tenant-tagging-dag
Co-authored-by: Isayah Vidito <isayah@developmentseed.org>
Summary:
https://github.com/NASA-IMPACT/veda-architecture/issues/677
Changes
PR Checklist
terraform validateandterraform plan