[ftr] add first-class support for playwrite journeys

.buildkite/ftr_configs.yml

@@ -56,7 +56,7 @@ disabled:
   - x-pack/test/fleet_packages/config.ts
 
   # Scalability testing config that we run in its own pipeline
-  - x-pack/test/performance/scalability/config.ts
+  - x-pack/test/scalability/config.ts
 
 defaultQueue: 'n2-4-spot'
 enabled:
@@ -267,10 +267,10 @@ enabled:
   - x-pack/test/ui_capabilities/spaces_only/config.ts
   - x-pack/test/upgrade_assistant_integration/config.js
   - x-pack/test/usage_collection/config.ts
-  - x-pack/test/performance/journeys/ecommerce_dashboard/config.ts
-  - x-pack/test/performance/journeys/flight_dashboard/config.ts
-  - x-pack/test/performance/journeys/login/config.ts
-  - x-pack/test/performance/journeys/many_fields_discover/config.ts
-  - x-pack/test/performance/journeys/promotion_tracking_dashboard/config.ts
-  - x-pack/test/performance/journeys/web_logs_dashboard/config.ts
-  - x-pack/test/performance/journeys/data_stress_test_lens/config.ts
+  - x-pack/performance/journeys/ecommerce_dashboard.ts
+  - x-pack/performance/journeys/flight_dashboard.ts
+  - x-pack/performance/journeys/login.ts
+  - x-pack/performance/journeys/many_fields_discover.ts
+  - x-pack/performance/journeys/promotion_tracking_dashboard.ts
+  - x-pack/performance/journeys/web_logs_dashboard.ts
+  - x-pack/performance/journeys/data_stress_test_lens.ts

.buildkite/pipelines/performance/daily.yml

@@ -1,33 +1,33 @@
 steps:
-  - label: ':male-mechanic::skin-tone-2: Pre-Build'
+  - label: '👨‍🔧 Pre-Build'
     command: .buildkite/scripts/lifecycle/pre_build.sh
     agents:
       queue: kibana-default
 
   - wait
 
-  - label: ':factory_worker: Build Kibana Distribution and Plugins'
+  - label: '🧑‍🏭 Build Kibana Distribution and Plugins'
     command: .buildkite/scripts/steps/build_kibana.sh
     agents:
       queue: c2-16
     key: build
     if: "build.env('KIBANA_BUILD_ID') == null || build.env('KIBANA_BUILD_ID') == ''"
 
-  - label: ':muscle: Performance Tests with Playwright config'
+  - label: '💪 Performance Tests with Playwright config'
     command: .buildkite/scripts/steps/functional/performance_playwright.sh
     agents:
       queue: kb-static-ubuntu
     depends_on: build
     key: tests
     timeout_in_minutes: 60
 
-  - label: ':shipit: Performance Tests dataset extraction for scalability benchmarking'
+  - label: '🚢 Performance Tests dataset extraction for scalability benchmarking'
     command: .buildkite/scripts/steps/functional/scalability_dataset_extraction.sh
     agents:
       queue: n2-2
     depends_on: tests
 
-  - label: ':chart_with_upwards_trend: Report performance metrics to ci-stats'
+  - label: '📈 Report performance metrics to ci-stats'
     command: .buildkite/scripts/steps/functional/report_performance_metrics.sh
     agents:
       queue: n2-2
@@ -36,7 +36,7 @@ steps:
   - wait: ~
     continue_on_failure: true
 
-  - label: ':male_superhero::skin-tone-2: Post-Build'
+  - label: '🦸 Post-Build'
     command: .buildkite/scripts/lifecycle/post_build.sh
     agents:
       queue: kibana-default

.buildkite/pull_requests.json

@@ -42,7 +42,12 @@
       "kibana_versions_check": true,
       "kibana_build_reuse": true,
       "kibana_build_reuse_pipeline_slugs": ["kibana-pull-request", "kibana-on-merge"],
-      "kibana_build_reuse_regexes": ["^test/", "^x-pack/test/"]
+      "kibana_build_reuse_regexes": [
+        "^test/",
+        "^x-pack/test/",
+        "/__snapshots__/",
+        "\\.test\\.(ts|tsx|js|jsx)"
+      ]
     }
   ]
 }

.buildkite/scripts/steps/functional/performance_playwright.sh

@@ -4,16 +4,33 @@ set -euo pipefail
 
 source .buildkite/scripts/common/util.sh
 
+is_test_execution_step
+
 .buildkite/scripts/bootstrap.sh
 # These tests are running on static workers so we have to make sure we delete previous build of Kibana
 rm -rf "$KIBANA_BUILD_LOCATION"
 .buildkite/scripts/download_build_artifacts.sh
 
-echo --- Run Performance Tests with Playwright config
+echo "--- 🦺 Starting Elasticsearch"
 
 node scripts/es snapshot&
+export esPid=$!
+trap 'kill ${esPid}' EXIT
 
-esPid=$!
+export TEST_ES_URL=http://elastic:changeme@localhost:9200
+export TEST_ES_DISABLE_STARTUP=true
+
+# Pings the es server every second for up to 2 minutes until it is green
+curl \
+  --fail \
+  --silent \
+  --retry 120 \
+  --retry-delay 1 \
+  --retry-connrefused \
+  -XGET "${TEST_ES_URL}/_cluster/health?wait_for_nodes=>=1&wait_for_status=yellow" \
+  > /dev/null
+
+echo "✅ ES is ready and will continue to run in the background"
 
 # unset env vars defined in other parts of CI for automatic APM collection of
 # Kibana. We manage APM config in our FTR config and performance service, and
@@ -29,39 +46,27 @@ unset ELASTIC_APM_SERVER_URL
 unset ELASTIC_APM_SECRET_TOKEN
 unset ELASTIC_APM_GLOBAL_LABELS
 
-
-export TEST_ES_URL=http://elastic:changeme@localhost:9200
-export TEST_ES_DISABLE_STARTUP=true
-
-# Pings the es server every seconds 2 mins until it is status is green
-curl --retry 120 \
-  --retry-delay 1 \
-  --retry-all-errors \
-  -I -XGET "${TEST_ES_URL}/_cluster/health?wait_for_nodes=>=1&wait_for_status=yellow"
-
-journeys=("login" "ecommerce_dashboard" "flight_dashboard" "web_logs_dashboard" "promotion_tracking_dashboard" "many_fields_discover" "data_stress_test_lens")
-
-for i in "${journeys[@]}"; do
-    echo "JOURNEY[${i}] is running"
-
-    export TEST_PERFORMANCE_PHASE=WARMUP
-    export JOURNEY_NAME="${i}"
-
-    checks-reporter-with-killswitch "Run Performance Tests with Playwright Config (Journey:${i},Phase: WARMUP)" \
-      node scripts/functional_tests \
-        --config "x-pack/test/performance/journeys/${i}/config.ts" \
-        --kibana-install-dir "$KIBANA_BUILD_LOCATION" \
-        --debug \
-        --bail
-
-    export TEST_PERFORMANCE_PHASE=TEST
-
-    checks-reporter-with-killswitch "Run Performance Tests with Playwright Config (Journey:${i},Phase: TEST)" \
-      node scripts/functional_tests \
-        --config "x-pack/test/performance/journeys/${i}/config.ts" \
-        --kibana-install-dir "$KIBANA_BUILD_LOCATION" \
-        --debug \
-        --bail
+for journey in x-pack/performance/journeys/*; do
+  set +e
+
+  phases=("WARMUP" "TEST")
+  for phase in "${phases[@]}"; do
+    echo "--- $journey - $phase"
+
+    export TEST_PERFORMANCE_PHASE="$phase"
+    node scripts/functional_tests \
+      --config "$journey" \
+      --kibana-install-dir "$KIBANA_BUILD_LOCATION" \
+      --debug \
+      --bail
+
+    status=$?
+    if [ $status -ne 0 ]; then
+      echo "^^^ +++"
+      echo "❌ FTR failed with status code: $status"
+      exit 1
+    fi
+  done
+
+  set -e
 done
-
-kill "$esPid"

.buildkite/scripts/steps/functional/scalability_dataset_extraction.sh

@@ -15,19 +15,16 @@ OUTPUT_DIR="${KIBANA_DIR}/${OUTPUT_REL}"
 .buildkite/scripts/bootstrap.sh
 
 echo "--- Extract APM metrics"
-scalabilityJourneys=("login" "ecommerce_dashboard" "flight_dashboard" "web_logs_dashboard" "promotion_tracking_dashboard" "many_fields_discover")
-
-for i in "${scalabilityJourneys[@]}"; do
-    JOURNEY_NAME="${i}"
-    echo "Looking for JOURNEY=${JOURNEY_NAME} and BUILD_ID=${BUILD_ID} in APM traces"
-
-    node scripts/extract_performance_testing_dataset \
-        --config "x-pack/test/performance/journeys/${i}/config.ts" \
-        --buildId "${BUILD_ID}" \
-        --es-url "${ES_SERVER_URL}" \
-        --es-username "${USER_FROM_VAULT}" \
-        --es-password "${PASS_FROM_VAULT}" \
-        --without-static-resources
+for journey in x-pack/performance/journeys/*; do
+  echo "Looking for journey=${journey} and BUILD_ID=${BUILD_ID} in APM traces"
+
+  node scripts/extract_performance_testing_dataset \
+    --config "${journey}" \
+    --buildId "${BUILD_ID}" \
+    --es-url "${ES_SERVER_URL}" \
+    --es-username "${USER_FROM_VAULT}" \
+    --es-password "${PASS_FROM_VAULT}" \
+    --without-static-resources
 done
 
 echo "--- Creating scalability dataset in ${OUTPUT_REL}"

.eslintrc.js

@@ -178,6 +178,7 @@ const DEV_PATTERNS = [
   'x-pack/{dev-tools,tasks,scripts,test,build_chromium}/**/*',
   'x-pack/plugins/*/server/scripts/**/*',
   'x-pack/plugins/fleet/cypress',
+  'x-pack/performance/**/*',
 ];
 
 /** Restricted imports with suggested alternatives */

.github/CODEOWNERS

@@ -860,9 +860,12 @@ packages/kbn-eslint-plugin-disable @elastic/kibana-operations
 packages/kbn-eslint-plugin-eslint @elastic/kibana-operations
 packages/kbn-eslint-plugin-imports @elastic/kibana-operations
 packages/kbn-expect @elastic/kibana-operations
+packages/kbn-failed-test-reporter-cli @elastic/kibana-operations
 packages/kbn-field-types @elastic/kibana-app-services
 packages/kbn-find-used-node-modules @elastic/kibana-operations
 packages/kbn-flot-charts @elastic/kibana-operations
+packages/kbn-ftr-common-functional-services @elastic/kibana-operations
+packages/kbn-ftr-screenshot-filename @elastic/kibana-operations
 packages/kbn-generate @elastic/kibana-operations
 packages/kbn-get-repo-files @elastic/kibana-operations
 packages/kbn-handlebars @elastic/kibana-security
@@ -873,6 +876,7 @@ packages/kbn-import-resolver @elastic/kibana-operations
 packages/kbn-interpreter @elastic/kibana-app-services
 packages/kbn-io-ts-utils @elastic/apm-ui
 packages/kbn-jest-serializers @elastic/kibana-operations
+packages/kbn-journeys @elastic/kibana-operations
 packages/kbn-kibana-manifest-schema @elastic/kibana-operations
 packages/kbn-logging @elastic/kibana-core
 packages/kbn-logging-mocks @elastic/kibana-core

dev_docs/operations/operations_landing.mdx

@@ -26,6 +26,7 @@ layout: landing
     { pageId: "kibDevDocsOpsWritingStableFunctionalTests" },
     { pageId: "kibDevDocsOpsFlakyTestRunner" },
     { pageId: "kibDevDocsOpsCiStats" },
+    { pageId: "kibDevDocsOpsJourneys" },
   ]}
 />
 

package.json

@@ -702,11 +702,15 @@
     "@kbn/eslint-plugin-eslint": "link:bazel-bin/packages/kbn-eslint-plugin-eslint",
     "@kbn/eslint-plugin-imports": "link:bazel-bin/packages/kbn-eslint-plugin-imports",
     "@kbn/expect": "link:bazel-bin/packages/kbn-expect",
+    "@kbn/failed-test-reporter-cli": "link:bazel-bin/packages/kbn-failed-test-reporter-cli",
     "@kbn/find-used-node-modules": "link:bazel-bin/packages/kbn-find-used-node-modules",
+    "@kbn/ftr-common-functional-services": "link:bazel-bin/packages/kbn-ftr-common-functional-services",
+    "@kbn/ftr-screenshot-filename": "link:bazel-bin/packages/kbn-ftr-screenshot-filename",
     "@kbn/generate": "link:bazel-bin/packages/kbn-generate",
     "@kbn/get-repo-files": "link:bazel-bin/packages/kbn-get-repo-files",
     "@kbn/import-resolver": "link:bazel-bin/packages/kbn-import-resolver",
     "@kbn/jest-serializers": "link:bazel-bin/packages/kbn-jest-serializers",
+    "@kbn/journeys": "link:bazel-bin/packages/kbn-journeys",
     "@kbn/kibana-manifest-schema": "link:bazel-bin/packages/kbn-kibana-manifest-schema",
     "@kbn/managed-vscode-config": "link:bazel-bin/packages/kbn-managed-vscode-config",
     "@kbn/managed-vscode-config-cli": "link:bazel-bin/packages/kbn-managed-vscode-config-cli",
@@ -1017,8 +1021,11 @@
     "@types/kbn__es-types": "link:bazel-bin/packages/kbn-es-types/npm_module_types",
     "@types/kbn__eslint-plugin-disable": "link:bazel-bin/packages/kbn-eslint-plugin-disable/npm_module_types",
     "@types/kbn__eslint-plugin-imports": "link:bazel-bin/packages/kbn-eslint-plugin-imports/npm_module_types",
+    "@types/kbn__failed-test-reporter-cli": "link:bazel-bin/packages/kbn-failed-test-reporter-cli/npm_module_types",
     "@types/kbn__field-types": "link:bazel-bin/packages/kbn-field-types/npm_module_types",
     "@types/kbn__find-used-node-modules": "link:bazel-bin/packages/kbn-find-used-node-modules/npm_module_types",
+    "@types/kbn__ftr-common-functional-services": "link:bazel-bin/packages/kbn-ftr-common-functional-services/npm_module_types",
+    "@types/kbn__ftr-screenshot-filename": "link:bazel-bin/packages/kbn-ftr-screenshot-filename/npm_module_types",
     "@types/kbn__generate": "link:bazel-bin/packages/kbn-generate/npm_module_types",
     "@types/kbn__get-repo-files": "link:bazel-bin/packages/kbn-get-repo-files/npm_module_types",
     "@types/kbn__handlebars": "link:bazel-bin/packages/kbn-handlebars/npm_module_types",
@@ -1032,6 +1039,7 @@
     "@types/kbn__interpreter": "link:bazel-bin/packages/kbn-interpreter/npm_module_types",
     "@types/kbn__io-ts-utils": "link:bazel-bin/packages/kbn-io-ts-utils/npm_module_types",
     "@types/kbn__jest-serializers": "link:bazel-bin/packages/kbn-jest-serializers/npm_module_types",
+    "@types/kbn__journeys": "link:bazel-bin/packages/kbn-journeys/npm_module_types",
     "@types/kbn__kbn-ci-stats-performance-metrics": "link:bazel-bin/packages/kbn-kbn-ci-stats-performance-metrics/npm_module_types",
     "@types/kbn__kibana-manifest-schema": "link:bazel-bin/packages/kbn-kibana-manifest-schema/npm_module_types",
     "@types/kbn__logging": "link:bazel-bin/packages/kbn-logging/npm_module_types",

packages/BUILD.bazel

@@ -204,9 +204,12 @@ filegroup(
     "//packages/kbn-eslint-plugin-eslint:build",
     "//packages/kbn-eslint-plugin-imports:build",
     "//packages/kbn-expect:build",
+    "//packages/kbn-failed-test-reporter-cli:build",
     "//packages/kbn-field-types:build",
     "//packages/kbn-find-used-node-modules:build",
     "//packages/kbn-flot-charts:build",
+    "//packages/kbn-ftr-common-functional-services:build",
+    "//packages/kbn-ftr-screenshot-filename:build",
     "//packages/kbn-generate:build",
     "//packages/kbn-get-repo-files:build",
     "//packages/kbn-handlebars:build",
@@ -217,6 +220,7 @@ filegroup(
     "//packages/kbn-interpreter:build",
     "//packages/kbn-io-ts-utils:build",
     "//packages/kbn-jest-serializers:build",
+    "//packages/kbn-journeys:build",
     "//packages/kbn-kibana-manifest-schema:build",
     "//packages/kbn-logging:build",
     "//packages/kbn-logging-mocks:build",
@@ -514,8 +518,11 @@ filegroup(
     "//packages/kbn-es-types:build_types",
     "//packages/kbn-eslint-plugin-disable:build_types",
     "//packages/kbn-eslint-plugin-imports:build_types",
+    "//packages/kbn-failed-test-reporter-cli:build_types",
     "//packages/kbn-field-types:build_types",
     "//packages/kbn-find-used-node-modules:build_types",
+    "//packages/kbn-ftr-common-functional-services:build_types",
+    "//packages/kbn-ftr-screenshot-filename:build_types",
     "//packages/kbn-generate:build_types",
     "//packages/kbn-get-repo-files:build_types",
     "//packages/kbn-handlebars:build_types",
@@ -526,6 +533,7 @@ filegroup(
     "//packages/kbn-interpreter:build_types",
     "//packages/kbn-io-ts-utils:build_types",
     "//packages/kbn-jest-serializers:build_types",
+    "//packages/kbn-journeys:build_types",
     "//packages/kbn-kibana-manifest-schema:build_types",
     "//packages/kbn-logging:build_types",
     "//packages/kbn-logging-mocks:build_types",

packages/kbn-dev-cli-runner/index.ts

@@ -9,4 +9,5 @@
 export * from './src/run';
 export * from './src/run_with_commands';
 export * from './src/flags';
+export * from './src/flags_reader';
 export type { CleanupTask } from './src/cleanup';

packages/kbn-dev-cli-runner/src/flags.ts

@@ -53,6 +53,10 @@ export function mergeFlagOptions(global: FlagOptions = {}, local: FlagOptions =
   };
 }
 
+export const DEFAULT_FLAG_ALIASES = {
+  v: 'verbose',
+};
+
 export function getFlags(
   argv: string[],
   flagOptions: RunOptions['flags'] = {},
@@ -67,7 +71,7 @@ export function getFlags(
     boolean: [...(flagOptions.boolean || []), ...logLevelFlags, 'help'],
     alias: {
       ...flagOptions.alias,
-      v: 'verbose',
+      ...DEFAULT_FLAG_ALIASES,
     },
     default: flagOptions.default,
     unknown: (name: string) => {