D. Ferruzzi
D. Ferruzzi
As Niko said, the "finally" should be executed whether there is an exception or not: 
I think it's perfectly reasonable to not want it in the docs if we can't back up the claim with testing.
@potiuk - your new handy dandy script will update that checklist auto-magically on a schedule, right? You merged Athena yesterday.
Huh. I'm positive I did that but it's definitely still there. I'll do a PR for you in a sec and tag you in it.
Quick update, the following are not checked off above: - airflow/providers/amazon/aws/example_dags/example_emr.py - @syedahsn is working on this one right now. - airflow/providers/amazon/aws/example_dags/example_eks_templated.py - All of the operators in this DAG...
If they get moved there, then they will be run by our CI and always fail and cause alerts. Maybe a subdirectory? @vincbeck what do you think of adding an...
Yeah, I've noticed the same in the AWS package, some new ones have been added since so we'll have to do a "final pass" after the checklist is cleared.
Seems to make sense.
@potiuk - I appreciate the ping. I'm not familiar enough with the Secrets Backend yet to have any useful opinion on this one. At face value, it sounds like a...
If I'm not mistaken, that would require changes to the system test runner which I think currently triggers on the "example" prefix. This was a holdout to when they were...