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.

Sign up for GitHub

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

Jump to bottom

Fix log parsing #882

Closed
wants to merge 7 commits into from
Closed

Fix log parsing #882

wants to merge 7 commits into from

Conversation

hagenmt
Copy link

@hagenmt hagenmt commented Oct 28, 2023 •
edited
Loading

by injecting extra double quotes here it break json logging and causes errors like this in fluentd:

pattern not matched message="{\"timestamp\":\"2023-10-28T14:01:56.949Z\",\"logger\":\"com.wavefront.agent.listeners.AbstractPortUnificationHandler\",\"region\":\"region-x\",\"hostname\":\"xxx-xxxx-xxxx\",\"service\":\"vmc-wavefront-proxy\",\"app_id\":\"vmc-wavefront-proxy\",\"commit\":\"\",\"build\":\"0.1.0-20231004-252-5ca6a9a\",\"level\":\"WARN\",\"thread\":\"epollEventLoopGroup-17-7\",\"marker\":\"\",\"exception\":\"\",\"message\":\"Received line is too long, consider increasing pushListenerMaxReceivedLength; remote: x.x.x.x [30000]; reason: \"frame length (18463928) exceeds the allowed maximum (1048576)\"\",\"mdc\":\"{}\",\"component\":\"\",\"sub_component\":\"\",\"sddc_id\":\"\",\"org_id\":\"\",\"op_id\":\"\"}"

and from the CLI the log looks like this with extra double quotes:

2023-10-28 12:02:37,176 WARN  [AbstractPortUnificationHandler:logWarning] Received line is too long, consider increasing pushListenerMaxReceivedLength; remote: x.x.x.x [30000]; reason: "frame length (18468978) exceeds the allowed maximum (1048576)"

laullon and others added 6 commits June 26, 2023 11:57
@laullon
Proxy V13.0 release (Merge branch 'release-13.x')
6a5dd46
@laullon
MONIT-40095: The README.md example of how to utilize Dokcer with OAut… (
34b21a4 
#858)

* MONIT-40095: The README.md example of how to utilize Dokcer with OAuth apps needs to be improved.

* Update README.md

* Minor change.

---------

Co-authored-by: German Laullon <[email protected]>
@shavidissa
Update README.md ro include logs (#859)
7d4f5d1 
Fix MONIT-36115
@laullon
Proxy V13.2 release (Merge branch 'release-13.x')
0cc2509 
* release-13.x:
  [release] prepare release for proxy-13.2
  update open_source_licenses.txt for release 13.2
  Proxy V13.2 release (Merge branch 'release-13.x') (#867)
  [release] prepare for next development iteration
  [release] prepare release for proxy-13.1
  update open_source_licenses.txt for release 13.1
  Dev (#864)
  [release] prepare for next development iteration
@laullon
Proxy V13.3 release (Merge branch 'release-13.x')
c8b226e
Fix log parsing
d5d43d8
@hagenmt
Copy link
Author

hagenmt commented Oct 28, 2023

@basilisk487 please review

@laullon laullon changed the base branch from master to dev October 31, 2023 22:54
@laullon
Copy link
Contributor

laullon commented Oct 31, 2023

I'm closing this with out merging... @hagenmt please recreate the PR using dev branch and base (you may need to update your fork). Add me or @joannak-vmware as reviewers.

@laullon laullon closed this Oct 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@hagenmt @laullon @shavidissa