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 doesn't handle malformed yaml files gracefully #3527

Closed
3 tasks done
andoriyu opened this issue Apr 30, 2020 · 1 comment
Closed
3 tasks done

ArgoCD doesn't handle malformed yaml files gracefully #3527

andoriyu opened this issue Apr 30, 2020 · 1 comment
Labels
bug/in-triage This issue needs further triage to be correctly classified bug Something isn't working

Comments

@andoriyu
Copy link

Checklist:

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

Describe the bug

ArgoCD Application with auto-sync and pruning enabled. Someone checked in malformed yaml - indentations was wrong. ArgoCD silently-ish ignored that error and pruned the resources since it wasn't in "new state".

By silently ignored I mean the issue was only visible in logs and not in dashboard.

To Reproduce

  • Create ArgoCD application with auto-sync and auto-prune enabled
  • Add resources
  • "break" the yaml file
  • see resources get wiped

Expected behavior

ArgoCD would not try to apply such changes if yaml file is malformed.

Version

argocd: v1.3.6+89be1c9
  BuildDate: 2019-12-10T22:46:45Z
  GitCommit: 89be1c9ce6db0f727c81277c1cfdfb1e385bf248
  GitTreeState: clean
  GoVersion: go1.12.6
  Compiler: gc
  Platform: linux/amd64
argocd-server: v1.3.6+89be1c9
  BuildDate: 2019-12-10T22:47:48Z
  GitCommit: 89be1c9ce6db0f727c81277c1cfdfb1e385bf248
  GitTreeState: clean
  GoVersion: go1.12.6
  Compiler: gc
  Platform: linux/amd64
  Ksonnet Version: v0.13.1
  Kustomize Version: Version: {Version:kustomize/v3.2.1 GitCommit:d89b448c745937f0cf1936162f26a5aac688f840 BuildDate:2019-09-27T00:10:52Z GoOs:linux GoArch:amd64}
  Helm Version: v2.15.2
  Kubectl Version: v1.14.0

Logs

error: error parsing kafka.yaml: error converting YAML to JSON: yaml: line 207: did not find expected key
@andoriyu andoriyu added the bug Something isn't working label Apr 30, 2020
@jannfis jannfis added the bug/in-triage This issue needs further triage to be correctly classified label May 2, 2020
@todaywasawesome
Copy link
Contributor

Closed with #2267

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug/in-triage This issue needs further triage to be correctly classified bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants