-
Notifications
You must be signed in to change notification settings - Fork 518
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: Remove txmetrics config #10140
Conversation
Removing transaction metrics config as they are not officially supported.
This pull request does not have a backport label. Could you fix it @carsonip? 🙏
NOTE: |
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.
cc @bmorelli25 for visibility since he was the owner of the original PR
@@ -73,8 +72,6 @@ include::{libbeat-dir}/shared-ssl-config.asciidoc[] | |||
include::ssl-input-settings.asciidoc[] | |||
// END SSL SECTION -------------------------------------------- | |||
|
|||
include::./transaction-metrics.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.
issue: this might require coordinating with the kibana team to remove the companion page on their documentation (should be a config option): elastic/kibana#82379
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.
Yup. This will need to be removed and backported from the Kibana docs before docs ci will pass on this PR (as that content links to this content). I'll open a PR in the Kibana repo take care of that.
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.
Opened elastic/kibana#149896
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.
Merged and backported in the Kibana repo. Rerunning docs ci now...
### Summary This PR reverts #82379 and removes `xpack.apm.searchAggregatedTransactions` from the documentation. This is for elastic/apm-server#10140: > Removing transaction metrics config as they are not officially supported.
### Summary This PR reverts elastic#82379 and removes `xpack.apm.searchAggregatedTransactions` from the documentation. This is for elastic/apm-server#10140: > Removing transaction metrics config as they are not officially supported. (cherry picked from commit b90ddba)
@elasticmachine, run elasticsearch-ci/docs |
### Summary This PR reverts elastic#82379 and removes `xpack.apm.searchAggregatedTransactions` from the documentation. This is for elastic/apm-server#10140: > Removing transaction metrics config as they are not officially supported. (cherry picked from commit b90ddba) # Conflicts: # docs/settings/apm-settings.asciidoc
…#149901) # Backport This will backport the following commits from `main` to `8.6`: - [docs: remove `xpack.apm.searchAggregatedTransactions` (#149896)](#149896) <!--- Backport version: 8.9.7 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Brandon Morelli","email":"brandon.morelli@elastic.co"},"sourceCommit":{"committedDate":"2023-01-30T22:33:03Z","message":"docs: remove `xpack.apm.searchAggregatedTransactions` (#149896)\n\n### Summary\r\n\r\nThis PR reverts #82379 and removes\r\n`xpack.apm.searchAggregatedTransactions` from the documentation. This is\r\nfor elastic/apm-server#10140> Removing transaction metrics config as they are not officially\r\nsupported.","sha":"b90ddba88244fdf81593737cbdb5211664090a0c","branchLabelMapping":{"^v8.7.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:APM","release_note:skip","v8.7.0","v7.17.9","v8.6.2"],"number":149896,"url":"#149896: remove `xpack.apm.searchAggregatedTransactions` (#149896)\n\n### Summary\r\n\r\nThis PR reverts #82379 and removes\r\n`xpack.apm.searchAggregatedTransactions` from the documentation. This is\r\nfor elastic/apm-server#10140> Removing transaction metrics config as they are not officially\r\nsupported.","sha":"b90ddba88244fdf81593737cbdb5211664090a0c"}},"sourceBranch":"main","suggestedTargetBranches":["7.17","8.6"],"targetPullRequestStates":[{"branch":"main","label":"v8.7.0","labelRegex":"^v8.7.0$","isSourceBranch":true,"state":"MERGED","url":"#149896: remove `xpack.apm.searchAggregatedTransactions` (#149896)\n\n### Summary\r\n\r\nThis PR reverts #82379 and removes\r\n`xpack.apm.searchAggregatedTransactions` from the documentation. This is\r\nfor elastic/apm-server#10140> Removing transaction metrics config as they are not officially\r\nsupported.","sha":"b90ddba88244fdf81593737cbdb5211664090a0c"}},{"branch":"7.17","label":"v7.17.9","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.6","label":"v8.6.2","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}] BACKPORT--> Co-authored-by: Brandon Morelli <brandon.morelli@elastic.co>
#149902) # Backport This will backport the following commits from `main` to `7.17`: - [docs: remove `xpack.apm.searchAggregatedTransactions` (#149896)](#149896) <!--- Backport version: 8.9.7 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Brandon Morelli","email":"brandon.morelli@elastic.co"},"sourceCommit":{"committedDate":"2023-01-30T22:33:03Z","message":"docs: remove `xpack.apm.searchAggregatedTransactions` (#149896)\n\n### Summary\r\n\r\nThis PR reverts #82379 and removes\r\n`xpack.apm.searchAggregatedTransactions` from the documentation. This is\r\nfor elastic/apm-server#10140> Removing transaction metrics config as they are not officially\r\nsupported.","sha":"b90ddba88244fdf81593737cbdb5211664090a0c","branchLabelMapping":{"^v8.7.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:APM","release_note:skip","v8.7.0","v7.17.9","v8.6.2"],"number":149896,"url":"#149896: remove `xpack.apm.searchAggregatedTransactions` (#149896)\n\n### Summary\r\n\r\nThis PR reverts #82379 and removes\r\n`xpack.apm.searchAggregatedTransactions` from the documentation. This is\r\nfor elastic/apm-server#10140> Removing transaction metrics config as they are not officially\r\nsupported.","sha":"b90ddba88244fdf81593737cbdb5211664090a0c"}},"sourceBranch":"main","suggestedTargetBranches":["7.17"],"targetPullRequestStates":[{"branch":"main","label":"v8.7.0","labelRegex":"^v8.7.0$","isSourceBranch":true,"state":"MERGED","url":"#149896: remove `xpack.apm.searchAggregatedTransactions` (#149896)\n\n### Summary\r\n\r\nThis PR reverts #82379 and removes\r\n`xpack.apm.searchAggregatedTransactions` from the documentation. This is\r\nfor elastic/apm-server#10140> Removing transaction metrics config as they are not officially\r\nsupported.","sha":"b90ddba88244fdf81593737cbdb5211664090a0c"}},{"branch":"7.17","label":"v7.17.9","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.6","label":"v8.6.2","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"url":"#149901"}]}] BACKPORT-->
@bmorelli25 sorry for the unclear comms. The intention of this PR is to stop documenting these apm-server transaction aggregation configs because they are not officially supported. e.g. I thought the change in kibana docs will be to only remove the pointer but not the paragraph. I'm not sure if we need to remove the paragraph as it is not exactly about apm-server but more about the UI, and they may still be valid and officially supported. cc @simitt |
Oops, I might've jumped the gun. Sorry about that. I can re-add the setting to the Kibana docs. What should we say about
|
TBH I'm not very sure. Transaction aggregations should be always enabled in apm-server. I assume nothing extra is needed for transaction aggregation to run properly. @kruskall what do you think? |
@simitt Do you know what "additional configuration" it is trying to imply in the kibana docs? |
@bmorelli25, I discussed with @sqren that removing the paragraph from Kibana UI is fine, and even overdue. |
@bmorelli25 let me know if this PR is ready to merge or simply takeover from here. Thanks! |
I'm in favour of removing the setting on the UI side as well. In the past we allowed the user to disable transaction metrics but I don't see any good reason for doing that anymore. When the setting is removed it'll keep the same default behaviour ( I've created elastic/kibana#150164 to remove the setting from Kibana. |
### Summary This PR reverts elastic#82379 and removes `xpack.apm.searchAggregatedTransactions` from the documentation. This is for elastic/apm-server#10140: > Removing transaction metrics config as they are not officially supported.
### Summary This PR reverts elastic#82379 and removes `xpack.apm.searchAggregatedTransactions` from the documentation. This is for elastic/apm-server#10140: > Removing transaction metrics config as they are not officially supported.
Removing transaction metrics config as they are not officially supported. (cherry picked from commit 54698bd) # Conflicts: # docs/legacy/transaction-metrics.asciidoc
Removing transaction metrics config as they are not officially supported. (cherry picked from commit 54698bd)
Removing transaction metrics config as they are not officially supported.
* docs: Remove txmetrics config (#10140) Removing transaction metrics config as they are not officially supported. (cherry picked from commit 54698bd) # Conflicts: # docs/legacy/transaction-metrics.asciidoc * docs: Remove txmetrics config (#10140) Removing transaction metrics config as they are not officially supported. --------- Co-authored-by: Carson Ip <carsonip@users.noreply.github.com>
Removing transaction metrics config as they are not officially supported.