Add content to the Observability Guide#16
Conversation
|
@elasticmachine, run elasticsearch-ci/docs |
|
@elasticmachine, run elasticsearch-ci/docs |
bmorelli25
left a comment
There was a problem hiding this comment.
Good stuff! Here's my first round of feedback.
bmorelli25
left a comment
There was a problem hiding this comment.
Something else I just noticed: We should probably match the file names to the id. Otherwise, we'd just have to rename these files later.
dedemorton
left a comment
There was a problem hiding this comment.
This is great! I read through everything and tested the log steps. I want to get these initial comments added so they don't sit for too long.
I still need to test the other steps and will be submitting any feedback tomorrow.
|
@elasticmachine, run elasticsearch-ci/docs |
bmorelli25
left a comment
There was a problem hiding this comment.
Looking better and better with every iteration!
There was a problem hiding this comment.
These docs are going to make it a lot easier for users to get running without bouncing between multiple books! I went through and tested the content about logs, metrics, and uptime. I didn't test the APM steps, though.
Seeing how much content is taken directly from the Filebeat and Metricbeat quick starts, I'm worried the docs are going to get out of sync. We'll need to have to plan to keep the obs and Beats docs in sync, or bite the bullet and use shared regions. At any rate, looks like we're off to a good start!
@dedemorton I'll add this to our agenda for an Obs sync discussion |
|
What about something like this to replace the main image? I'm not a designer, so I'm more than open to any criticism or feedback. |
|
A docs preview will be available soon: |
|
@bmorelli25 @dedemorton I updated the content on the Observability introduction page. When you have time, could you give it a review? Thanks. |
|
pinging for review: @elastic/observability |
masci
left a comment
There was a problem hiding this comment.
Good stuff! Left some feedback, hope it makes sense :)
* add draft content * Add widgets and update images * add beats root attribute * Fix broken links * Add tab widgets * Minor edits * Updates following reviews * Additional updates * Update image * Update landing page content * Updates after review * Remove attributes
* add draft content * Add widgets and update images * add beats root attribute * Fix broken links * Add tab widgets * Minor edits * Updates following reviews * Additional updates * Update image * Update landing page content * Updates after review * Remove attributes
* add draft content * Add widgets and update images * add beats root attribute * Fix broken links * Add tab widgets * Minor edits * Updates following reviews * Additional updates * Update image * Update landing page content * Updates after review * Remove attributes
* add draft content * Add widgets and update images * add beats root attribute * Fix broken links * Add tab widgets * Minor edits * Updates following reviews * Additional updates * Update image * Update landing page content * Updates after review * Remove attributes
* new links to apm docs yay * do not copy content from enObservabilityIngestTraces to serverlessObservabilityApmGetStarted * testing testing 123 * test with a serverless page that has multiple classic sources * Revert "testing testing 123" This reverts commit bca9ff98059e73651a5bed2935e11060662745ae. * Revert "test with a serverless page that has multiple classic sources" This reverts commit 6af87d486620c7109a23f621871aa0d8dd6681b0.
This PR adds content to the new Observability Guide:
HTML preview
https://observability-docs_16.docs-preview.app.elstc.co/guide/en/observability/master/index.html
Related PRs
elastic/beats#19947
elastic/docs#1901
elastic/apm-server#3976
elastic/apm-server#4006
Related issue
Closes https://github.com/elastic/observability-dev/issues/252