Incorrect error formatting in trial controller package

[ruskaruma]

What happened?

Three error formatting issues in pkg/controller.v1beta1/trial/:

1. trial_controller_util.go:229 uses %e (scientific notation for floats) instead of %w. Produces garbage output like &{%!e(string=connection refused)} on timestamp parse errors.

2. trial_controller.go:61-63 sentinel errors use fmt.Errorf() instead of errors.New(). These are compared with errors.Is() so they should be immutable sentinels

3. trial/util/job_util.go:82,103 uses %v instead of %w, breaking the error chain for errors.Is() and errors.As() callers

What did you expect to happen?

Errors should use correct format verbs and follow Go error handling conventions.

Environment

Spotted while reading through the trial controller code

Impacted by this bug?

Give it a 👍 We prioritize the issues with most 👍

[github-actions]

🎉 Welcome to the Kubeflow Katib repo! 🎉

Thanks for opening your first issue! We're happy to have you as part of our community 🚀

Next steps:

• Our team will review your issue soon! cc @kubeflow/wg-automl-leads
• Check out the Contributing Guide and the Kubeflow Contributor Guide
• Join the Kubeflow Slack channels: https://www.kubeflow.org/docs/about/community/#kubeflow-slack-channels
• Join the AutoML & Training WG meetings: https://bit.ly/2PWVCkV

Feel free to ask questions in the comments. Thanks again for contributing! 🙏