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

JSON Logging #1914

Closed
DWSR opened this issue Mar 17, 2022 · 11 comments · Fixed by #2701
Closed

JSON Logging #1914

DWSR opened this issue Mar 17, 2022 · 11 comments · Fixed by #2701
Labels
docs Needs documentation/release notes enhancement New feature or request

Comments

@DWSR
Copy link

DWSR commented Mar 17, 2022

Summary

Argo Rollouts is one of the few Argo projects that doesn't support JSON logging

Use Cases

When trying to naively parse logs using a log aggregator like Fluentd.


Message from the maintainers:

Impacted by this bug? Give it a 👍. We prioritize the issues with the most 👍.

@DWSR DWSR added the enhancement New feature or request label Mar 17, 2022
@hcelaloner
Copy link

I am not sure since I didn't use it. However, it seems that the controller provides flags to change log format: https://github.com/argoproj/argo-rollouts/blob/master/cmd/rollouts-controller/main.go#L221

Therefore, I don't know whether it is documented or not but passing additional command-line arguments to the rollouts-controller may change its log format.

@harikrongali harikrongali added the docs Needs documentation/release notes label Mar 22, 2022
@perenesenko
Copy link
Member

@DWSR
yes, that's correct --logformat=json will work.
We just need to fix a documentation

hcelaloner added a commit to hcelaloner/argo-rollouts that referenced this issue Mar 22, 2022
@hcelaloner
Copy link

hcelaloner commented Mar 22, 2022

Hi @perenesenko if you do not have any different plans for this one, I can submit a PR similar to ArgoCD server command docs as follows: hcelaloner@67d3900 (may also move this new section to the end of installation.md, if that would be more appropriate?)

Edit: Submitted a PR may work on it or change it based on comments 👍

hcelaloner added a commit to hcelaloner/argo-rollouts that referenced this issue Mar 22, 2022
hcelaloner added a commit to hcelaloner/argo-rollouts that referenced this issue Mar 24, 2022
@AliDotS
Copy link

AliDotS commented Sep 11, 2022

When leader election is enabled there is one line that doesn't follow JSON format.
12:08:56.641662 1 leaderelection.go:248] attempting to acquire leader lease argocd/argo-rollouts-controller-lock...

hcelaloner added a commit to hcelaloner/argo-rollouts that referenced this issue Nov 9, 2022
hcelaloner added a commit to hcelaloner/argo-rollouts that referenced this issue Nov 9, 2022
@github-actions
Copy link
Contributor

This issue is stale because it has been open 60 days with no activity.

@DWSR
Copy link
Author

DWSR commented Nov 11, 2022

Not stale.

@github-actions
Copy link
Contributor

This issue is stale because it has been open 60 days with no activity.

@Sineaggi
Copy link
Contributor

Not stale.

@github-actions
Copy link
Contributor

This issue is stale because it has been open 60 days with no activity.

@DWSR
Copy link
Author

DWSR commented Mar 19, 2023

Not stale.

@Sineaggi
Copy link
Contributor

Sineaggi commented Apr 6, 2023

@AliDotS presumably logs from klog aren't being mapped to logrus.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Needs documentation/release notes enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants