Skip to content

Conversation

@jieguangzhou
Copy link
Member

Purpose of the pull request

Overview

Kubeflow task type is used to create tasks on Kubeflow.

The backend mainly uses the kubectl command to create kubeflow tasks, and continues to monitor the resource status on Kubeflow until the task is completed.

Now it mainly supports creating kubeflow tasks using yaml files. If you need to publish kubeflow pipeline tasks, you can use the python task type.

Here are some specific parameters for the Kubeflow plugin

  • Namespace:The namespace parameter of the cluster
  • yamlContent:CRD YAML file content

close: #12166

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.

(or)

This pull request is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows:

(or)

If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

@github-actions github-actions bot added backend document UI ui and front end related labels Nov 9, 2022
@mergeable mergeable bot removed UI ui and front end related backend document labels Nov 9, 2022
@jieguangzhou jieguangzhou self-assigned this Nov 9, 2022
@jieguangzhou jieguangzhou added the 3.2.0 for 3.2.0 version label Nov 9, 2022
@jieguangzhou jieguangzhou added this to the 3.2.0 milestone Nov 9, 2022
@jieguangzhou jieguangzhou added feature new feature UI ui and front end related backend and removed 3.2.0 for 3.2.0 version labels Nov 9, 2022
@jieguangzhou jieguangzhou force-pushed the kubeflow branch 2 times, most recently from 0726ba3 to a2b4863 Compare November 10, 2022 02:26
@codecov-commenter
Copy link

codecov-commenter commented Nov 10, 2022

Codecov Report

Merging #12843 (87789f7) into dev (6896fa9) will increase coverage by 0.09%.
The diff coverage is 80.00%.

@@             Coverage Diff              @@
##                dev   #12843      +/-   ##
============================================
+ Coverage     39.15%   39.25%   +0.09%     
- Complexity     4214     4230      +16     
============================================
  Files          1046     1051       +5     
  Lines         39740    39849     +109     
  Branches       4574     4585      +11     
============================================
+ Hits          15562    15644      +82     
- Misses        22407    22432      +25     
- Partials       1771     1773       +2     
Impacted Files Coverage Δ
...r/server/master/runner/task/BaseTaskProcessor.java 16.73% <0.00%> (ø)
...olphinscheduler/plugin/task/api/TaskConstants.java 0.00% <0.00%> (ø)
...scheduler/plugin/kubeflow/KubeflowTaskChannel.java 0.00% <0.00%> (ø)
...er/plugin/kubeflow/KubeflowTaskChannelFactory.java 0.00% <0.00%> (ø)
...lphinscheduler/plugin/kubeflow/KubeflowHelper.java 86.84% <86.84%> (ø)
...dolphinscheduler/plugin/kubeflow/KubeflowTask.java 90.00% <90.00%> (ø)
...nscheduler/plugin/kubeflow/KubeflowParameters.java 100.00% <100.00%> (ø)
...erver/master/processor/queue/TaskEventService.java 69.64% <0.00%> (-10.72%) ⬇️
...er/master/dispatch/host/assign/RandomSelector.java 77.77% <0.00%> (-5.56%) ⬇️
...r/plugin/task/sqoop/parameter/SqoopParameters.java 52.38% <0.00%> (+1.19%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 25 Code Smells

81.0% 81.0% Coverage
0.0% 0.0% Duplication

Copy link
Contributor

@caishunfeng caishunfeng left a comment

Choose a reason for hiding this comment

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

LGTM overall

@zhongjiajie zhongjiajie merged commit 229c554 into apache:dev Nov 11, 2022
@zhongjiajie zhongjiajie changed the title [Feature][Task Plugin] Add Kubeflow task plugin for MLOps scenario #12166 [feature][task] Add Kubeflow task plugin for MLOps scenario Nov 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend document feature new feature UI ui and front end related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature][Task Plugin] Add Kubeflow task plugin for MLOps scenario

5 participants