fix: update execution logs and states for alerts

[eschutho]

SUMMARY

This change runs alerts that were currently in success or grace period if they are no longer in the grace period. It fixes a bug when two subsequent alerts that should both be triggering were only triggering one.

TESTING INSTRUCTIONS

If you run an alert that will always trigger you should see an execution for each schedule.

ADDITIONAL INFORMATION

• Has associated issue:
• Required feature flags:
• Changes UI
• Includes DB Migration (follow approval process in SIP-59)
  • Migration is atomic, supports rollback & is backwards-compatible
  • Confirm DB migration upgrade and downgrade tested
  • Runtime estimates and downtime expectations provided
• Introduces new feature or API
• Removes existing feature or API

[codecov]

Codecov Report

Merging #16736 (7cb709e) into master (986ed68) will increase coverage by 0.00%.
The diff coverage is 77.77%.

@@           Coverage Diff           @@
##           master   #16736   +/-   ##
=======================================
  Coverage   76.98%   76.98%           
=======================================
  Files        1007     1007           
  Lines       54168    54180   +12     
  Branches     7463     7463           
=======================================
+ Hits        41702    41713   +11     
- Misses      12226    12227    +1     
  Partials      240      240           

Flag	Coverage Δ
hive	81.36% <77.77%> (+0.01%)	⬆️
mysql	81.70% <77.77%> (-0.03%)	⬇️
postgres	81.80% <77.77%> (+0.05%)	⬆️
presto	81.65% <77.77%> (-0.04%)	⬇️
python	82.31% <77.77%> (+<0.01%)	⬆️
sqlite	81.41% <77.77%> (+0.05%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
superset/reports/commands/execute.py	90.83% <77.77%> (+0.60%)	⬆️
superset/db_engine_specs/presto.py	89.95% <0.00%> (-0.42%)	⬇️
superset/charts/schemas.py	100.00% <0.00%> (ø)
superset/utils/pandas_postprocessing.py	84.42% <0.00%> (+0.33%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 986ed68...7cb709e. Read the comment docs.

[eschutho]

/testenv up

[github-actions]

@eschutho Ephemeral environment spinning up at http://54.187.236.65:8080. Credentials are admin/admin. Please allow several minutes for bootstrapping and startup.

[yousoph]

/testenv up FEATURE_ALERT_REPORTS=true

[github-actions]

@yousoph Ephemeral environment spinning up at http://54.244.66.198:8080. Credentials are admin/admin. Please allow several minutes for bootstrapping and startup.

[github-actions]

Ephemeral environment shutdown and build artifacts deleted.

[rosemarie-chiu]

🏷 2021.38