Skip to content

[ska] relocate dataset quality api-integration tests#227826

Merged
dmlemeshko merged 8 commits intoelastic:mainfrom
dmlemeshko:ska/relocate-dataset_quality
Jul 15, 2025
Merged

[ska] relocate dataset quality api-integration tests#227826
dmlemeshko merged 8 commits intoelastic:mainfrom
dmlemeshko:ska/relocate-dataset_quality

Conversation

@dmlemeshko
Copy link
Copy Markdown
Contributor

@dmlemeshko dmlemeshko commented Jul 14, 2025

Summary

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

This PR is mostly about moving platform data quality api tests from x-pack/test

x-pack/test/
     | - dataset_quality_api_integration

After:

x-pack/solutions/observability/test/
     | - dataset_quality_api_integration

I also did a small cleanup of no longer used duplicated services under x-pack/test/api_integration

@dmlemeshko dmlemeshko self-assigned this Jul 14, 2025
@dmlemeshko dmlemeshko added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting v9.1.0 v8.19.0 v9.2.0 labels Jul 14, 2025
@dmlemeshko dmlemeshko marked this pull request as ready for review July 14, 2025 15:28
@dmlemeshko dmlemeshko requested review from a team as code owners July 14, 2025 15:28
@dmlemeshko dmlemeshko requested a review from a team July 14, 2025 15:28
@dmlemeshko dmlemeshko requested a review from a team as a code owner July 14, 2025 15:28
Copy link
Copy Markdown
Contributor

@rmyz rmyz left a comment

Choose a reason for hiding this comment

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

infra & services changes LGTM

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Unknown metric groups

ESLint disabled line counts

id before after diff
@kbn/test-suites-xpack 137 135 -2
@kbn/test-suites-xpack-observability 10 11 +1
total -1

Total ESLint disabled count

id before after diff
@kbn/test-suites-xpack 149 147 -2
@kbn/test-suites-xpack-observability 13 14 +1
total -1

History

cc @dmlemeshko

Comment thread .github/CODEOWNERS


# Data Discovery
/x-pack/test/api_integration/services/data_view_api.ts @elastic/kibana-data-discovery
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I see this file was moved to x-pack/test/api_integration/services/data_view_api.ts (in a separate PR). Why not add a CODEOWNER entry for the new location?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

+1, please let's add a CODEOWNER entry for the new file's location instead of removing it completely 🙏

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

at the moment appex-qa owns all services, but agree to move it to the Team

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

fixed in 02371e1

Copy link
Copy Markdown
Member

@csr csr left a comment

Choose a reason for hiding this comment

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

LGTM

Comment thread .github/CODEOWNERS


# Data Discovery
/x-pack/test/api_integration/services/data_view_api.ts @elastic/kibana-data-discovery
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

+1, please let's add a CODEOWNER entry for the new file's location instead of removing it completely 🙏

@kibanamachine
Copy link
Copy Markdown
Contributor

Starting backport for target branches: 8.19, 9.1

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

@kibanamachine
Copy link
Copy Markdown
Contributor

Starting backport for target branches: 8.19, 9.1

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

@kibanamachine
Copy link
Copy Markdown
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 227826

Questions ?

Please refer to the Backport tool documentation

1 similar comment
@kibanamachine
Copy link
Copy Markdown
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 227826

Questions ?

Please refer to the Backport tool documentation

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

Part of elastic/kibana-team#1503

This PR is mostly about moving platform data quality api tests from
x-pack/test

```
x-pack/test/
     | - dataset_quality_api_integration
```

After:
```
x-pack/solutions/observability/test/
     | - dataset_quality_api_integration
```

I also did a small cleanup of no longer used duplicated services under
`x-pack/test/api_integration`

---------

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

# Conflicts:
#	.github/CODEOWNERS
#	x-pack/test/api_integration/services/slo.ts
#	x-pack/test/functional/services/index.ts
#	x-pack/test/tsconfig.json
@dmlemeshko
Copy link
Copy Markdown
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 17, 2025
@kibanamachine
Copy link
Copy Markdown
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 to dmlemeshko/kibana that referenced this pull request Jul 17, 2025
## Summary

Part of elastic/kibana-team#1503

This PR is mostly about moving platform data quality api tests from
x-pack/test

```
x-pack/test/
     | - dataset_quality_api_integration
```

After:
```
x-pack/solutions/observability/test/
     | - dataset_quality_api_integration
```

I also did a small cleanup of no longer used duplicated services under
`x-pack/test/api_integration`

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit e7c6fe5)
@dmlemeshko
Copy link
Copy Markdown
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 that referenced this pull request Jul 17, 2025
…228217)

# Backport

This will backport the following commits from `main` to `9.1`:
- [[ska] relocate dataset quality api-integration tests
(#227826)](#227826)

<!--- 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-15T11:35:02Z","message":"[ska]
relocate dataset quality api-integration tests (#227826)\n\n##
Summary\n\nPart of
https://github.com/elastic/kibana-team/issues/1503\n\nThis PR is mostly
about moving platform data quality api tests
from\nx-pack/test\n\n```\nx-pack/test/\n | -
dataset_quality_api_integration\n```\n\nAfter:
\n```\nx-pack/solutions/observability/test/\n | -
dataset_quality_api_integration\n```\n\nI also did a small cleanup of no
longer used duplicated services
under\n`x-pack/test/api_integration`\n\n---------\n\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"e7c6fe5717ba2ccc5a7b1a65d3697c9e2c109c41","branchLabelMapping":{"^v9.2.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","backport:version","v9.1.0","v8.19.0","v9.2.0"],"title":"[ska]
relocate dataset quality api-integration
tests","number":227826,"url":"https://github.com/elastic/kibana/pull/227826","mergeCommit":{"message":"[ska]
relocate dataset quality api-integration tests (#227826)\n\n##
Summary\n\nPart of
https://github.com/elastic/kibana-team/issues/1503\n\nThis PR is mostly
about moving platform data quality api tests
from\nx-pack/test\n\n```\nx-pack/test/\n | -
dataset_quality_api_integration\n```\n\nAfter:
\n```\nx-pack/solutions/observability/test/\n | -
dataset_quality_api_integration\n```\n\nI also did a small cleanup of no
longer used duplicated services
under\n`x-pack/test/api_integration`\n\n---------\n\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"e7c6fe5717ba2ccc5a7b1a65d3697c9e2c109c41"}},"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/227826","number":227826,"mergeCommit":{"message":"[ska]
relocate dataset quality api-integration tests (#227826)\n\n##
Summary\n\nPart of
https://github.com/elastic/kibana-team/issues/1503\n\nThis PR is mostly
about moving platform data quality api tests
from\nx-pack/test\n\n```\nx-pack/test/\n | -
dataset_quality_api_integration\n```\n\nAfter:
\n```\nx-pack/solutions/observability/test/\n | -
dataset_quality_api_integration\n```\n\nI also did a small cleanup of no
longer used duplicated services
under\n`x-pack/test/api_integration`\n\n---------\n\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"e7c6fe5717ba2ccc5a7b1a65d3697c9e2c109c41"}}]}]
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 17, 2025
Bluefinger pushed a commit to Bluefinger/kibana that referenced this pull request Jul 22, 2025
## Summary

Part of elastic/kibana-team#1503

This PR is mostly about moving platform data quality api tests from
x-pack/test

```
x-pack/test/
     | - dataset_quality_api_integration
```

After: 
```
x-pack/solutions/observability/test/
     | - dataset_quality_api_integration
```

I also did a small cleanup of no longer used duplicated services under
`x-pack/test/api_integration`

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.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 data quality api tests from
x-pack/test

```
x-pack/test/
     | - dataset_quality_api_integration
```

After: 
```
x-pack/solutions/observability/test/
     | - dataset_quality_api_integration
```

I also did a small cleanup of no longer used duplicated services under
`x-pack/test/api_integration`

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
@fake-haris
Copy link
Copy Markdown
Contributor

💚 All backports created successfully

Status Branch Result
8.19

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

Questions ?

Please refer to the Backport tool documentation

fake-haris added a commit that referenced this pull request Aug 5, 2025
…#230438)

# Backport

This will backport the following commits from `main` to `8.19`:
- [[ska] relocate dataset quality api-integration tests
(#227826)](#227826)

<!--- 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-15T11:35:02Z","message":"[ska]
relocate dataset quality api-integration tests (#227826)\n\n##
Summary\n\nPart of
https://github.com/elastic/kibana-team/issues/1503\n\nThis PR is mostly
about moving platform data quality api tests
from\nx-pack/test\n\n```\nx-pack/test/\n | -
dataset_quality_api_integration\n```\n\nAfter:
\n```\nx-pack/solutions/observability/test/\n | -
dataset_quality_api_integration\n```\n\nI also did a small cleanup of no
longer used duplicated services
under\n`x-pack/test/api_integration`\n\n---------\n\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"e7c6fe5717ba2ccc5a7b1a65d3697c9e2c109c41","branchLabelMapping":{"^v9.2.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","backport:version","v9.1.0","v8.19.0","v9.2.0"],"title":"[ska]
relocate dataset quality api-integration
tests","number":227826,"url":"https://github.com/elastic/kibana/pull/227826","mergeCommit":{"message":"[ska]
relocate dataset quality api-integration tests (#227826)\n\n##
Summary\n\nPart of
https://github.com/elastic/kibana-team/issues/1503\n\nThis PR is mostly
about moving platform data quality api tests
from\nx-pack/test\n\n```\nx-pack/test/\n | -
dataset_quality_api_integration\n```\n\nAfter:
\n```\nx-pack/solutions/observability/test/\n | -
dataset_quality_api_integration\n```\n\nI also did a small cleanup of no
longer used duplicated services
under\n`x-pack/test/api_integration`\n\n---------\n\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"e7c6fe5717ba2ccc5a7b1a65d3697c9e2c109c41"}},"sourceBranch":"main","suggestedTargetBranches":["8.19"],"targetPullRequestStates":[{"branch":"9.1","label":"v9.1.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/228217","number":228217,"state":"MERGED","mergeCommit":{"sha":"818f613f82139b4ebbf6f941b4ebd37dad8a620f","message":"[9.1]
[ska] relocate dataset quality api-integration tests (#227826)
(#228217)\n\n# Backport\n\nThis will backport the following commits from
`main` to `9.1`:\n- [[ska] relocate dataset quality api-integration
tests\n(#227826)](https://github.com/elastic/kibana/pull/227826)\n\n\n\n###
Questions ?\nPlease refer to the [Backport
tool\ndocumentation](https://github.com/sorenlouv/backport)\n\n\n\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>"}},{"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/227826","number":227826,"mergeCommit":{"message":"[ska]
relocate dataset quality api-integration tests (#227826)\n\n##
Summary\n\nPart of
https://github.com/elastic/kibana-team/issues/1503\n\nThis PR is mostly
about moving platform data quality api tests
from\nx-pack/test\n\n```\nx-pack/test/\n | -
dataset_quality_api_integration\n```\n\nAfter:
\n```\nx-pack/solutions/observability/test/\n | -
dataset_quality_api_integration\n```\n\nI also did a small cleanup of no
longer used duplicated services
under\n`x-pack/test/api_integration`\n\n---------\n\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"e7c6fe5717ba2ccc5a7b1a65d3697c9e2c109c41"}}]}]
BACKPORT-->

---------

Co-authored-by: Dzmitry Lemechko <dzmitry.lemechko@elastic.co>
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 v8.19.2 v9.1.0 v9.2.0