23 - the cleanup_notify config parameter was required but undocumented. This PR documents it, and also makes it optional with a default of an empty list.
24 - we previously required the us-east-1 region to be configured and run first out of all regions. The only reason I could find for this (aside from it's where we happen to run mailer, but that's just a convention of ours internally) is we used it to ensure that we only run the Policygen and Docs Generation steps once, in the first region. However, we could instead just run them in the first-configured region (first config.regions list item). That's what this PR does.
Description
This PR rolls up fixes for two issues:
23 - the
cleanup_notifyconfig parameter was required but undocumented. This PR documents it, and also makes it optional with a default of an empty list.24 - we previously required the
us-east-1region to be configured and run first out of all regions. The only reason I could find for this (aside from it's where we happen to run mailer, but that's just a convention of ours internally) is we used it to ensure that we only run the Policygen and Docs Generation steps once, in the first region. However, we could instead just run them in the first-configured region (firstconfig.regionslist item). That's what this PR does.Testing Done
Unit tests.