Skip to content

Replace _remote_access with REMOTE_CLUSTER_PROFILE in comment #130489

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

JeremyDahlgren
Copy link
Contributor

There isn't a _remote_access profile, it looks like the author meant to write _remote_cluster. This commit changes it to REMOTE_CLUSTER_PROFILE to match the code below and avoid confusing the reader with a profile name that doesn't exist.

There isn't a '_remote_access' profile, it looks like the
author meant to write '_remote_cluster'.  This commit changes
it to REMOTE_CLUSTER_PROFILE to match the code below and avoid
confusing the reader with a profile name that doesn't exist.
@JeremyDahlgren JeremyDahlgren requested a review from ywangd July 2, 2025 19:57
@JeremyDahlgren JeremyDahlgren added >non-issue :Distributed Coordination/Network Http and internode communication implementations Team:Distributed Coordination Meta label for Distributed Coordination team v9.2.0 labels Jul 2, 2025
@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

@JeremyDahlgren JeremyDahlgren merged commit e07f9fe into elastic:main Jul 3, 2025
32 checks passed
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 >non-issue 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