Skip to content

Correct XGBoostJob CRD group name and add singular name#1313

Merged
k8s-ci-robot merged 2 commits into
masterfrom
terrytangyuan-patch-1
Jun 23, 2020
Merged

Correct XGBoostJob CRD group name and add singular name#1313
k8s-ci-robot merged 2 commits into
masterfrom
terrytangyuan-patch-1

Conversation

@terrytangyuan

Copy link
Copy Markdown
Member

Which issue is resolved by this Pull Request:
Resolves kubeflow/xgboost-operator#94

Description of your changes:

Checklist:

  • Unit tests have been rebuilt:
    1. cd manifests/tests
    2. make generate-changed-only
    3. make test

@kubeflow-bot

Copy link
Copy Markdown
Contributor

This change is Reviewable

@terrytangyuan terrytangyuan changed the title Correct XGBoostJob CRD group name and add singular name [WIP] Correct XGBoostJob CRD group name and add singular name Jun 23, 2020
Signed-off-by: terrytangyuan <terrytangyuan@gmail.com>
@terrytangyuan terrytangyuan changed the title [WIP] Correct XGBoostJob CRD group name and add singular name Correct XGBoostJob CRD group name and add singular name Jun 23, 2020
@terrytangyuan

Copy link
Copy Markdown
Member Author

/cc @johnugeorge

@k8s-ci-robot k8s-ci-robot requested a review from johnugeorge June 23, 2020 16:47
@Jeffwan

Jeffwan commented Jun 23, 2020

Copy link
Copy Markdown
Member

@terrytangyuan

Groups has been changed? Is this regenerated?

@Jeffwan

Jeffwan commented Jun 23, 2020

Copy link
Copy Markdown
Member

/lgtm

@terrytangyuan

Copy link
Copy Markdown
Member Author

@Jeffwan This is for better consistency among the operators and xgboostjob prefix was unnecessary.

@terrytangyuan

Copy link
Copy Markdown
Member Author

/approve

@k8s-ci-robot

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: terrytangyuan

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

@terrytangyuan

Copy link
Copy Markdown
Member Author

/test kubeflow-manifests-presubmit

@k8s-ci-robot k8s-ci-robot merged commit e15922e into master Jun 23, 2020
@juliusvonkohout juliusvonkohout deleted the terrytangyuan-patch-1 branch January 12, 2024 09:25
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.

why the name of xgboost-operator’s group is "xgboostjob.kubeflow.org"

5 participants