Fixed handling of failed task

[micahhausler]

Signed-off-by: Micah Hausler mhausler@amazon.com

Description

Failed tasks did not properly update the workflow status, and the overall workflow status perpetually showed as STATE_RUNNING

Why is this needed

To properly report a failed workflow

Fixes: #

How Has This Been Tested?

Added unit tests

How are existing users impacted? What migration steps/scripts do we need?

N/A

Checklist:

I have:

• updated the documentation and/or roadmap (if required)
• added unit or e2e tests
• provided instructions on how to upgrade

[codecov]

Codecov Report

Merging #630 (290fda8) into main (bd1c1c0) will increase coverage by 0.04%.
The diff coverage is 100.00%.

❗ Current head 290fda8 differs from pull request most recent head c8602de. Consider uploading reports for the commit c8602de to get more accurate results

@@            Coverage Diff             @@
##             main     #630      +/-   ##
==========================================
+ Coverage   44.52%   44.57%   +0.04%     
==========================================
  Files          61       61              
  Lines        3519     3520       +1     
==========================================
+ Hits         1567     1569       +2     
+ Misses       1870     1869       -1     
  Partials       82       82              

Impacted Files	Coverage Δ
server/kubernetes_api_workflow.go	30.30% <100.00%> (+1.29%)	⬆️

---

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 bd1c1c0...c8602de. Read the comment docs.