[Discussion] The future of Trial metadata parameters.

[Electronic-Waste]

As we discuss in issue #2369 , we choose to always pass the Trial name as env variable KATIB_TRIAL_NAME to the training container. And we implemented this feature as function mutatePodEnv and used it here:

katib/pkg/webhook/v1beta1/pod/inject_webhook.go

Lines 140 to 142 in 206fe1c

	// Add Katib Trial labels to the Pod metadata.
	mutatePodMetadata(mutatedPod, trial)

Here are some discussions about it: #2369 (comment) #2369 (comment) #2369 (comment)

We use this issue to track the discussion. Any suggestions are welcome!

[github-actions]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[andreyvelich]

/remove-lifecycle stale
/good-first-issue

[google-oss-prow]

@andreyvelich:
This request has been marked as suitable for new contributors.

Please ensure the request meets the requirements listed here.

If this request no longer meets these requirements, the label can be removed
by commenting with the /remove-good-first-issue command.

In response to this:

/remove-lifecycle stale
/good-first-issue

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.