Skip to content
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

Fix the error when the user did not fill in the .yml when creating the pipeline #2685

Conversation

kakj-go
Copy link
Contributor

@kakj-go kakj-go commented Oct 29, 2021

What type of this PR

/kind bug

What this PR does / why we need it:

An error is reported when the user does not fill in the .yml when creating the pipeline, which is a bad experience for the user

Which issue(s) this PR fixes:

ChangeLog

Language Changelog
🇺🇸 English When the user creates the pipeline, it is automatically spliced ​​when there is no .yml suffix
🇨🇳 中文 用户创建流水线的时候没有.yml后缀时自动拼接上

Need cherry-pick to release versions?

/cherry-pick release/1.4

@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 October 29, 2021 03:52
@codecov
Copy link

codecov bot commented Oct 29, 2021

Codecov Report

Merging #2685 (e0fb2a5) into master (da0feca) will decrease coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2685      +/-   ##
==========================================
- Coverage   16.65%   16.65%   -0.01%     
==========================================
  Files        1263     1262       -1     
  Lines      128389   128349      -40     
==========================================
- Hits        21379    21372       -7     
+ Misses     104798   104744      -54     
- Partials     2212     2233      +21     
Impacted Files Coverage Δ
modules/cmp/cache/cache.go 59.52% <0.00%> (-5.04%) ⬇️
modules/cmp/steve/proxy/proxy_store.go 0.00% <0.00%> (-0.69%) ⬇️
modules/cmp/tasks/collect_daily_quota.go 8.23% <0.00%> (-0.36%) ⬇️
modules/core-services/services/project/project.go 9.93% <0.00%> (-0.05%) ⬇️
modules/cmp/steve/server.go 0.00% <0.00%> (ø)
modules/cmp/steve/aggregator.go 0.00% <0.00%> (ø)
...dules/scheduler/executor/plugins/k8s/deployment.go 29.80% <0.00%> (ø)
modules/cmp/steve/proxy/cache_store.go
.../cmp-dashboard-workload-detail/podsTable/render.go 23.23% <0.00%> (+0.20%) ⬆️
modules/cmp/metrics/metrics.go 24.18% <0.00%> (+0.71%) ⬆️
... and 3 more

@kakj-go kakj-go force-pushed the feature/create-pipeline-yml-auito-append-yml branch from f50b773 to bf15c94 Compare October 29, 2021 07:33
@kakj-go kakj-go force-pushed the feature/create-pipeline-yml-auito-append-yml branch from bf15c94 to e0fb2a5 Compare October 29, 2021 07:50
@Effet
Copy link
Member

Effet commented Oct 29, 2021

/approve

@erda-bot erda-bot merged commit a09adbb into erda-project:master Oct 29, 2021
@kakj-go
Copy link
Contributor Author

kakj-go commented Oct 29, 2021

/cherry-pick release/1.4

erda-bot pushed a commit to erda-bot/erda that referenced this pull request Oct 29, 2021
…e pipeline (erda-project#2685)

* update projectID to projectId (erda-project#2534)

* Assuming that the user did not fill in the .yml suffix, it will be filled in automatically when creating

Co-authored-by: littlejian <17816869670@163.com>
erda-bot added a commit that referenced this pull request Oct 29, 2021
…e pipeline (#2685) (#2693)

* update projectID to projectId (#2534)

* Assuming that the user did not fill in the .yml suffix, it will be filled in automatically when creating

Co-authored-by: littlejian <17816869670@163.com>

Co-authored-by: kakj <18579115540@163.com>
Co-authored-by: littlejian <17816869670@163.com>
@kakj-go kakj-go added the dop devops plaoform label Nov 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved dop devops plaoform
Development

Successfully merging this pull request may close these issues.

4 participants