Support for retries and timeout

[Purushottamlpa]

Hi,

Is there support for retry and orchestrator side timeout option available in Semaphore?

[github-actions]

Hello @Purushottamlpa, thank you for your interest in our work!

If this is a bug report, please provide the minimum viable code and steps to reproduce your issue, otherwise we can not help you.
When submitting a new feature request or suggestion please make sure to check for duplicate issues.

[Jeroen Rinzema]

👋 @Purushottamlpa, Thank you for submitting your issue. Currently is there no retry functionallity build into Semaphore. Flow retries could be implemented through middleware. Could you maybe elaborate a little bit on why you need this feature?

Orchestrator side timeout

It is possible to set request duration timeouts inside the listeners. There is currently no possibility of defining a custom timeout for a specific flow.

[Purushottamlpa]

Thanks @jeroenrinzema. My query is within in a flow, if an HTTP call fails, is there any inbuilt support to put a delay in flow and retry failed service with configurable delay.

[Jeroen Rinzema]

Hi @Purushottamlpa, There is currently no built-in support for such use-case. It should be possible to implement this inside the node caller (https://github.com/jexia/semaphore/blob/master/pkg/flow/caller.go#L77-L90). An additional option has to be introduced inside the specs.Node. This option should be populated inside the HCL syntax to configure the retry policy.

semaphore/pkg/providers/hcl/specs.go

Lines 545 to 566 in eed88a8

 // ParseIntermediateNode parses the given intermediate call to a spec call 
 func ParseIntermediateNode(ctx *broker.Context, dependencies specs.Dependencies, node Resource) (*specs.Node, error) { 
 	call, err := ParseIntermediateCall(ctx, node.Request) 
 	if err != nil { 
 		return nil, err 
 	} 
  
 	rollback, err := ParseIntermediateCall(ctx, node.Rollback) 
 	if err != nil { 
 		return nil, err 
 	} 
  
 	result := specs.Node{ 
 		Type:         specs.NodeIntermediate, 
 		DependsOn:    make(specs.Dependencies, len(node.DependsOn)), 
 		ID:           node.Name, 
 		Name:         node.Name, 
 		Call:         call, 
 		ExpectStatus: node.ExpectStatus, 
 		Rollback:     rollback, 
 		OnError:      &specs.OnError{}, 
 	}