Add `Execution` to the `types.ExecuteRequest` struct

[cobolbaby]

Is your feature request related to a problem? Please describe.

I wish to add Execution metadata to the reportingWriter struct to implement some extended features. However, to achieve this, it is necessary to adjust the structure of the ExecuteRequest. I am not sure whether such a request is acceptable.

Describe the solution you'd like
A clear and concise description of what you want to happen.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Additional context
Add any other context or screenshots about the feature request here.

https://github.com/distribworks/dkron/blob/53864a0cf5567e31305b5b004474fb546062eb09/dkron/grpc_agent.go#L84-L94

[vcastellm]

Can you explain what metadata do you want to include?

[cobolbaby]

Can you explain what metadata do you want to include?

Currently I need to get the following attributes, such as ExecutionId, ExecutionStartTime, ExecutionNodeName.

https://github.com/distribworks/dkron/blob/9982ceb9af37543d5133e506899d245e8fa53b6e/dkron/execution.go#L13-L40

Then pass it to the shell command as a custom environment variable.

func (s *Shell) ExecuteImpl(args *dktypes.ExecuteRequest, cb dkplugin.StatusHelper) ([]byte, error) {
	....

	executionInfo := strings.Split(
		fmt.Sprintf("ENV_JOB_NAME=%s,ENV_EXECUTION_ID=%s,ENV_EXECUTION_NODENAME=%s",
			args.JobName, args.Execution.Id, args.Execution.NodeName), ",")
	env = append(env, executionInfo...)

        cmd, err := buildCmd(command, shell, env, cwd)

[vcastellm]

@cobolbaby sorry for the delay, can you let us know if you're still interested in this?

[cobolbaby]

Yes, this is an important improvement that facilitates integration with other systems.