-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[APM] docs: update agent instructions #85855
Conversation
Pinging @elastic/apm-ui (Team:apm) |
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.
🎉
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.
Just want to take out the (default: production)
since we're not actually setting a default in the agent config, here we're just suggesting in the snippets that you use something in the config. We'll switch to enforce production
as default in a later implementation, we're just not there yet.
src/plugins/apm_oss/server/tutorial/instructions/apm_agent_instructions.ts
Outdated
Show resolved
Hide resolved
src/plugins/apm_oss/server/tutorial/instructions/apm_agent_instructions.ts
Outdated
Show resolved
Hide resolved
src/plugins/apm_oss/server/tutorial/instructions/apm_agent_instructions.ts
Outdated
Show resolved
Hide resolved
src/plugins/apm_oss/server/tutorial/instructions/apm_agent_instructions.ts
Outdated
Show resolved
Hide resolved
src/plugins/apm_oss/server/tutorial/instructions/apm_agent_instructions.ts
Outdated
Show resolved
Hide resolved
src/plugins/apm_oss/server/tutorial/instructions/apm_agent_instructions.ts
Outdated
Show resolved
Hide resolved
src/plugins/apm_oss/server/tutorial/instructions/apm_agent_instructions.ts
Outdated
Show resolved
Hide resolved
src/plugins/apm_oss/server/tutorial/instructions/apm_agent_instructions.ts
Outdated
Show resolved
Hide resolved
src/plugins/apm_oss/server/tutorial/instructions/apm_agent_instructions.ts
Outdated
Show resolved
Hide resolved
@axw @felixbarny: Did you have any reservations about adding a default environment to the getting started snippets? |
Co-authored-by: Casper Hübertz <casper@formgeist.com>
💚 Build SucceededMetrics [docs]Distributable file count
History
To update your PR or re-run it, just comment with: |
IMO it's fine to add that for now, we can change it later if/when we add a means of setting a configurable default. |
Co-authored-by: Casper Hübertz <casper@formgeist.com> Co-authored-by: Casper Hübertz <casper@formgeist.com>
Summary
This PR updates the APM getting started tutorial snippets for each agent to include the service environment.
I also changed some words here and there to improve consistency between agent instructions.
Related issues
Closes #77739.