Skip to content

Conversation

@ywangd
Copy link
Member

@ywangd ywangd commented Jun 19, 2025

The stateful side tests generally do not exercise multi-project. Therefore we need to use the default project-id instead of random one.

Relates: #127631

The stateful side tests generally do not exercise multi-project.
Therefore we need to use the default project-id instead of random one.

Relates: elastic#127631
@ywangd ywangd requested a review from nicktindall June 19, 2025 05:09
@ywangd ywangd added >test Issues or PRs that are addressing/adding tests :Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs v9.1.0 labels Jun 19, 2025
@ywangd
Copy link
Member Author

ywangd commented Jun 19, 2025

Here is the buildscan for the test failure.

@elasticsearchmachine elasticsearchmachine added the Team:Distributed Coordination Meta label for Distributed Coordination team label Jun 19, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-distributed-coordination (Team:Distributed Coordination)

Copy link
Contributor

@nicktindall nicktindall left a comment

Choose a reason for hiding this comment

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

LGTM

@ywangd ywangd added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Jun 19, 2025
@elasticsearchmachine elasticsearchmachine merged commit 91ee26f into elastic:main Jun 19, 2025
27 checks passed
@ywangd ywangd deleted the use-default-project-for-s3-repo-tests branch June 19, 2025 06:42
kderusso pushed a commit to kderusso/elasticsearch that referenced this pull request Jun 23, 2025
The stateful side tests generally do not exercise multi-project.
Therefore we need to use the default project-id instead of random one.

Relates: elastic#127631
mridula-s109 pushed a commit to mridula-s109/elasticsearch that referenced this pull request Jun 25, 2025
The stateful side tests generally do not exercise multi-project.
Therefore we need to use the default project-id instead of random one.

Relates: elastic#127631
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs Team:Distributed Coordination Meta label for Distributed Coordination team >test Issues or PRs that are addressing/adding tests v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants