Remove support for MXJob

[tariq-hasan]

What this PR does / why we need it:

This PR addresses the deprecation of MXJob from Training Operator.

Related PR: #1953

Which issue(s) this PR fixes (optional, in Fixes #<issue number>, #<issue number>, ... format, will close the issue(s) when PR gets merged):
Fixes #1996

Checklist:

• Docs included if any changes are user facing

[coveralls]

Pull Request Test Coverage Report for Build 9655675174

Details

• 1 of 2 (50.0%) changed or added relevant lines in 2 files are covered.
• 1 unchanged line in 1 file lost coverage.
• Overall coverage decreased (-0.5%) to 34.9%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
cmd/training-operator.v1/main.go	0	1	0.0%

Files with Coverage Reduction	New Missed Lines	%
pkg/controller.v1/mpi/mpijob.go	1	91.06%

Totals
Change from base Build 9543811102:	-0.5%
Covered Lines:	3944
Relevant Lines:	11301

---

💛 - Coveralls

[coveralls]

Pull Request Test Coverage Report for Build 9655678457

Details

• 1 of 2 (50.0%) changed or added relevant lines in 2 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage decreased (-0.4%) to 34.926%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
cmd/training-operator.v1/main.go	0	1	0.0%

Totals
Change from base Build 9543811102:	-0.4%
Covered Lines:	3947
Relevant Lines:	11301

---

💛 - Coveralls

[andreyvelich]

Thank you for working on this @tariq-hasan!
We should merge this after Training Operator 1.8 release is finalized.
/hold

[andreyvelich]

I think, we should be ready to merge it since Training Operator 1.8 was released.
@tariq-hasan please can you rebase your PR ?

[coveralls]

Pull Request Test Coverage Report for Build 10121292822

Details

• 1 of 2 (50.0%) changed or added relevant lines in 2 files are covered.
• 2 unchanged lines in 1 file lost coverage.
• Overall coverage decreased (-0.5%) to 34.908%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
cmd/training-operator.v1/main.go	0	1	0.0%

Files with Coverage Reduction	New Missed Lines	%
pkg/controller.v1/mpi/mpijob_controller.go	2	81.05%

Totals
Change from base Build 10049707114:	-0.5%
Covered Lines:	3945
Relevant Lines:	11301

---

💛 - Coveralls

[terrytangyuan]

Thank you!

[terrytangyuan]

/lgtm

[tariq-hasan]

Looking into the failed integration tests now.

[tariq-hasan]

/rerun-workflow "integration test"

[tariq-hasan]

I updated manifests/base/webhook/patch.yaml to fix the integration tests.

If we re-run the integration tests all of them should pass now.

[tenzen-y]

Thanks!
/lgtm
/approve

[google-oss-prow]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: tenzen-y

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 [tenzen-y]

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

[tenzen-y]

/hold cancel