Skip to content

[7.17] Shareable function for partitioning integration tests (backport #17223)#17304

Merged
donoghuc merged 1 commit into7.17from
mergify/bp/7.17/pr-17223
Mar 12, 2025
Merged

[7.17] Shareable function for partitioning integration tests (backport #17223)#17304
donoghuc merged 1 commit into7.17from
mergify/bp/7.17/pr-17223

Conversation

@mergify
Copy link
Copy Markdown
Contributor

@mergify mergify bot commented Mar 12, 2025

Release notes

[rn:skip]

What does this PR do?

For the fedramp high work https://github.com/elastic/logstash/pull/17038/files a use case for multiple scripts consuming the partitioning functionality emerged. As we look to more advanced partitioning we want to ensure that the functionality will be consumable from multiple scripts.

See #17219 (comment)

Why is it important/What is the impact to the user?

NA, test setup only.

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • [ ] I have made corresponding changes to the documentation
  • [ ] I have made corresponding change to the default configuration files (and/or docker env variables)
  • [ ] I have added tests that prove my fix is effective or that my feature works

Related issues


This is an automatic backport of pull request #17223 done by [Mergify](https://mergify.com).

For the fedramp high work https://github.com/elastic/logstash/pull/17038/files a
use case for multiple scripts consuming the partitioning functionality emerged.
As we look to more advanced partitioning we want to ensure that the
functionality will be consumable from multiple scripts.

See #17219 (comment)

(cherry picked from commit d916972)
@mergify mergify bot added the backport label Mar 12, 2025
@elasticmachine
Copy link
Copy Markdown

💚 Build Succeeded

cc @donoghuc

@donoghuc donoghuc merged commit 70b9924 into 7.17 Mar 12, 2025
4 checks passed
@donoghuc donoghuc deleted the mergify/bp/7.17/pr-17223 branch March 12, 2025 18:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants