pytorchjob and Tfjob labels changed

[hougangliu]

kubeflow/training-operator#1064
kubeflow/pytorch-operator#204

Above PRs update pytorchjob and tfjob pod labels, and now katib depends on these labels. For backward compatibility, now katib support both the old and new version of labels.

In future, when pytorchjob and Tfjob repo drops the old one, we should update katib related code, too.

[gaocegege]

Should we fix it before 0.7? @johnugeorge @hougangliu

[johnugeorge]

In this release, katib will use older labels while job operators supports both(old and new) labels.

Post 0.7 release, katib should use newer labels. We can keep this PR open to track this.

[gaocegege]

SGTM

[andreyvelich]

@johnugeorge @gaocegege Should we close this, I think we have new labels: https://github.com/kubeflow/katib/blob/master/pkg/common/v1alpha3/common.go#L22?

[gaocegege]

/close

[k8s-ci-robot]

@gaocegege: Closing this issue.

In response to this:

/close

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.