Skip to content

Fix preview chart in APM rules#156625

Closed
benakansara wants to merge 9 commits intoelastic:mainfrom
benakansara:bugfix/preview-chart-apm-rules
Closed

Fix preview chart in APM rules#156625
benakansara wants to merge 9 commits intoelastic:mainfrom
benakansara:bugfix/preview-chart-apm-rules

Conversation

@benakansara
Copy link
Copy Markdown
Contributor

@benakansara benakansara commented May 3, 2023

@benakansara benakansara added release_note:skip Skip the PR/issue when compiling release notes Team: Actionable Observability - DEPRECATED For Observability Alerting and SLOs use "Team:obs-ux-management", for AIops "Team:obs-knowledge" v8.9.0 labels May 3, 2023
@benakansara benakansara requested a review from a team May 3, 2023 21:50
@benakansara benakansara self-assigned this May 3, 2023
@ghost
Copy link
Copy Markdown

ghost commented May 3, 2023

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
  • run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

benakansara added a commit that referenced this pull request May 8, 2023
Hides preview charts in the following APM rules until
#156625 (WIP) gets merged.

- APM Latency threshold
- APM Failed transaction rate
- APM Error count threshold
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request May 8, 2023
Hides preview charts in the following APM rules until
elastic#156625 (WIP) gets merged.

- APM Latency threshold
- APM Failed transaction rate
- APM Error count threshold

(cherry picked from commit d10c48b)
kibanamachine added a commit that referenced this pull request May 8, 2023
# Backport

This will backport the following commits from `main` to `8.8`:
- [Hide preview charts in APM rules
(#156966)](#156966)

<!--- Backport version: 8.9.7 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Bena
Kansara","email":"69037875+benakansara@users.noreply.github.com"},"sourceCommit":{"committedDate":"2023-05-08T11:12:08Z","message":"Hide
preview charts in APM rules (#156966)\n\nHides preview charts in the
following APM rules
until\r\nhttps://github.com//pull/156625 (WIP) gets
merged.\r\n\r\n- APM Latency threshold\r\n- APM Failed transaction
rate\r\n- APM Error count
threshold","sha":"d10c48b983404025c6b0009c899ccc59c655bc6d","branchLabelMapping":{"^v8.9.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:APM","release_note:skip","Team:
Actionable
Observability","v8.8.0","v8.9.0"],"number":156966,"url":"https://github.com/elastic/kibana/pull/156966","mergeCommit":{"message":"Hide
preview charts in APM rules (#156966)\n\nHides preview charts in the
following APM rules
until\r\nhttps://github.com//pull/156625 (WIP) gets
merged.\r\n\r\n- APM Latency threshold\r\n- APM Failed transaction
rate\r\n- APM Error count
threshold","sha":"d10c48b983404025c6b0009c899ccc59c655bc6d"}},"sourceBranch":"main","suggestedTargetBranches":["8.8"],"targetPullRequestStates":[{"branch":"8.8","label":"v8.8.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.9.0","labelRegex":"^v8.9.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/156966","number":156966,"mergeCommit":{"message":"Hide
preview charts in APM rules (#156966)\n\nHides preview charts in the
following APM rules
until\r\nhttps://github.com//pull/156625 (WIP) gets
merged.\r\n\r\n- APM Latency threshold\r\n- APM Failed transaction
rate\r\n- APM Error count
threshold","sha":"d10c48b983404025c6b0009c899ccc59c655bc6d"}}]}]
BACKPORT-->

Co-authored-by: Bena Kansara <69037875+benakansara@users.noreply.github.com>
@kibana-ci
Copy link
Copy Markdown

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
apm 3.5MB 3.5MB -95.0B
Unknown metric groups

ESLint disabled line counts

id before after diff
enterpriseSearch 19 21 +2
securitySolution 400 404 +4
total +6

Total ESLint disabled count

id before after diff
enterpriseSearch 20 22 +2
securitySolution 480 484 +4
total +6

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @benakansara

@benakansara benakansara deleted the bugfix/preview-chart-apm-rules branch November 13, 2023 03:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release_note:skip Skip the PR/issue when compiling release notes Team: Actionable Observability - DEPRECATED For Observability Alerting and SLOs use "Team:obs-ux-management", for AIops "Team:obs-knowledge" v8.9.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants