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.

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

docs: update APM links #1181

Merged
merged 3 commits into from
Oct 29, 2021
Merged

Conversation

bmorelli25
Copy link
Member

@bmorelli25 bmorelli25 requested a review from a team as a code owner October 19, 2021 19:30
@bmorelli25 bmorelli25 self-assigned this Oct 19, 2021
Comment on lines -15 to -17
Don't forget to also read about
{apm-server-ref-v}/securing-apm-server.html[securing APM Server], and
{apm-server-ref-v}/monitoring.html[monitoring APM Server].
Copy link
Member Author

@bmorelli25 bmorelli25 Oct 19, 2021

Choose a reason for hiding this comment

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

Removed as they don't apply to APM Server in 7.16+. More content rework will need to be done in a future PR.

@apmmachine
Copy link
Contributor

apmmachine commented Oct 19, 2021

A documentation preview will be available soon:

Copy link
Contributor

@EamonnTP EamonnTP left a comment

Choose a reason for hiding this comment

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

LGTM!

@bmorelli25
Copy link
Member Author

@elasticmachine, run elasticsearch-ci/docs

1 similar comment
@bmorelli25
Copy link
Member Author

@elasticmachine, run elasticsearch-ci/docs

@bmorelli25 bmorelli25 merged commit d91404a into elastic:master Oct 29, 2021
@bmorelli25 bmorelli25 deleted the update-apm-links branch October 29, 2021 21:20
bmorelli25 added a commit to bmorelli25/observability-docs that referenced this pull request Oct 29, 2021
* docs: update APM links

* docs: change includes
bmorelli25 added a commit to bmorelli25/observability-docs that referenced this pull request Oct 29, 2021
* docs: update APM links

* docs: change includes
bmorelli25 added a commit that referenced this pull request Oct 29, 2021
* docs: update APM links

* docs: change includes
bmorelli25 added a commit that referenced this pull request Oct 29, 2021
* docs: update APM links

* docs: change includes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants