commit 9d168142c83dbfb67ba325d44b1b9fe9d8389583 · veryroundbird.house/core

veryroundbird.house / core

this repo has no description

spindle/engine: run DestroyWorkflow when setup fails

It's possible for SetupWorkflow to provision resources and also fail, so
this makes sense for robustness.

Signed-off-by: Winter <winter@winter.cafe>

winter.bsky.social 4 months ago 9d168142 68f9c1aa

options

unified split

Changed files

spindle

engine

engine.go

spindle/engine/engine.go

···

       54
       54
        
       					// In the original, we only do in a subset of cases.

     

       55
       55
        
       					l.Error("setting up worklow", "wid", wid, "err", err)

     

       56
       56
        
       

     

       57
       57
       +
       					destroyErr := eng.DestroyWorkflow(ctx, wid)

     

       58
       58
       +
       					if destroyErr != nil {

     

       59
       59
       +
       						l.Error("failed to destroy workflow after setup failure", "error", destroyErr)

     

       60
       60
       +
       					}

     

       61
       61
       +
       

     

       57
       62
        
       					dbErr := db.StatusFailed(wid, err.Error(), -1, n)

     

       58
       63
        
       					if dbErr != nil {

     

       59
       64
        
       						return dbErr