Skip to content

Conversation

@rjernst
Copy link
Member

@rjernst rjernst commented Aug 18, 2025

This commit makes the resource root of transport resources configurable when building and loading transport versions. This allows the multiple sets of transport resources to exist simultaneously when running non-modular in tests, eg in serverless.

This commit adds the manifest file name as an argument when loading
transport versions from resource files. This allows multiple manifests
to exist on the test classpath at once, eg in serverless.
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

@elasticsearchmachine elasticsearchmachine added Team:Core/Infra Meta label for core/infra team v9.2.0 serverless-linked Added by automation, don't add manually labels Aug 18, 2025
@rjernst rjernst requested a review from a team as a code owner August 19, 2025 02:39
@rjernst rjernst changed the title Allow configuring manifest file when loading transport versions Allow configuring resource root of transport versions Aug 19, 2025
@rjernst
Copy link
Member Author

rjernst commented Aug 19, 2025

@JVerwolf I changed this to allow configuring the resource root instead of manifest file name so that the entire tree of files is under a different root for serverless.

Copy link
Contributor

@JVerwolf JVerwolf left a comment

Choose a reason for hiding this comment

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

Makes sense, lgtm

@rjernst rjernst merged commit a19a74e into elastic:main Aug 19, 2025
34 checks passed
@rjernst rjernst deleted the transport/configurable_manifest_file branch August 19, 2025 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Core/Infra/Core Core issues without another label >refactoring serverless-linked Added by automation, don't add manually Team:Core/Infra Meta label for core/infra team v8.18.6 v8.19.3 v9.0.6 v9.1.3 v9.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants