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: Restore default pod name version to v1 #6998

Merged
merged 2 commits into from
Oct 21, 2021

Conversation

JPZ13
Copy link
Member

@JPZ13 JPZ13 commented Oct 20, 2021

Signed-off-by: J.P. Zivalich [email protected]

This PR:

  • restores the pod names to v1

Given some of the edge case issues (#6989, #6947) with the default v2 pod names, switch the default back to v1

@codecov
Copy link

codecov bot commented Oct 20, 2021

Codecov Report

Merging #6998 (91d632c) into master (6384e5f) will decrease coverage by 0.06%.
The diff coverage is 11.11%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #6998      +/-   ##
==========================================
- Coverage   48.52%   48.45%   -0.07%     
==========================================
  Files         265      265              
  Lines       19272    19273       +1     
==========================================
- Hits         9352     9339      -13     
- Misses       8867     8885      +18     
+ Partials     1053     1049       -4     
Impacted Files Coverage Δ
workflow/util/pod_name.go 46.66% <11.11%> (-26.67%) ⬇️
server/workflow/workflow_server.go 44.41% <0.00%> (-2.40%) ⬇️
workflow/controller/operator.go 71.13% <0.00%> (-0.10%) ⬇️
server/auth/sso/sso.go 27.27% <0.00%> (+0.87%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6384e5f...91d632c. Read the comment docs.

@JPZ13 JPZ13 marked this pull request as ready for review October 21, 2021 00:16
@sarabala1979 sarabala1979 merged commit 86ddda5 into argoproj:master Oct 21, 2021
@JPZ13 JPZ13 deleted the restore-pod-name-default branch October 21, 2021 01:19
@sarabala1979 sarabala1979 mentioned this pull request Oct 21, 2021
24 tasks
sarabala1979 pushed a commit that referenced this pull request Oct 21, 2021
* fix: Restore default pod name version to v1

Signed-off-by: J.P. Zivalich <[email protected]>

* Update ui test

Signed-off-by: J.P. Zivalich <[email protected]>
kriti-sc pushed a commit to kriti-sc/argo-workflows that referenced this pull request Oct 24, 2021
* fix: Restore default pod name version to v1

Signed-off-by: J.P. Zivalich <[email protected]>

* Update ui test

Signed-off-by: J.P. Zivalich <[email protected]>
Signed-off-by: kriti-sc <[email protected]>
@alexec alexec mentioned this pull request Nov 5, 2021
25 tasks
@sarabala1979 sarabala1979 mentioned this pull request Dec 15, 2021
73 tasks
@sarabala1979 sarabala1979 mentioned this pull request Mar 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants