remove comments about previous functionality

Signed-off-by: Scott Rigby <scott@r6by.com>
pull/30570/head
Scott Rigby 7 months ago
parent 3cd6afe845
commit c77f4ec6c4
No known key found for this signature in database
GPG Key ID: C7C6FBB5B91C1155

@ -104,8 +104,6 @@ func (cfg *Configuration) execHook(rl *release.Release, hook release.HookEvent,
// under failed condition. If so, then clear the corresponding resource object in the hook // under failed condition. If so, then clear the corresponding resource object in the hook
if errDeleting := cfg.deleteHookByPolicy(h, release.HookFailed, timeout); err != nil { if errDeleting := cfg.deleteHookByPolicy(h, release.HookFailed, timeout); err != nil {
// We log the error here as we want to propagate the hook failure upwards to the release object. // We log the error here as we want to propagate the hook failure upwards to the release object.
// This is a change in behaviour as the edge case previously would lose the hook error and only
// raise the delete hook error.
log.Printf("error the hook resource on hook failure: %v", errDeleting) log.Printf("error the hook resource on hook failure: %v", errDeleting)
} }
return err return err

Loading…
Cancel
Save