-
Notifications
You must be signed in to change notification settings - Fork 166
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
Update quick starts for 7.16 #1248
Conversation
@@ -63,7 +63,7 @@ installation package. | |||
-- | |||
// tag::install-elastic-agent[] | |||
|
|||
include::{tab-widgets}/download-widget.asciidoc[] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These {tab-widgets}
attributes are nice, but they don't allow cross-doc includes (as far as I can tell)
A documentation preview will be available soon: |
// this if directive includes the same file, but changes the file path based on where this guide is built | ||
ifndef::apm-integration-docs[] | ||
include::{apm-repo-dir}/legacy/tab-widgets/install-agents-widget.asciidoc[] | ||
endif::[] | ||
|
||
ifdef::apm-integration-docs[] | ||
include::{docdir}/legacy/tab-widgets/install-agents-widget.asciidoc[] | ||
endif::[] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a bit of a mess, but it's the only way I could include this file in both the Observability Guide and APM Guide. Without the if directive, the file would error in one of the books.
// THIS CONTENT IS ALSO USED IN THE APM GUIDE | ||
// tag::apm-quick-start[] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This tag allows the content to be reused in the APM Guide.
* docs: new screenshots * docs: update text * docs: use aboslute file paths instead of attrs * docs: remove extra include
* docs: new screenshots * docs: update text * docs: use aboslute file paths instead of attrs * docs: remove extra include
* docs: new screenshots * docs: update text * docs: use aboslute file paths instead of attrs * docs: remove extra include
Summary
This PR updates the traces quick start and logs/metrics/uptime quick start for 7.16. It also updates the includes so that the traces quick start guide can be reused in the APM Guide in the APM Server repo.
Related
For #1073
For #1231
elastic/docs
PR: elastic/docs#2281elastic/apm-server
PR: elastic/apm-server#6554