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 cli always sends GOMAXPROCS log message to STDERR #20690

Closed
3 tasks done
jmf-mordis opened this issue Nov 7, 2024 · 4 comments
Closed
3 tasks done

ArgoCD cli always sends GOMAXPROCS log message to STDERR #20690

jmf-mordis opened this issue Nov 7, 2024 · 4 comments
Assignees
Labels
bug Something isn't working component:cli Affects the Argo CD CLI version:2.13 Latest confirmed affected version is 2.13

Comments

@jmf-mordis
Copy link

jmf-mordis commented Nov 7, 2024

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

Issue only present in the argocd cli 2.13.0, tested on argocd-darwin-arm64 and argocd-linux-amd64

All argocd cli commands I tried send to STDERR this kind of message:

2024/11/07 10:21:35 maxprocs: Leaving GOMAXPROCS=12: CPU quota undefined

If you check that the STDERR of your argocd cli command is empty in your CI/CD, this new version produces always an issue because it always sends this message to STDERR, you can only rely on the status code from now on. I don't think that this a relevant message to send to STDERR.

I think that it is not an expected change, checking this code and this PR.

Setting the GOMAXPROCS env variable doesn't fix the issue, it just makes the argo cli send this other message to STDERR:

GOMAXPROCS=12 argocd version
2024/11/07 10:57:35 maxprocs: Honoring GOMAXPROCS="12" as set in environment

To Reproduce

Just invoking an argocd version, argocd app list or an argocd app diff is enough to reproduce the issue.

argocd app list
2024/11/07 10:52:34 maxprocs: Leaving GOMAXPROCS=12: CPU quota undefined

Expected behavior

STDERR of the argocd cli is empty then there is no relevant error. (I don't consider the GOMAXPROCS logs a relevant error).

Screenshots

Version

(The issue is also in the argocd version )

 argocd version                                                                                                                                                                          2 ✘ 
2024/11/07 10:54:02 maxprocs: Leaving GOMAXPROCS=12: CPU quota undefined
argocd: v2.13.0+347f221
  BuildDate: 2024-11-04T12:29:51Z
  GitCommit: 347f221adba5599ef4d5f12ee572b2c17d01db4d
  GitTreeState: clean
  GoVersion: go1.22.8
  Compiler: gc
  Platform: darwin/arm64

Logs

Paste any relevant application logs here.
@jmf-mordis jmf-mordis added the bug Something isn't working label Nov 7, 2024
@Andrew-Jayne
Copy link

Also able to replicate

If anyone thinks that it is the OhMyZsh Plugin it is just argoCD CLI itself,

~ argocd version
2024/11/07 20:30:46 maxprocs: Leaving GOMAXPROCS=14: CPU quota undefined
argocd: v2.13.0+347f221
  BuildDate: 2024-11-04T15:31:13Z
  GitCommit: 347f221adba5599ef4d5f12ee572b2c17d01db4d
  GitTreeState: clean
  GoVersion: go1.23.2
  Compiler: gc
  Platform: darwin/arm64
FATA[0000] Argo CD server address unspecified           
➜  ~ argocd
2024/11/07 20:30:55 maxprocs: Leaving GOMAXPROCS=14: CPU quota undefined

@pasha-codefresh pasha-codefresh self-assigned this Nov 8, 2024
@pasha-codefresh
Copy link
Member

This issue exists in release-2.13, but not exists in master, probably we just didnt cherry-pick the fix

@pasha-codefresh
Copy link
Member

#20069

@ishitasequeira
Copy link
Member

Fix has been merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working component:cli Affects the Argo CD CLI version:2.13 Latest confirmed affected version is 2.13
Projects
None yet
Development

No branches or pull requests

5 participants