Replace reconciler implementation with kubeflow/common JobController

[Jeffwan]

In #1144, we upgrade Kubernetes to v1.16.x and update kubeflow/common to v0.3.1 which is good.
The next step, we plan to replace reconciler implementation with kubeflow/common's JobController.

in v0.3.1, I expose ReconcileJob, ReconcilePod and ReconcileService methods and if TF can not use common implementation, it's fine to override the methods and customize logic.

[issue-label-bot]

Issue-Label Bot is automatically applying the labels:

Label	Probability
feature	0.87

Please mark this comment with 👍 or 👎 to give our bot feedback!
Links: app homepage, dashboard and code for this bot.

[gaocegege]

I think it is a high-priority feature for the next release.

/priority p1

[gaocegege]

@Jeffwan

Do you have the bandwidth for this enhancement?

[Jeffwan]

@gaocegege

Yeah. I can definitely help on that. @ChanYiLin Said he is interested in it. I will check with him and see if he's comfortable to make the change.

[gaocegege]

Thanks

/assign @Jeffwan @ChanYiLin

[terrytangyuan]

Any concrete plan on this yet? @Jeffwan @ChanYiLin

[ChanYiLin]

I will start working on it this week, it might take 1~2 weeks
and maybe @Jeffwan can help me to check it functions well the end of next week?

[Jeffwan]

@ChanYiLin Sounds good. Feel free to pin me for any help.

[stale]

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.

[terrytangyuan]

Should we close this now?

[gaocegege]

I think we can /cc @ChanYiLin

[ChanYiLin]

Yes, we can close this now.
This issue has been resolved here #1171