Skip to content

Document more source types v2 - #610

Open
jorbaum wants to merge 5 commits into
cloudfoundry:masterfrom
jorbaum:document-more-source-types-v2
Open

Document more source types v2#610
jorbaum wants to merge 5 commits into
cloudfoundry:masterfrom
jorbaum:document-more-source-types-v2

Conversation

@jorbaum

@jorbaum jorbaum commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Follow up of #577 / #609 .

@jorbaum
jorbaum force-pushed the document-more-source-types-v2 branch from fa0559a to b74ae11 Compare August 20, 2026 07:47
I took a deeper look to double check and adjusted as per my findings
@jorbaum
jorbaum force-pushed the document-more-source-types-v2 branch from b74ae11 to 3c71eb1 Compare August 20, 2026 07:48

@chombium chombium left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jorbaum Nice improvements, but I have few comments.

Comment thread deploy-apps/streaming-logs.html.md.erb Outdated
Comment thread deploy-apps/streaming-logs.html.md.erb Outdated
Comment thread deploy-apps/streaming-logs.html.md.erb
By changing the Java log output, you can force your app to reformat stack trace messages, replacing newline
characters with a token. Set your log parsing code to replace that token with newline characters again to display the
logs properly in Kibana. No newline at end of file
logs properly in Kibana.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please add back the new line at the end of the file.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My changes actually added the newline so it follows normal POSIX style. However, I could revert if you like to keep changes more minimal.

@jorbaum

jorbaum commented Aug 21, 2026

Copy link
Copy Markdown
Contributor Author

@chombium thanks for the review. I worked in most of your comments. Marking it ready for review now even though I left the one comment about the newline unresolved.

@jorbaum
jorbaum marked this pull request as ready for review August 21, 2026 09:26
### <a id='health'></a> HEALTH

The Diego Cell emits `HEALTH` logs when running health check actions for an app. These logs are generated by the health check binary that monitors app instances. By default, health check log output is suppressed unless an error occurs.
The Diego Cell emits `HEALTH` logs for health check activity of an app. This inlcudes lifecycle events as well as monitoring start and health-state transitions. By default, most health check log output is suppressed.

@chombium chombium Aug 21, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

on an app=> of an app

The last sentence in the paragraph By default, health check log output is suppressed unless an error occurs. which is deleted should be brought back in as it describes the default behavior

@chombium chombium Aug 21, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add the rest of the sentence unless an error occurs. as well, as the output is written in case of errors.

### <a id='health'></a> HEALTH

The Diego Cell emits `HEALTH` logs when running health check actions for an app. These logs are generated by the health check binary that monitors app instances. By default, health check log output is suppressed unless an error occurs.
The Diego Cell emits `HEALTH` logs for health check activity of an app. This inlcudes lifecycle events as well as monitoring start and health-state transitions. By default, most health check log output is suppressed.

@chombium chombium Aug 21, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add the rest of the sentence unless an error occurs. as well, as the output is written in case of errors.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants