-
Notifications
You must be signed in to change notification settings - Fork 706
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
TFJob 1.0 #968
Comments
Sounds good |
SGTM |
Performance issues: #965 |
I can help on the "Scheduling policy" to integrate with related scheduler :) BTW, should "Scheduling policy" also be part of common operator? |
I've included SchedulingPolicy in the common API here: https://github.com/kubeflow/common/blob/master/operator/v1/types.go#L184 |
Discussed with community members - for TFJob 1.0, we will not be migrating to the new common API until the latter has been stabilized and tested with a new operator. Instead the goal will be to release v1 from the v1beta2 implementation, with smaller improvements like enhancing metrics. /cc @johnugeorge |
Update from TFJob 1.0 |
@jlewi I will complete it in 2 days |
We can close this after #965 is fixed. |
Regarding the UI bug: #1019. Should we get eventually rid of or keep the TFJobs dashboard? Would it be better to have a more general UI that isn't specific to TFJob? How far can we get with existing monitoring tools like the K8s dashboard? Should we open up a separate issue to figure out the future of the TFJobs dashboard? |
Related slack conversation: https://kubeflow.slack.com/archives/C985VJN9F/p1560184440012500 |
/close |
@richardsliu: Closing this issue. In response to this:
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. |
Tracking items needed for TFJob 1.0:
/cc @gaocegege /cc @johnugeorge
The text was updated successfully, but these errors were encountered: