Closed jantman closed 4 years ago
c7n-mailer is broken in the current release of this project. This is due to an upstream bug, https://github.com/cloud-custodian/cloud-custodian/issues/5750. We have a fix PR opened upstream ( https://github.com/cloud-custodian/cloud-custodian/pull/5751 ) and are waiting for review and merge. Once that PR is merged upstream, we'll bump our c7n and c7n-mailer installs to the latest upstream master branch, which includes the fix.
In addition, I've added unit tests both in my upstream PR and directly in manheim-c7n-tools to catch issues like this (with mailer) in the future.
c7n-mailer is broken in the current release of this project. This is due to an upstream bug, https://github.com/cloud-custodian/cloud-custodian/issues/5750. We have a fix PR opened upstream ( https://github.com/cloud-custodian/cloud-custodian/pull/5751 ) and are waiting for review and merge. Once that PR is merged upstream, we'll bump our c7n and c7n-mailer installs to the latest upstream master branch, which includes the fix.
In addition, I've added unit tests both in my upstream PR and directly in manheim-c7n-tools to catch issues like this (with mailer) in the future.