Skip to content

task policy #2989

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Nov 17, 2021
Merged

task policy #2989

merged 2 commits into from
Nov 17, 2021

Conversation

kakj-go
Copy link
Contributor

@kakj-go kakj-go commented Nov 10, 2021

What type of this PR

/kind feature

What this PR does / why we need it:

The pipeline task execution strategy is added. The new strategy can make the task no longer execute but directly fetch the latest successful result. This function can be used in automated testing, and users can save time to execute the same set of scenarios

Which issue(s) this PR fixes:

ChangeLog

Language Changelog
🇺🇸 English The task of the pipeline increases the configuration of the execution strategy
🇨🇳 中文 流水线的任务增加执行策略的配置

Need cherry-pick to release versions?

@erda-bot
Copy link
Member

Automated cherry pick can ONLY be triggered when this PR is MERGED!

@kakj-go kakj-go requested a review from sfwn November 10, 2021 05:43
@kakj-go kakj-go added pipeline pipeline service feature labels Nov 10, 2021
@kakj-go kakj-go force-pushed the feature/task-policy branch from 21a19bb to 207b869 Compare November 10, 2021 05:44
* add pod name and notIn

* modify pod_name to component_name
@kakj-go kakj-go force-pushed the feature/task-policy branch from 207b869 to 6d23ecb Compare November 10, 2021 05:46
@codecov
Copy link

codecov bot commented Nov 10, 2021

Codecov Report

Merging #2989 (ac6285d) into master (d2121ba) will decrease coverage by 0.27%.
The diff coverage is 59.37%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2989      +/-   ##
==========================================
- Coverage   17.58%   17.30%   -0.28%     
==========================================
  Files        1325     1355      +30     
  Lines      135083   139791    +4708     
==========================================
+ Hits        23750    24192     +442     
- Misses     108887   113124    +4237     
- Partials     2446     2475      +29     
Impacted Files Coverage Δ
apistructs/pipeline_yml.go 82.60% <ø> (ø)
modules/pipeline/pipengine/reconciler/reconcile.go 0.00% <0.00%> (ø)
...ine/pipengine/reconciler/reconcile_data_process.go 1.36% <0.00%> (-0.03%) ⬇️
modules/pipeline/spec/pipeline_task.go 25.97% <ø> (ø)
pkg/parser/pipelineyml/define.go 19.56% <ø> (ø)
pkg/parser/pipelineyml/graph_pipelineyml.go 51.94% <0.00%> (-1.56%) ⬇️
modules/pipeline/pipengine/reconciler/policy.go 71.69% <71.69%> (ø)
apistructs/autotest_space.go 53.84% <0.00%> (-11.38%) ⬇️
apistructs/pipeline_task.go 70.83% <0.00%> (-6.36%) ⬇️
pkg/mock/mock.go 42.42% <0.00%> (-3.04%) ⬇️
... and 79 more

@kakj-go kakj-go force-pushed the feature/task-policy branch from 6d23ecb to 4c6ba53 Compare November 10, 2021 10:04
@kakj-go kakj-go marked this pull request as draft November 10, 2021 11:12
@kakj-go kakj-go marked this pull request as ready for review November 16, 2021 02:11
@kakj-go kakj-go force-pushed the feature/task-policy branch 2 times, most recently from b5676d2 to 69837c9 Compare November 16, 2021 05:41
…ake the task no longer execute but directly take the latest successful result
@kakj-go kakj-go force-pushed the feature/task-policy branch from 69837c9 to ac6285d Compare November 17, 2021 02:45
@sfwn
Copy link
Member

sfwn commented Nov 17, 2021

/approve

@erda-bot erda-bot merged commit 67cdc59 into erda-project:master Nov 17, 2021
shuofan pushed a commit to shuofan/erda that referenced this pull request Nov 17, 2021
* add pod_name (erda-project#2939)

* add pod name and notIn

* modify pod_name to component_name

* Increase the pipeline task execution strategy, the new strategy can make the task no longer execute but directly take the latest successful result

Co-authored-by: panjiayao <[email protected]>
Effet pushed a commit that referenced this pull request Nov 23, 2021
…factor (#3039)

* Scenario auto test scene set up in dop

* add groupID to scene and update their apis

* add move v2 api

* add cors to openapi

* add sceneStage temp

* remove auto-test-scene render of openapi

* add scenesStage render

* fix: typo in cp auto-test-scenes.yaml

* rename cp file issue-stats.yml to issue-dashboard.yml

* fix: update operation of scenesStages

* add gshelper

* add global active config key

* refactor component structure of auto-test-scenes

* Fix auto test scene execution

* add addParallel operation

* copy scenes set group filed

* add copy operation

* add copyTo operation

* refactor auto-test-scenes.yml: remove useless state binds

* fix go mod

* add addScenesButton operations

* remove default rendering

* add update scene operation

* autotest scene set execute table

* add export scene set button

* Fix api editor scene id

* Auto test scene set execution set up

* add edit scene operations

* execute task table

* Remove useless sceneset execute button

* Add refer sceneset

* task policy (#2989)

* add pod_name (#2939)

* add pod name and notIn

* modify pod_name to component_name

* Increase the pipeline task execution strategy, the new strategy can make the task no longer execute but directly take the latest successful result

Co-authored-by: panjiayao <[email protected]>

* cancel button visible

* add click item operation

* set hideHeader true

* recovery pass rate and execution rate statistics of auto test scene

* Scene set refer policy

* 🐛 fix bug

* fix bug

* fix bug

* add group_id migrations

* Component render reaction, openapi forward

* update copy operations

* Fix build error, license check

* add checkSceneSetNotHaveSamePreID

* i18n migration, remove duplicate sql.

Co-authored-by: wangxiaojian <[email protected]>
Co-authored-by: sfwn <[email protected]>
Co-authored-by: joey <[email protected]>
Co-authored-by: kakj <[email protected]>
Co-authored-by: panjiayao <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

4 participants