Skip to content

Conversation

JeremyDahlgren
Copy link
Contributor

Moves the test-only settings-based updateRemoteCluster() method to the RemoteClusterService unit test class.
Part of work to eliminate remaining settings-based methods in RemoteClusterService, identified in this comment of PR #133834.

Relates: ES-12864

Moves the test-only settings-based updateRemoteCluster()
method to the RemoteClusterService unit test class.
Part of work to eliminate remaining settings-based methods
in RemoteClusterService.

Relates: ES-12864
@JeremyDahlgren JeremyDahlgren added :Distributed Coordination/Network Http and internode communication implementations >refactoring Team:Distributed Coordination Meta label for Distributed Coordination team v9.2.0 labels Sep 12, 2025
@JeremyDahlgren JeremyDahlgren marked this pull request as ready for review September 12, 2025 19:35
@elasticsearchmachine
Copy link
Collaborator

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

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

I like it 👍

@JeremyDahlgren JeremyDahlgren merged commit 90197b0 into elastic:main Sep 15, 2025
34 checks passed
mridula-s109 pushed a commit to mridula-s109/elasticsearch that referenced this pull request Sep 17, 2025
Moves the test-only settings-based updateRemoteCluster()
method to the RemoteClusterService unit test class.
Part of work to eliminate remaining settings-based methods
in RemoteClusterService.

Relates: ES-12864
gmjehovich pushed a commit to gmjehovich/elasticsearch that referenced this pull request Sep 18, 2025
Moves the test-only settings-based updateRemoteCluster()
method to the RemoteClusterService unit test class.
Part of work to eliminate remaining settings-based methods
in RemoteClusterService.

Relates: ES-12864
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed Coordination/Network Http and internode communication implementations >refactoring Team:Distributed Coordination Meta label for Distributed Coordination team v9.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants