Skip to content

[ftr] re-group configs for deployment agnostic tests#190672

Merged
dmlemeshko merged 4 commits intoelastic:mainfrom
dmlemeshko:ftr/re-group-configs-for-depl-agnostic-tests
Aug 21, 2024
Merged

[ftr] re-group configs for deployment agnostic tests#190672
dmlemeshko merged 4 commits intoelastic:mainfrom
dmlemeshko:ftr/re-group-configs-for-depl-agnostic-tests

Conversation

@dmlemeshko
Copy link
Contributor

@dmlemeshko dmlemeshko commented Aug 19, 2024

Summary

While reviewing #187924 I realised that Solution Teams will have their own stateful FTR configs and the overall number of configs in api_integration/deployment_agnostic folder might grow up soon and it is better to re-organise it in advance.

With new structure we have configs folder with 2 sub-folders: stateful and serverless
image

When Platform teams add deployment-agnostic tests, it is expected to be loaded in configs/stateful/platform.index.ts and at least one of <project>.serverless.config under configs/serverless

When Solution team adds deployment-agnostic tests (e.g. one of Oblt teams), it is expected to be loaded both in configs/stateful/oblt.index.ts and configs/serverless/oblt.index.ts

@dmlemeshko
Copy link
Contributor Author

/ci

@dmlemeshko dmlemeshko added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting FTR v8.16.0 labels Aug 19, 2024
@dmlemeshko
Copy link
Contributor Author

/ci

@dmlemeshko dmlemeshko marked this pull request as ready for review August 19, 2024 14:54
@dmlemeshko dmlemeshko requested review from a team as code owners August 19, 2024 14:54
@dmlemeshko dmlemeshko requested a review from pheyos August 19, 2024 14:55
@kibana-ci
Copy link

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

Copy link
Member

@pheyos pheyos left a comment

Choose a reason for hiding this comment

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

LGTM

@dmlemeshko dmlemeshko merged commit b50eb08 into elastic:main Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting FTR release_note:skip Skip the PR/issue when compiling release notes v8.16.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants