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

Clarify descriptions of APM indicators for SLOs #3953

Merged
merged 5 commits into from
Jun 17, 2024

Conversation

dedemorton
Copy link
Contributor

Closes #3913

@graphaelli @grabowskit WDYT? Is this what you had in mind?

If you want me to add a note about the impact of unknown outcomes, I can do that, but I need to know what the impact is. :-)

@dedemorton dedemorton added backport-8.13 Automated backport with mergify backport-8.14 Automated backport with mergify labels May 30, 2024
@dedemorton dedemorton requested a review from a team as a code owner May 30, 2024 22:49
@dedemorton dedemorton self-assigned this May 30, 2024
Copy link
Contributor

A documentation preview will be available soon.

Request a new doc build by commenting
  • Rebuild this PR: run docs-build
  • Rebuild this PR and all Elastic docs: run docs-build rebuild

run docs-build is much faster than run docs-build rebuild. A rebuild should only be needed in rare situations.

If your PR continues to fail for an unknown reason, the doc build pipeline may be broken. Elastic employees can check the pipeline status here.

@elasticdocs
Copy link

🚀 Built co-preview-docs successfully!

Issues? Visit #next-docs in Slack

@dedemorton
Copy link
Contributor Author

bump...I need a technical review before I can merge this. Thanks!

Co-authored-by: Gil Raphaelli <graphaelli@gmail.com>
@bmorelli25
Copy link
Member

run docs-build

@dedemorton dedemorton enabled auto-merge (squash) June 17, 2024 21:48
@elasticdocs
Copy link

🚀 Built elastic-dot-co-docs-preview-docs successfully!

Issues? Visit #next-docs in Slack

@dedemorton dedemorton merged commit 4770b0c into elastic:main Jun 17, 2024
4 checks passed
mergify bot pushed a commit that referenced this pull request Jun 17, 2024
---------

Co-authored-by: Gil Raphaelli <graphaelli@gmail.com>
(cherry picked from commit 4770b0c)

# Conflicts:
#	docs/en/serverless/slos/create-an-slo.mdx
@dedemorton dedemorton deleted the issue#3913 branch June 17, 2024 22:09
mergify bot pushed a commit that referenced this pull request Jun 17, 2024
---------

Co-authored-by: Gil Raphaelli <graphaelli@gmail.com>
(cherry picked from commit 4770b0c)

# Conflicts:
#	docs/en/serverless/slos/create-an-slo.mdx
dedemorton pushed a commit that referenced this pull request Jul 1, 2024
#4018)

* Clarify descriptions of APM indicators for SLOs (#3953)

---------

Co-authored-by: Gil Raphaelli <graphaelli@gmail.com>
(cherry picked from commit 4770b0c)
dedemorton added a commit that referenced this pull request Jul 1, 2024
#4017)

* Clarify descriptions of APM indicators for SLOs (#3953)

---------

Co-authored-by: Gil Raphaelli <graphaelli@gmail.com>
(cherry picked from commit 4770b0c)

Co-authored-by: DeDe Morton <dede.morton@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-8.13 Automated backport with mergify backport-8.14 Automated backport with mergify ci:doc-build
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Clarify indicator for APM availability SLO
4 participants