Skip to content

[9.3] [scout] migrate FTR logstash api tests (#262953)#264361

Merged
dmlemeshko merged 2 commits intoelastic:9.3from
dmlemeshko:backport/9.3/pr-262953
Apr 20, 2026
Merged

[9.3] [scout] migrate FTR logstash api tests (#262953)#264361
dmlemeshko merged 2 commits intoelastic:9.3from
dmlemeshko:backport/9.3/pr-262953

Conversation

@dmlemeshko
Copy link
Copy Markdown
Contributor

@dmlemeshko dmlemeshko commented Apr 20, 2026

Backport

This will backport the following commits from main to 9.3:

Questions ?

Please refer to the Backport tool documentation

@dmlemeshko
[scout] migrate FTR logstash api tests (elastic#262953)
6470c19 
## 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)

# Conflicts:
#	.buildkite/scout_ci_config.yml
#	x-pack/platform/plugins/private/logstash/moon.yml
#	x-pack/platform/plugins/private/logstash/tsconfig.json
@dmlemeshko dmlemeshko added the backport This PR is a backport of another PR label Apr 20, 2026
@dmlemeshko dmlemeshko enabled auto-merge (squash) April 20, 2026 10:10
@dmlemeshko dmlemeshko mentioned this pull request Apr 20, 2026
Merged
@dmlemeshko dmlemeshko requested a review from a team April 20, 2026 10:11
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Apr 20, 2026 •
edited
Loading

💔 Build Failed

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #35 / Cloud Security Posture - Group 2 Test adding Cloud Security Posture Integrations CSPM AWS CIS_AWS Organization Cloud Formation On Add Agent modal there should be modal that has Cloud Formation details as well as button that redirects user to Cloud formation page on AWS upon clicking them

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 265 267 +2
@kbn/scout-search 238 240 +2
@kbn/scout-security 247 249 +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 665 667 +2
@kbn/scout-search 631 633 +2
@kbn/scout-security 640 642 +2
total +6

History

@dmlemeshko dmlemeshko merged commit f4630ce into elastic:9.3 Apr 20, 2026
10 checks passed
dmlemeshko added a commit that referenced this pull request Apr 22, 2026
@dmlemeshko
[8.19] [scout] migrate FTR logstash api tests (#262953) (#264648)
0f672a9 
# Backport

This will backport the following commits from `main` to `8.19`:
- [[scout] migrate FTR logstash api tests
(#262953)](#262953)

<!--- Backport version: 11.0.1 -->

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

<!--BACKPORT [{"author":{"name":"Dzmitry
Lemechko","email":"dzmitry.lemechko@elastic.co"},"sourceCommit":{"committedDate":"2026-04-18T10:59:59Z","message":"[scout]
migrate FTR logstash api tests (#262953)\n\n## Summary\n\nCloses
https://github.com/elastic/appex-qa-team/issues/851\n\nLogstash API
tests migrated from FTR → Scout\n\nSource:
`x-pack/platform/test/api_integration/apis/logstash/` (5 test\nfiles,
now deleted)\nTarget:
`x-pack/platform/plugins/private/logstash/test/scout/api/`\n\nNew spec
files:\n\n\nFile | Covers\n-- | --\ncluster.spec.ts | GET
/api/logstash/cluster\npipeline.spec.ts | GET, PUT, DELETE
/api/logstash/pipeline/{id}\npipelines_delete.spec.ts | POST
/api/logstash/pipelines/delete\npipelines_list.spec.ts | GET
/api/logstash/pipelines (was skipped in FTR\n— now unskipped)\n\n- No
esArchiver — `.logstash` is a restricted system index; bulk\nindexing is
blocked even for superuser. Pipelines are seeded
via\n`esClient.logstash.putPipeline` in `beforeAll` per spec instead\n-
Minimum roles — `LOGSTASH_MANAGER_ROLE` (manage_logstash_pipelines)\nfor
pipeline ops; `LOGSTASH_CLUSTER_ROLE` (monitor) for the
cluster\nendpoint (FTR used superuser for everything)\n- Stateful only —
no serverless Logstash tests exist or were added\n- Coverage
improvements — added 404 assertion for non-existing pipeline\nGET;
explicit numSuccesses/numErrors check on bulk delete;
sort-order\nassertion on pipeline list\n\n---------\n\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"4720f964d72ec70a6a13345dd656949dc0613400","branchLabelMapping":{"^v9.5.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","backport:all-open","v9.5.0","v9.3.4","v9.2.9"],"title":"[scout]
migrate FTR logstash api
tests","number":262953,"url":"https://github.com/elastic/kibana/pull/262953","mergeCommit":{"message":"[scout]
migrate FTR logstash api tests (#262953)\n\n## Summary\n\nCloses
https://github.com/elastic/appex-qa-team/issues/851\n\nLogstash API
tests migrated from FTR → Scout\n\nSource:
`x-pack/platform/test/api_integration/apis/logstash/` (5 test\nfiles,
now deleted)\nTarget:
`x-pack/platform/plugins/private/logstash/test/scout/api/`\n\nNew spec
files:\n\n\nFile | Covers\n-- | --\ncluster.spec.ts | GET
/api/logstash/cluster\npipeline.spec.ts | GET, PUT, DELETE
/api/logstash/pipeline/{id}\npipelines_delete.spec.ts | POST
/api/logstash/pipelines/delete\npipelines_list.spec.ts | GET
/api/logstash/pipelines (was skipped in FTR\n— now unskipped)\n\n- No
esArchiver — `.logstash` is a restricted system index; bulk\nindexing is
blocked even for superuser. Pipelines are seeded
via\n`esClient.logstash.putPipeline` in `beforeAll` per spec instead\n-
Minimum roles — `LOGSTASH_MANAGER_ROLE` (manage_logstash_pipelines)\nfor
pipeline ops; `LOGSTASH_CLUSTER_ROLE` (monitor) for the
cluster\nendpoint (FTR used superuser for everything)\n- Stateful only —
no serverless Logstash tests exist or were added\n- Coverage
improvements — added 404 assertion for non-existing pipeline\nGET;
explicit numSuccesses/numErrors check on bulk delete;
sort-order\nassertion on pipeline list\n\n---------\n\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"4720f964d72ec70a6a13345dd656949dc0613400"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.5.0","branchLabelMappingKey":"^v9.5.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/262953","number":262953,"mergeCommit":{"message":"[scout]
migrate FTR logstash api tests (#262953)\n\n## Summary\n\nCloses
https://github.com/elastic/appex-qa-team/issues/851\n\nLogstash API
tests migrated from FTR → Scout\n\nSource:
`x-pack/platform/test/api_integration/apis/logstash/` (5 test\nfiles,
now deleted)\nTarget:
`x-pack/platform/plugins/private/logstash/test/scout/api/`\n\nNew spec
files:\n\n\nFile | Covers\n-- | --\ncluster.spec.ts | GET
/api/logstash/cluster\npipeline.spec.ts | GET, PUT, DELETE
/api/logstash/pipeline/{id}\npipelines_delete.spec.ts | POST
/api/logstash/pipelines/delete\npipelines_list.spec.ts | GET
/api/logstash/pipelines (was skipped in FTR\n— now unskipped)\n\n- No
esArchiver — `.logstash` is a restricted system index; bulk\nindexing is
blocked even for superuser. Pipelines are seeded
via\n`esClient.logstash.putPipeline` in `beforeAll` per spec instead\n-
Minimum roles — `LOGSTASH_MANAGER_ROLE` (manage_logstash_pipelines)\nfor
pipeline ops; `LOGSTASH_CLUSTER_ROLE` (monitor) for the
cluster\nendpoint (FTR used superuser for everything)\n- Stateful only —
no serverless Logstash tests exist or were added\n- Coverage
improvements — added 404 assertion for non-existing pipeline\nGET;
explicit numSuccesses/numErrors check on bulk delete;
sort-order\nassertion on pipeline list\n\n---------\n\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"4720f964d72ec70a6a13345dd656949dc0613400"}},{"branch":"9.3","label":"v9.3.4","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/264361","number":264361,"state":"MERGED","mergeCommit":{"sha":"f4630ceb53a9f2b718492394667895d82852f9f1","message":"[9.3]
[scout] migrate FTR logstash api tests (#262953) (#264361)\n\n#
Backport\n\nThis will backport the following commits from `main` to
`9.3`:\n- [[scout] migrate FTR logstash api
tests\n(#262953)](https://github.com/elastic/kibana/pull/262953)\n\n\n\n###
Questions ?\nPlease refer to the [Backport
tool\ndocumentation](https://github.com/sorenlouv/backport)\n\n"}},{"branch":"9.2","label":"v9.2.9","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/264365","number":264365,"state":"MERGED","mergeCommit":{"sha":"c687030e1a03e0485de03e2f001ec1a68d0dc392","message":"[9.2]
[scout] migrate FTR logstash api tests (#262953) (#264365)\n\n#
Backport\n\nThis will backport the following commits from `main` to
`9.2`:\n- [[scout] migrate FTR logstash api
tests\n(#262953)](https://github.com/elastic/kibana/pull/262953)\n\n\n\n###
Questions ?\nPlease refer to the [Backport
tool\ndocumentation](https://github.com/sorenlouv/backport)\n\n"}},{"url":"https://github.com/elastic/kibana/pull/264265","number":264265,"branch":"9.4","state":"MERGED","mergeCommit":{"sha":"d3629c41337d5a523f769a50c995cb9bb8cbd8a2","message":"[9.4]
[scout] migrate FTR logstash api tests (#262953) (#264265)\n\n#
Backport\n\nThis will backport the following commits from `main` to
`9.4`:\n - [scout] migrate FTR logstash api tests (#262953)
(4720f96)\n\n\n\n### Questions ?\nPlease refer to the [Backport
tool\ndocumentation](https://github.com/sorenlouv/backport)\n\n\n\nCo-authored-by:
Dzmitry Lemechko <dzmitry.lemechko@elastic.co>"}}]}] BACKPORT-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@csr csr csr approved these changes

@kibanamachine kibanamachine Awaiting requested review from kibanamachine kibanamachine is a code owner

Assignees

No one assigned

Labels

backport This PR is a backport of another PR

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@dmlemeshko @elasticmachine @csr