[8.13] Make in-product APM alerting links smart (stateful vs serverle…

…ss) (#176252) (#179093)

# Backport

This will backport the following commits from `main` to `8.13`:
- [Make in-product APM alerting links smart (stateful vs serverless)
(#176252)](#176252)

<!--- Backport version: 9.4.3 -->

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

<!--BACKPORT [{"author":{"name":"Brandon
Morelli","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-03-20T18:43:41Z","message":"Make
in-product APM alerting links smart (stateful vs serverless)
(#176252)\n\n## Summary\r\n\r\nThis is a part
of\r\nhttps://github.com/elastic/observability-docs/issues/3622.\r\n\r\nBefore
this PR, all four in-product APM alerting documentation links\r\npointed
to\r\nhttps://www.elastic.co/guide/en/kibana/master/apm-alerts.html.
This PR\r\nupdates our in-product links to point to either our
Serverless or\r\nStateful documentation, depending on the product
type.\r\n\r\nCloses
https://github.com/elastic/observability-docs/issues/3586.\r\n\r\n---------\r\n\r\nCo-authored-by:
Kibana Machine
<[email protected]>\r\nCo-authored-by:
Caue Marcondes
<[email protected]>","sha":"62aa3279dfdfb12112827826b6ff2d76b50fbe96","branchLabelMapping":{"^v8.14.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:obs-ux-infra_services","v8.14.0","v8.13.1"],"title":"Make
in-product APM alerting links smart (stateful vs
serverless)","number":176252,"url":"https://github.com/elastic/kibana/pull/176252","mergeCommit":{"message":"Make
in-product APM alerting links smart (stateful vs serverless)
(#176252)\n\n## Summary\r\n\r\nThis is a part
of\r\nhttps://github.com/elastic/observability-docs/issues/3622.\r\n\r\nBefore
this PR, all four in-product APM alerting documentation links\r\npointed
to\r\nhttps://www.elastic.co/guide/en/kibana/master/apm-alerts.html.
This PR\r\nupdates our in-product links to point to either our
Serverless or\r\nStateful documentation, depending on the product
type.\r\n\r\nCloses
https://github.com/elastic/observability-docs/issues/3586.\r\n\r\n---------\r\n\r\nCo-authored-by:
Kibana Machine
<[email protected]>\r\nCo-authored-by:
Caue Marcondes
<[email protected]>","sha":"62aa3279dfdfb12112827826b6ff2d76b50fbe96"}},"sourceBranch":"main","suggestedTargetBranches":["8.13"],"targetPullRequestStates":[{"branch":"main","label":"v8.14.0","branchLabelMappingKey":"^v8.14.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/176252","number":176252,"mergeCommit":{"message":"Make
in-product APM alerting links smart (stateful vs serverless)
(#176252)\n\n## Summary\r\n\r\nThis is a part
of\r\nhttps://github.com/elastic/observability-docs/issues/3622.\r\n\r\nBefore
this PR, all four in-product APM alerting documentation links\r\npointed
to\r\nhttps://www.elastic.co/guide/en/kibana/master/apm-alerts.html.
This PR\r\nupdates our in-product links to point to either our
Serverless or\r\nStateful documentation, depending on the product
type.\r\n\r\nCloses
https://github.com/elastic/observability-docs/issues/3586.\r\n\r\n---------\r\n\r\nCo-authored-by:
Kibana Machine
<[email protected]>\r\nCo-authored-by:
Caue Marcondes
<[email protected]>","sha":"62aa3279dfdfb12112827826b6ff2d76b50fbe96"}},{"branch":"8.13","label":"v8.13.1","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Brandon Morelli <[email protected]>

packages/kbn-doc-links/src/get_doc_links.ts

@@ -599,7 +599,18 @@ export const getDocLinks = ({ kibanaBranch, buildFlavor }: GetDocLinkOptions): D
       actionTypes: isServerless
         ? `${SERVERLESS_DOCS}action-connectors`
         : `${KIBANA_DOCS}action-types.html`,
-      apmRules: `${KIBANA_DOCS}apm-alerts.html`,
+      apmRulesErrorCount: isServerless
+        ? `${SERVERLESS_OBSERVABILITY_DOCS}create-error-count-threshold-alert-rule`
+        : `${KIBANA_DOCS}apm-alerts.html`,
+      apmRulesTransactionDuration: isServerless
+        ? `${SERVERLESS_OBSERVABILITY_DOCS}create-latency-threshold-alert-rule`
+        : `${KIBANA_DOCS}apm-alerts.html`,
+      apmRulesTransactionError: isServerless
+        ? `${SERVERLESS_OBSERVABILITY_DOCS}create-failed-transaction-rate-threshold-alert-rule`
+        : `${KIBANA_DOCS}apm-alerts.html`,
+      apmRulesAnomaly: isServerless
+        ? `${SERVERLESS_OBSERVABILITY_DOCS}create-anomaly-alert-rule`
+        : `${KIBANA_DOCS}apm-alerts.html`,
       emailAction: `${KIBANA_DOCS}email-action-type.html`,
       emailActionConfig: `${KIBANA_DOCS}email-action-type.html`,
       emailExchangeClientSecretConfig: `${KIBANA_DOCS}email-action-type.html#exchange-client-secret`,

packages/kbn-doc-links/src/types.ts

@@ -436,7 +436,10 @@ export interface DocLinks {
   readonly alerting: Readonly<{
     guide: string;
     actionTypes: string;
-    apmRules: string;
+    apmRulesErrorCount: string;
+    apmRulesTransactionDuration: string;
+    apmRulesTransactionError: string;
+    apmRulesAnomaly: string;
     emailAction: string;
     emailActionConfig: string;
     emailExchangeClientSecretConfig: string;

x-pack/plugins/apm/ftr_e2e/cypress/e2e/dependencies/dependencies.cy.ts

@@ -142,10 +142,7 @@ describe('Dependencies with high volume of data', () => {
 
   it('shows dependencies inventory page', () => {
     cy.visitKibana(
-      `/app/apm/dependencies/inventory?${new URLSearchParams({
-        ...timeRange,
-        kuery: 'elasticsearch*',
-      })}`
+      `/app/apm/dependencies/inventory?${new URLSearchParams(timeRange)}`
     );
 
     cy.getByTestSubj('dependenciesTable');

x-pack/plugins/apm/public/components/alerting/rule_types/register_apm_rule_types.ts

@@ -47,7 +47,7 @@ export function registerApmRuleTypes(
     },
     iconClass: 'bell',
     documentationUrl(docLinks) {
-      return `${docLinks.links.alerting.apmRules}`;
+      return `${docLinks.links.alerting.apmRulesErrorCount}`;
     },
     ruleParamsExpression: lazy(() => import('./error_count_rule_type')),
     validate: () => ({
@@ -80,7 +80,7 @@ export function registerApmRuleTypes(
     },
     iconClass: 'bell',
     documentationUrl(docLinks) {
-      return `${docLinks.links.alerting.apmRules}`;
+      return `${docLinks.links.alerting.apmRulesTransactionDuration}`;
     },
     ruleParamsExpression: lazy(
       () => import('./transaction_duration_rule_type')
@@ -116,7 +116,7 @@ export function registerApmRuleTypes(
     }),
     iconClass: 'bell',
     documentationUrl(docLinks) {
-      return `${docLinks.links.alerting.apmRules}`;
+      return `${docLinks.links.alerting.apmRulesTransactionError}`;
     },
     ruleParamsExpression: lazy(
       () => import('./transaction_error_rate_rule_type')
@@ -146,7 +146,7 @@ export function registerApmRuleTypes(
     }),
     iconClass: 'bell',
     documentationUrl(docLinks) {
-      return `${docLinks.links.alerting.apmRules}`;
+      return `${docLinks.links.alerting.apmRulesAnomaly}`;
     },
     ruleParamsExpression: lazy(() => import('./anomaly_rule_type')),
     validate: validateAnomalyRule,