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

ArgoCD Dockerfile contains broken symlink (/app/config/ssh/ssh_known_hosts) #9672

Closed
3 tasks done
JorenZ opened this issue Jun 15, 2022 · 0 comments · Fixed by #9674
Closed
3 tasks done

ArgoCD Dockerfile contains broken symlink (/app/config/ssh/ssh_known_hosts) #9672

JorenZ opened this issue Jun 15, 2022 · 0 comments · Fixed by #9674
Labels
bug Something isn't working cherry-pick/2.4 Candidate for cherry picking into the 2.4 release branch regression Bug is a regression, should be handled with high priority

Comments

@JorenZ
Copy link
Contributor

JorenZ commented Jun 15, 2022

Checklist:

  • I've searched in the docs and FAQ for my answer: https://bit.ly/argocd-faq.
  • I've included steps to reproduce the bug.
  • I've pasted the output of argocd version.

Describe the bug

I noticed that the Dockerfile contains a broken symlink. See https://github.com/argoproj/argo-cd/blob/master/Dockerfile#L66:

  • current: ln -s ssh_known_hosts /etc/ssh/ssh_known_hosts (in WORKDIR /app/config/ssh)
  • proposed: ln -s /app/config/ssh/ssh_known_hosts /etc/ssh/ssh_known_hosts (in WORKDIR /app/config/ssh) instead

The current code effectively creates the symlink /etc/ssh/ssh_known_hosts > /etc/ssh/ssh_known_hosts, which does not work because it loops to itself.

To Reproduce

$ docker run --entrypoint bash --rm -it \
  argoproj/argocd:v2.4.0 \
  -c "cat /etc/ssh/ssh_known_hosts"
cat: /etc/ssh/ssh_known_hosts: Too many levels of symbolic links

Expected behavior

$ docker run --entrypoint bash --rm -it \
  argoproj/argocd:v2.4.0 \
  -c "cat /etc/ssh/ssh_known_hosts"

# no output since empty file

Screenshots

N/A

Version

argocd@b512e45c0d29:~$ argocd version
argocd: v2.4.0+91aefab
  BuildDate: 2022-06-10T17:23:37Z
  GitCommit: 91aefabc5b213a258ddcfe04b8e69bb4a2dd2566
  GitTreeState: clean
  GoVersion: go1.18.3
  Compiler: gc
  Platform: linux/amd64
FATA[0001] Argo CD server address unspecified       

Logs

N/A

@JorenZ JorenZ added the bug Something isn't working label Jun 15, 2022
@crenshaw-dev crenshaw-dev added cherry-pick/2.4 Candidate for cherry picking into the 2.4 release branch regression Bug is a regression, should be handled with high priority labels Jun 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working cherry-pick/2.4 Candidate for cherry picking into the 2.4 release branch regression Bug is a regression, should be handled with high priority
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants