[PaddlePaddle] support paddlejob

[kuizhiqing]

What this PR does / why we need it:

Since stand-alone operator is not preferred, we integrate with the unified one. Discussion goes here kubeflow/community#520.

Thanks to the scaffold and PyTorch implementation, this implementation profits a lot.

Checklist:

• [v] Docs included if any changes are user facing

[johnugeorge]

Awesome! Thanks for this @kuizhiqing

[coveralls]

Pull Request Test Coverage Report for Build 3383985883

• 324 of 894 (36.24%) changed or added relevant lines in 9 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage decreased (-0.5%) to 39.205%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
pkg/controller.v1/register_controller.go	0	3	0.0%
pkg/apis/kubeflow.org/v1/paddlepaddle_validation.go	27	32	84.38%
pkg/apis/kubeflow.org/v1/zz_generated.defaults.go	0	10	0.0%
pkg/controller.v1/paddlepaddle/envvar.go	83	112	74.11%
pkg/apis/kubeflow.org/v1/paddlepaddle_defautls.go	0	46	0.0%
pkg/apis/kubeflow.org/v1/zz_generated.deepcopy.go	0	99	0.0%
pkg/controller.v1/paddlepaddle/paddlepaddle_controller.go	210	372	56.45%
pkg/apis/kubeflow.org/v1/openapi_generated.go	0	216	0.0%

Totals
Change from base Build 3380673253:	-0.5%
Covered Lines:	2662
Relevant Lines:	6790

---

💛 - Coveralls

[terrytangyuan]

Great to see this! Let us know when it's ready for review.

[johnugeorge]

Can you add one e2e test in https://github.com/kubeflow/training-operator/tree/master/sdk/python/test/e2e?

[kuizhiqing]

Can you add one e2e test in https://github.com/kubeflow/training-operator/tree/master/sdk/python/test/e2e?

@johnugeorge , e2e test is added and tested locally, plz help rerun the CI.

[johnugeorge]

e2e tests failed. Can you try this locally ?

[kuizhiqing]

e2e tests failed. Can you try this locally ?

I did pass the e2e test locally,


while the CI test errors related to the last 2 commits are quite confused.

@johnugeorge could you give some tips to handle those errors, thx.

[johnugeorge]

Can you fix the tests?

[johnugeorge]

/lgtm
/assign @terrytangyuan

[gaocegege]

Do we need this const?

[kuizhiqing]

Thx.
Yes, they are referred in the expectation and logger part.

[gaocegege]

Thanks for your contribution! 🎉 👍

[kuizhiqing]

@johnugeorge plz help with the CI test.

[kuizhiqing]

@terrytangyuan , PTAL

[terrytangyuan]

/lgtm
/approve

[google-oss-prow]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: kuizhiqing, terrytangyuan

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [terrytangyuan]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment