intuit / QuickFabric

A one-stop shop for all management and monitoring of Amazon Elastic Map Reduce (EMR) clusters across different AWS accounts and purposes.
Apache License 2.0
26 stars 17 forks source link

add Ami violation report for weekly reporting email. #25

Open khilawar4 opened 4 years ago

khilawar4 commented 4 years ago

Describe the bug currently ami violation report email comes daily , but for weekly report , ami rotation table is missing.

To Reproduce Steps to reproduce the behavior. For example:

  1. open below scheduler class (com.intuit.quickfabric.schedulers.functions.EMRClusterMetricsSchedulers) in this location. https://github.com/intuit/QuickFabric/tree/master/Middleware/schedulers code
  2. go to this method (generateWeeklyReport() )
  3. Ami violation table is not included, please add that same as daily report email (take a reference from generateDailyReport() method).

Expected behavior We should be able to see ami violation report email for weekly report.

sivaganeshan commented 4 years ago

Please assign this issue to me, will work on this