Skip to content

Add MXJob v1 api and controller#1296

Merged
google-oss-robot merged 1 commit intokubeflow:all-in-one-operatorfrom
Jeffwan:mxnet
Jul 11, 2021
Merged

Add MXJob v1 api and controller#1296
google-oss-robot merged 1 commit intokubeflow:all-in-one-operatorfrom
Jeffwan:mxnet

Conversation

@Jeffwan
Copy link
Member

@Jeffwan Jeffwan commented Jul 10, 2021

  • Generate MXJob api and manifests
  • Rewrite MXNet controller logic in reconciler pattern

Signed-off-by: Jiaxin Shan [email protected]

@Jeffwan Jeffwan requested review from terrytangyuan and zw0610 and removed request for jinchihe and terrytangyuan July 10, 2021 23:44
@Jeffwan
Copy link
Member Author

Jeffwan commented Jul 10, 2021

/cc @kubeflow/wg-training-leads

- Generate MXJob api and manifests
- Rewrite MXNet controller logic in reconciler pattern

Signed-off-by: Jiaxin Shan <[email protected]>
@Jeffwan
Copy link
Member Author

Jeffwan commented Jul 11, 2021

I just rebase the upstream and resolve the merge conflicts

bin/
/tf-operator
vendor/
testbin/*
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Where is this being generated from?

Copy link
Member Author

@Jeffwan Jeffwan Jul 11, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member

@zw0610 zw0610 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

Copy link
Member

@terrytangyuan terrytangyuan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm
/approve

@Jeffwan
Copy link
Member Author

Jeffwan commented Jul 11, 2021

/approve

@google-oss-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Jeffwan, terrytangyuan, zw0610

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

The pull request process is described here

Details Needs approval from an approver in each of these files:

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

@Jeffwan
Copy link
Member Author

Jeffwan commented Jul 11, 2021

part of kubeflow/common#138 #1298

Jeffwan added a commit that referenced this pull request Aug 5, 2021
- Generate MXJob api and manifests
- Rewrite MXNet controller logic in reconciler pattern

Signed-off-by: Jiaxin Shan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants