Skip to content
This repository was archived by the owner on Apr 30, 2025. It is now read-only.

Extra line for all log lines #202

Closed
enocom opened this issue Dec 13, 2017 · 4 comments · Fixed by #280
Closed

Extra line for all log lines #202

enocom opened this issue Dec 13, 2017 · 4 comments · Fixed by #280
Labels

Comments

@enocom
Copy link

enocom commented Dec 13, 2017

Issue

When running cf logs, I see an extra log for every router log:

2017-12-13T14:51:41.11-0700 [RTR/10] OUT nn-accumulator.cfapps.io - [2017-12-13T21:51:41.104+0000] "GET / HTTP/1.1" 400 0 0 "-" "curl/7.47.0" "10.10.2.186:55276" "10.10.149.105:61003" x_forwarded_for:"50.232.79.2, 10.10.2.186" x_forwarded_proto:"http" vcap_request_id:"8acf9773-567f-4a33-6499-6cf0136c0c19" response_time:0.005741063 app_id:"fe9163db-24ef-4de8-939d-6c8848f05cfd" app_index:"0" x_b3_traceid:"f551776bbbd789e3" x_b3_spanid:"f551776bbbd789e3" x_b3_parentspanid:"-"
2017-12-13T14:51:41.11-0700 [RTR/10] OUT

Context

This is against PWS.

Steps to Reproduce

Just run cf logs on any running app.

Expected result

The unnecessary log line shouldn't be there.

/cc @bradylove

@cf-gitbot
Copy link

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/153652398

The labels on this github issue will be updated when the story is started.

@CAFxX
Copy link

CAFxX commented Dec 14, 2017

We see the same behaviour on our local CFs (we're currently on 262)

@shubhaat
Copy link
Contributor

I was able to reproduce this issue on the latest routing release candidate. The routing team is currently working on replacing the current routing subsystem with a Istio/Envoy based routing subsystem, so we will not be fixing this issue for now. Keeping it open for future reference.

@zachgersh
Copy link

closing due to age and won't fix. We can search if someone needs to find the extra log line.

reneighbor added a commit that referenced this issue Apr 21, 2021
* This line is necessary in the access log (WriteTo) so they show up  as
distinct lines
* It was redundant in `cf logs` (LogMessage); this was problematic
because it sent extra envelopes through the loggregator system.

Fixes cloudfoundry/routing-release#124
Fixes #202

Co-authored-by: Renee Chu <[email protected]>
jrussett pushed a commit that referenced this issue May 3, 2021
* This line is necessary in the access log (WriteTo) so they show up  as
distinct lines
* It was redundant in `cf logs` (LogMessage); this was problematic
because it sent extra envelopes through the loggregator system.

Fixes cloudfoundry/routing-release#124
Fixes #202

Co-authored-by: Renee Chu <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants