[Observability] Migrate observability API integration tests to Scout#272029
Draft
shahzad31 wants to merge 1 commit into
Draft
[Observability] Migrate observability API integration tests to Scout#272029shahzad31 wants to merge 1 commit into
shahzad31 wants to merge 1 commit into
Conversation
Migrates the trial-license observability API integration FTR suite to Scout API tests under the observability plugin: - annotations CRUD (`annotations.spec.ts`) - SLO annotation filtering (`find_annotations.spec.ts`) - AI assistant alert details contextual insights, split into focused specs with Synthtrace-generated APM traces and logs Removes the old FTR config, shared helpers, SLO archiver fixture, and the `ftr_oblt_stateful_configs.yml` manifest entry, and updates CODEOWNERS and the plugin README to point at the Scout suite. Co-authored-by: Cursor <cursoragent@cursor.com>
shahzad31
added
release_note:skip
|
🤖 Jobs for this PR can be triggered through checkboxes. 🚧
ℹ️ To trigger the CI, please tick the checkbox below 👇
|
12 tasks
Contributor
💛 Build succeeded, but was flaky
Failed CI StepsMetrics [docs] |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Migrates the trial-license
observability_api_integrationFTR suite to Scout API tests, co-located with the existing UI suite under the observability plugin (test/scout/api).Coverage migrated 1:1:
annotations.spec.tsfind_annotations.spec.tsalert_details_context*.spec.ts) using Synthtrace-generated APM traces and logsThis also removes the old FTR wiring:
x-pack/solutions/observability/test/observability_api_integration/(config, shared helpers, SLO kbn_archiver fixture)..buildkite/ftr-manifests/ftr_oblt_stateful_configs.yml..github/CODEOWNERS(kept the suite owned by@elastic/actionable-obs-teamvia atest/scout/apioverride) and refreshed the plugin README.Notes
esTestCluster.license: 'trial'), so no per-test license config is needed.requestAuth) for publicapi/*annotation routes; cookies (samlAuth.asInteractiveUser) for the internal alert-details route, including a least-privilege custom role for the 403 case.Testing
Run against a stateful classic server:
node scripts/type_check --project x-pack/solutions/observability/plugins/observability/tsconfig.jsonChecklist
Made with Cursor