Alex
A few autogenerated files slipped through the cracks. Addressing now. Also going to move the black execution to the pre-commit hooks.
Fixed. All checks passing.
The integration test failures are unrelated to this PR afaict. They both stop logging after `Collecting urllib3>=1.15.1 (from kubeflow-training==1.8.0)`.
@andreyvelich any chance we could expedite this? The changes are all superficial but the surface area is large so it's going to require constant rebasing.
Thanks for the merge, @andreyvelich! Happy to add more hooks and also bring the katib hooks into alignment.
> I think TTL after completion at pod level makes most sense. This can be configured at deployment time. And if we want to override the deployment setting for individual...
Will continue the convo in the design doc.
This is incredibly comprehensive and impressive, @juliusvonkohout and @akagami-harsh. What obstacles do you anticipate for end users upgrading from Minio to Seawead? Does it make sense to provide migration documentation...
Congrats, everyone! Phenomenal (and overdue) enhancement! You should consider submitting a talk on this to the next Kubeflow Summit in Europe.
DSLs all the way down 🤣. It's def unorthodox (our users / data scientists can be quite...creative), but it was supported in V1, is not supported in V2, is trivial...