Pass executor template configuration through the ApplicationGroup spec

[nitishm]

Currently the default executor is hardcoded,

func defaultExecutor(timeout time.Duration) v1alpha12.Template {
	executorArgs := []string{"--spec", "{{inputs.parameters.helmrelease}}", "--timeout", timeout.String()}
	return v1alpha12.Template{
		Name:               helmReleaseExecutor,
		ServiceAccountName: workflowServiceAccountName(),
		Inputs: v1alpha12.Inputs{
			Parameters: []v1alpha12.Parameter{
				{
					Name: "helmrelease",
				},
			},
		},
		Executor: &v1alpha12.ExecutorConfig{
			ServiceAccountName: workflowServiceAccountName(),
		},
		Outputs: v1alpha12.Outputs{},
		Container: &corev1.Container{
			Name:  "executor",
			Image: "azureorkestra/executor:v0.1.0",
			Args:  executorArgs,
		},
	}
}

Rather than using a static template we should pass the container configuration through the ApplicationGroup spec.

[nitishm]

@mahalrs will you be willing to take on this? This requires API changes. We can discuss this if you would like on slack

The idea is to replace the static WorkflowExecutor with a dynamically configurable executor through the ApplicationGroup CR.

[nitishm]

@jonathan-innis this would allow us to test any changes to the default workflow executor as well as any new executors without having to modify the orkestra controller source code.

[mahalrs]

@mahalrs will you be willing to take on this? This requires API changes. We can discuss this if you would like on slack

The idea is to replace the static WorkflowExecutor with a dynamically configurable executor through the ApplicationGroup CR.

Let's discuss it on slack.