Skip to content

[9.4] [scout] migrate FTR logstash api tests (#262953)#264265

Merged
kibanamachine merged 1 commit intoelastic:9.4from
kibanamachine:backport/9.4/commit-4720f964
Apr 18, 2026
Merged

[9.4] [scout] migrate FTR logstash api tests (#262953)#264265
kibanamachine merged 1 commit intoelastic:9.4from
kibanamachine:backport/9.4/commit-4720f964

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 9.4:

Questions ?

Please refer to the Backport tool documentation

## Summary

Closes elastic/appex-qa-team#851

Logstash API tests migrated from FTR → Scout

Source: `x-pack/platform/test/api_integration/apis/logstash/` (5 test
files, now deleted)
Target: `x-pack/platform/plugins/private/logstash/test/scout/api/`

New spec files:

File | Covers
-- | --
cluster.spec.ts | GET /api/logstash/cluster
pipeline.spec.ts | GET, PUT, DELETE /api/logstash/pipeline/{id}
pipelines_delete.spec.ts | POST /api/logstash/pipelines/delete
pipelines_list.spec.ts | GET /api/logstash/pipelines (was skipped in FTR
— now unskipped)

- No esArchiver — `.logstash` is a restricted system index; bulk
indexing is blocked even for superuser. Pipelines are seeded via
`esClient.logstash.putPipeline` in `beforeAll` per spec instead
- Minimum roles — `LOGSTASH_MANAGER_ROLE` (manage_logstash_pipelines)
for pipeline ops; `LOGSTASH_CLUSTER_ROLE` (monitor) for the cluster
endpoint (FTR used superuser for everything)
- Stateful only — no serverless Logstash tests exist or were added
- Coverage improvements — added 404 assertion for non-existing pipeline
GET; explicit numSuccesses/numErrors check on bulk delete; sort-order
assertion on pipeline list

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit 4720f96)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Apr 18, 2026
@kibanamachine kibanamachine enabled auto-merge (squash) April 18, 2026 11:10
@kibanamachine kibanamachine merged commit d3629c4 into elastic:9.4 Apr 18, 2026
19 checks passed
@elasticmachine
Copy link
Copy Markdown
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #95 / cases security and spaces enabled: basic Common migrations migrations 7.13.2 "after all" hook for "adds the owner field"
  • [job] [logs] FTR Configs #95 / cases security and spaces enabled: basic Common migrations migrations 7.13.2 "before all" hook for "adds the owner field"

Metrics [docs]

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
@kbn/scout-oblt 274 276 +2
@kbn/scout-search 244 246 +2
@kbn/scout-security 261 263 +2
total +6

Public APIs missing exports

Total count of every type that is part of your API that should be exported but is not. This will cause broken links in the API documentation system. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats exports for more detailed information.

id before after diff
logstash 0 1 +1
Unknown metric groups

API count

id before after diff
@kbn/scout-oblt 711 713 +2
@kbn/scout-search 670 672 +2
@kbn/scout-security 687 689 +2
total +6

cc @dmlemeshko

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants