Skip to content

Conversation

alexmt
Copy link
Collaborator

@alexmt alexmt commented Aug 23, 2021

Signed-off-by: Alexander Matyushentsev AMatyushentsev@gmail.com

Closes argoproj/argo-helm#882
Closes #7044

I think renaming of uid_entrypoint.sh to entrypoint.sh should be considered a backward-incompatible change. To make it easier to migrate let's create uid_entrypoint.sh symlink and delete it in next release.

Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
@alexmt alexmt requested a review from jannfis August 23, 2021 00:19
@alexmt
Copy link
Collaborator Author

alexmt commented Aug 23, 2021

@jannfis WDYT?

@codecov
Copy link

codecov bot commented Aug 23, 2021

Codecov Report

Merging #7047 (31acdd8) into master (d12eafa) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #7047   +/-   ##
=======================================
  Coverage   40.98%   40.98%           
=======================================
  Files         158      158           
  Lines       21305    21305           
=======================================
  Hits         8731     8731           
  Misses      11334    11334           
  Partials     1240     1240           
Impacted Files Coverage Δ
util/settings/settings.go 47.38% <0.00%> (ø)

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 d12eafa...31acdd8. Read the comment docs.

Copy link
Member

@jannfis jannfis left a comment

Choose a reason for hiding this comment

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

LGTM, good solution with the symlink. Also thanks for fixing the other Dockerfiles, good catches.

I should have applied more care when merging/cherry-picking the original change.

@jannfis jannfis merged commit 613db27 into argoproj:master Aug 23, 2021
@jannfis jannfis added the cherry-pick/2.1 Candidate for cherry picking into the 2.1 release branch label Aug 23, 2021
jannfis pushed a commit that referenced this pull request Aug 23, 2021
Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
plakyda-codefresh pushed a commit to plakyda-codefresh/argo-cd that referenced this pull request Sep 28, 2021
Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
Signed-off-by: viktorplakida <plakyda1@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cherry-pick/2.1 Candidate for cherry picking into the 2.1 release branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ArgoCD upgrade v2.0.5 to v2.1.0 cause argocd-repo-server CrashLoopBackOff argocd version v2.1.0 breaks repo-server deployment
2 participants