Skip to content

Conversation

@thecoop
Copy link
Member

@thecoop thecoop commented Mar 10, 2023

This change to use index compatibility was done in #41517, it should really be node compatibility.

Also fix up a few other instances where index compatibility was used instead of node compatibility in the same commit

@thecoop thecoop added :Core/Infra/Transport API Transport client API >refactoring :Security/Security Security issues without another label labels Mar 10, 2023
@elasticsearchmachine elasticsearchmachine added Team:Core/Infra Meta label for core/infra team Team:Security Meta label for security team v8.8.0 labels Mar 10, 2023
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-security (Team:Security)

@elasticsearchmachine
Copy link
Collaborator

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

Copy link
Member

@rjernst rjernst left a comment

Choose a reason for hiding this comment

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

LGTM

@thecoop thecoop requested a review from ywangd March 10, 2023 15:15
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

@thecoop thecoop merged commit 07a8e1e into elastic:main Mar 14, 2023
@thecoop thecoop deleted the transportversion-tokenmetadata branch March 14, 2023 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Core/Infra/Transport API Transport client API >refactoring :Security/Security Security issues without another label Team:Core/Infra Meta label for core/infra team Team:Security Meta label for security team v8.8.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants