Skip to content

Conversation

@nicktindall
Copy link
Contributor

@nicktindall nicktindall commented Mar 27, 2025

This reverts the upgrade. We will continue to investigate on a branch.

Relates: #125668

@nicktindall nicktindall requested a review from a team as a code owner March 27, 2025 06:31
@nicktindall nicktindall changed the title Revert GCS client upgrade Revert GCS client upgrade and unmute test Mar 27, 2025
@elasticsearchmachine elasticsearchmachine added needs:triage Requires assignment of a team area label v9.1.0 labels Mar 27, 2025
@nicktindall nicktindall requested review from mhl-b and ywangd March 27, 2025 06:31
@ywangd ywangd added >non-issue :Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs and removed needs:triage Requires assignment of a team area label labels Mar 27, 2025
@elasticsearchmachine elasticsearchmachine added the Team:Distributed Coordination Meta label for Distributed Coordination team label Mar 27, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@ywangd
Copy link
Member

ywangd commented Mar 27, 2025

I'd prefer the unmute to be a separate and keep this as a pure revert.

@DaveCTurner
Copy link
Contributor

Hold off on reverting the client upgrade until we've run with #125749 for a while, hopefully we get more clues about the problem with that.

@nicktindall nicktindall force-pushed the revert_gcs_client_upgrade branch from f133873 to d7e4d6a Compare March 28, 2025 01:23
@nicktindall nicktindall changed the title Revert GCS client upgrade and unmute test Revert GCS client upgrade and additional logging Mar 28, 2025
Copy link
Contributor

@mhl-b mhl-b left a comment

Choose a reason for hiding this comment

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

LGTM

@nicktindall nicktindall force-pushed the revert_gcs_client_upgrade branch from d7e4d6a to 0bcadcb Compare March 28, 2025 04:47
@nicktindall nicktindall force-pushed the revert_gcs_client_upgrade branch from 0bcadcb to 15afa3f Compare March 28, 2025 04:47
@nicktindall nicktindall changed the title Revert GCS client upgrade and additional logging Revert GCS client upgrade Mar 28, 2025
Copy link
Member

@ywangd ywangd left a comment

Choose a reason for hiding this comment

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

LGTM

@nicktindall nicktindall merged commit a256773 into elastic:main Mar 28, 2025
18 checks passed
@nicktindall nicktindall deleted the revert_gcs_client_upgrade branch March 28, 2025 06:49
omricohenn pushed a commit to omricohenn/elasticsearch that referenced this pull request Mar 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs >non-issue Team:Distributed Coordination Meta label for Distributed Coordination team v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants