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
*: Add CRD creation for TFJob #170
Conversation
0b51ee0
to
4cf061e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall looks pretty good can you sync so we can see the diff relative to your other PR?
@@ -111,6 +111,19 @@ roleRef: | |||
name: tf-job-operator | |||
|
|||
|
|||
--- |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lets just delete deploy_crd.yaml rather than trying to keep it up to date.
Signed-off-by: Ce Gao <ce.gao@outlook.com>
Signed-off-by: Ce Gao <ce.gao@outlook.com>
@jlewi Thanks for your review, I have rebased and PTAL again. |
/test all |
Can you explain the ":*" in the PR title? |
It's a commit convension in coala community, which means the PR changes lots of code in different components. And the second commit |
* remove unused variable viewer-clusterDomain from pipeline/pipelines-viewer/base/kustomization.yaml, regenerate tests * add namespace to kustomization.yaml * update unit-tests
* add mutex to studyjob controller Signed-off-by: YujiOshima <yuji.oshima0x3fd@gmail.com> * use sync.Map Signed-off-by: YujiOshima <yuji.oshima0x3fd@gmail.com> * update only when the instance was changed Signed-off-by: YujiOshima <yuji.oshima0x3fd@gmail.com>
Close #164 , ref kubeflow/training-operator#281
Blocked by #169