-
Notifications
You must be signed in to change notification settings - Fork 13
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
Offboarding - clean up email references #3379
Conversation
The following changes will be applied to the production Kubernetes cluster upon merge. BE AWARE this may not reveal changes that have been manually applied to the cluster getting undone—applying manual changes to the cluster should be avoided. No manifest changes found for prod. |
@evansiroky: can I remove this DAG completely because everyone in the email list is getting removed? Instead of an |
@vevetron: Which change did I make to do this? How do I not kill Kubernetes cluster? |
I think it might be worth keeping the dag for possible future development. |
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.
LGTM
I'm not sure why this came up, I think everything is fine. Nobody is making manual changes to the kubernetes cluster as far as I know. |
The following changes will be applied to the production Kubernetes cluster upon merge. BE AWARE this may not reveal changes that have been manually applied to the cluster getting undone—applying manual changes to the cluster should be avoided. No manifest changes found for prod. |
Description
Describe your changes and why you're making them. Please include the context, motivation, and relevant dependencies.
Resolves #3370
Type of change
How has this been tested?
Post-merge follow-ups
Document any actions that must be taken post-merge to deploy or otherwise implement the changes in this PR (for example, running a full refresh of some incremental model in dbt). If these actions will take more than a few hours after the merge or if they will be completed by someone other than the PR author, please create a dedicated follow-up issue and link it here to track resolution.