Skip to content

Comments

[ska] relocate deployment-agnostic platform tests#223534

Merged
dmlemeshko merged 51 commits intoelastic:mainfrom
dmlemeshko:ska/relocate-da-platform-tests
Jul 7, 2025
Merged

[ska] relocate deployment-agnostic platform tests#223534
dmlemeshko merged 51 commits intoelastic:mainfrom
dmlemeshko:ska/relocate-da-platform-tests

Conversation

@dmlemeshko
Copy link
Contributor

@dmlemeshko dmlemeshko commented Jun 12, 2025

Summary

Part of https://github.com/elastic/kibana-team/issues/1503

This PR is mostly about moving platform tests from x-pack/test/api_integration/deployment_agnostic

x-pack/test/api_integration/deployment_agnostic
     | - configs/
     | - default_configs/
     | - apis/

After:

x-pack/platform/test/api_integration_deployment_agnostic
     | - configs/
     | - default_configs/
     | - apis/

@dmlemeshko dmlemeshko self-assigned this Jun 12, 2025
dmlemeshko and others added 28 commits June 12, 2025 14:12
@dmlemeshko dmlemeshko requested a review from jeramysoucy July 7, 2025 12:50
@dmlemeshko
Copy link
Contributor Author

#223534 (comment)

@jeramysoucy FTR service and its references was updated to spacesSupertest in d65450c

Copy link
Contributor

@viduni94 viduni94 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

obs-ai-assistant changes LGTM
Thank you

Copy link
Contributor

@jeramysoucy jeramysoucy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Kibana Security will open an issue to take a look at our custom spaces supertest service and see if we should refactor it.

@dmlemeshko dmlemeshko enabled auto-merge (squash) July 7, 2025 13:03
Copy link
Contributor

@gsoldevila gsoldevila left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Core changes LGTM (5 files moved without changes)

@dmlemeshko dmlemeshko merged commit 47077ef into elastic:main Jul 7, 2025
14 checks passed
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Unknown metric groups

ESLint disabled line counts

id before after diff
@kbn/test-suites-xpack-platform 532 83 -449

Total ESLint disabled count

id before after diff
@kbn/test-suites-xpack-platform 539 90 -449

History

cc @dmlemeshko

@dmlemeshko
Copy link
Contributor Author

💚 All backports created successfully

Status Branch Result
9.1

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

dmlemeshko added a commit to dmlemeshko/kibana that referenced this pull request Jul 9, 2025
## Summary

Part of elastic/kibana-team#1503

This PR is mostly about moving platform tests from
x-pack/test/api_integration/deployment_agnostic

```
x-pack/test/api_integration/deployment_agnostic
     | - configs/
     | - default_configs/
     | - apis/
```

After:
```
x-pack/platform/test/api_integration_deployment_agnostic
     | - configs/
     | - default_configs/
     | - apis/
```

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit 47077ef)

# Conflicts:
#	.buildkite/ftr_platform_stateful_configs.yml
#	.buildkite/ftr_security_serverless_configs.yml
#	.github/CODEOWNERS
#	x-pack/platform/plugins/shared/onechat/public/application/pages/chat.tsx
#	x-pack/platform/test/api_integration_deployment_agnostic/default_configs/fixtures/package_registry_config.yml
#	x-pack/platform/test/api_integration_deployment_agnostic/default_configs/serverless.config.base.ts
#	x-pack/platform/test/api_integration_deployment_agnostic/default_configs/stateful.config.base.ts
#	x-pack/platform/test/spaces_api_integration/deployment_agnostic/services/role_scoped_supertest.ts
#	x-pack/platform/test/tsconfig.json
#	x-pack/test/api_integration/deployment_agnostic/apis/observability/apm/data_view/static.spec.ts
#	x-pack/test/api_integration/deployment_agnostic/configs/serverless/oblt.ai_assistant_local.serverless.config.ts
#	x-pack/test/api_integration/deployment_agnostic/configs/stateful/oblt.ai_assistant_local.stateful.config.ts
#	x-pack/test/spaces_api_integration/common/services.ts
#	x-pack/test/spaces_api_integration/deployment_agnostic/security_and_spaces/serverless.config.ts
#	x-pack/test/spaces_api_integration/deployment_agnostic/security_and_spaces/serverless.copy_to_space.config.ts
#	x-pack/test/spaces_api_integration/deployment_agnostic/security_and_spaces/stateful.config_trial.ts
#	x-pack/test/spaces_api_integration/deployment_agnostic/security_and_spaces/stateful.copy_to_space.config_trial.ts
#	x-pack/test/spaces_api_integration/deployment_agnostic/services/index.ts
#	x-pack/test/spaces_api_integration/deployment_agnostic/services/role_scoped_supertest.ts
#	x-pack/test/spaces_api_integration/deployment_agnostic/services/spaces_supertest.ts
#	x-pack/test/tsconfig.json
@dmlemeshko dmlemeshko added backport:version Backport to applied version labels and removed backport:skip This PR does not require backporting labels Jul 9, 2025
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.19, 9.1

https://github.com/elastic/kibana/actions/runs/16174056701

@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.19, 9.1

https://github.com/elastic/kibana/actions/runs/16174056716

@kibanamachine
Copy link
Contributor

💔 All backports failed

Status Branch Result
8.19 Backport failed because of merge conflicts
9.1 Backport failed because of merge conflicts

You might need to backport the following PRs to 9.1:
- [ska] relocate upgrade, usage_collection, search_sessions_integration and licensing tests (#226669)
- [OneChat] Simple Chat UI (#222816)

Manual backport

To create the backport manually run:

node scripts/backport --pr 223534

Questions ?

Please refer to the Backport tool documentation

@kibanamachine
Copy link
Contributor

💔 All backports failed

Status Branch Result
8.19 Backport failed because of merge conflicts
9.1 Backport failed because of merge conflicts

Manual backport

To create the backport manually run:

node scripts/backport --pr 223534

Questions ?

Please refer to the Backport tool documentation

dmlemeshko added a commit to dmlemeshko/kibana that referenced this pull request Jul 10, 2025
## Summary

Part of elastic/kibana-team#1503

This PR is mostly about moving platform tests from
x-pack/test/api_integration/deployment_agnostic

```
x-pack/test/api_integration/deployment_agnostic
     | - configs/
     | - default_configs/
     | - apis/
```

After:
```
x-pack/platform/test/api_integration_deployment_agnostic
     | - configs/
     | - default_configs/
     | - apis/
```

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit 47077ef)

# Conflicts:
#	.github/CODEOWNERS
#	x-pack/platform/plugins/shared/onechat/public/application/pages/chat.tsx
#	x-pack/test/api_integration/deployment_agnostic/apis/observability/apm/data_view/static.spec.ts
#	x-pack/test/api_integration/deployment_agnostic/configs/serverless/oblt.ai_assistant_local.serverless.config.ts
#	x-pack/test/api_integration/deployment_agnostic/configs/stateful/oblt.ai_assistant_local.stateful.config.ts
@dmlemeshko
Copy link
Contributor Author

💚 All backports created successfully

Status Branch Result
9.1

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Jul 10, 2025
@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.
cc: @dmlemeshko

dmlemeshko added a commit that referenced this pull request Jul 11, 2025
…27232)

# Backport

This will backport the following commits from `main` to `9.1`:
- [[ska] relocate deployment-agnostic platform tests
(#223534)](#223534)

<!--- Backport version: 10.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":"2025-07-07T16:58:30Z","message":"[ska]
relocate deployment-agnostic platform tests (#223534)\n\n##
Summary\n\nPart of
https://github.com/elastic/kibana-team/issues/1503\n\nThis PR is mostly
about moving platform tests
from\nx-pack/test/api_integration/deployment_agnostic\n\n```\nx-pack/test/api_integration/deployment_agnostic\n
| - configs/\n | - default_configs/\n | - apis/\n```\n\nAfter:
\n```\nx-pack/platform/test/api_integration_deployment_agnostic\n | -
configs/\n | - default_configs/\n | -
apis/\n```\n\n---------\n\nCo-authored-by: kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"47077ef858735687d39d1d70455786b88358ff5d","branchLabelMapping":{"^v9.2.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","backport:skip","v9.1.0","v8.19.0","v9.2.0"],"title":"[ska]
relocate deployment-agnostic platform
tests","number":223534,"url":"https://github.com/elastic/kibana/pull/223534","mergeCommit":{"message":"[ska]
relocate deployment-agnostic platform tests (#223534)\n\n##
Summary\n\nPart of
https://github.com/elastic/kibana-team/issues/1503\n\nThis PR is mostly
about moving platform tests
from\nx-pack/test/api_integration/deployment_agnostic\n\n```\nx-pack/test/api_integration/deployment_agnostic\n
| - configs/\n | - default_configs/\n | - apis/\n```\n\nAfter:
\n```\nx-pack/platform/test/api_integration_deployment_agnostic\n | -
configs/\n | - default_configs/\n | -
apis/\n```\n\n---------\n\nCo-authored-by: kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"47077ef858735687d39d1d70455786b88358ff5d"}},"sourceBranch":"main","suggestedTargetBranches":["9.1","8.19"],"targetPullRequestStates":[{"branch":"9.1","label":"v9.1.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.19","label":"v8.19.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.2.0","branchLabelMappingKey":"^v9.2.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/223534","number":223534,"mergeCommit":{"message":"[ska]
relocate deployment-agnostic platform tests (#223534)\n\n##
Summary\n\nPart of
https://github.com/elastic/kibana-team/issues/1503\n\nThis PR is mostly
about moving platform tests
from\nx-pack/test/api_integration/deployment_agnostic\n\n```\nx-pack/test/api_integration/deployment_agnostic\n
| - configs/\n | - default_configs/\n | - apis/\n```\n\nAfter:
\n```\nx-pack/platform/test/api_integration_deployment_agnostic\n | -
configs/\n | - default_configs/\n | -
apis/\n```\n\n---------\n\nCo-authored-by: kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"47077ef858735687d39d1d70455786b88358ff5d"}}]}]
BACKPORT-->

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
@kibanamachine kibanamachine removed the backport missing Added to PRs automatically when the are determined to be missing a backport. label Jul 11, 2025
dmlemeshko added a commit that referenced this pull request Jul 14, 2025
## Summary

replacement PR for #223534 backport to 8.19

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Co-authored-by: Robert Oskamp <traeluki@gmail.com>
kertal pushed a commit to kertal/kibana that referenced this pull request Jul 25, 2025
## Summary

Part of elastic/kibana-team#1503

This PR is mostly about moving platform tests from
x-pack/test/api_integration/deployment_agnostic

```
x-pack/test/api_integration/deployment_agnostic
     | - configs/
     | - default_configs/
     | - apis/
```

After: 
```
x-pack/platform/test/api_integration_deployment_agnostic
     | - configs/
     | - default_configs/
     | - apis/
```

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:version Backport to applied version labels release_note:skip Skip the PR/issue when compiling release notes v9.1.0 v9.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.