Skip to content
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

Data-containing data tiers roles are not mapped onto DATA legacy role #71464

Closed
DaveCTurner opened this issue Apr 8, 2021 · 2 comments
Closed
Assignees
Labels
>bug Team:Data Management Meta label for data/management team

Comments

@DaveCTurner
Copy link
Contributor

DaveCTurner commented Apr 8, 2021

Elasticsearch version (bin/elasticsearch --version): Rolling upgrade from <7.3 to ≥7.10

Description of the problem including expected versus actual behavior:

We have special handling for mapping the new pluggable master/data/ingest roles onto their legacy counterparts when talking to pre-7.3 nodes which did not support pluggable roles:

// an old node will only understand legacy roles since pluggable roles is a new concept
final List<DiscoveryNodeRole> rolesToWrite =
roles.stream().filter(DiscoveryNodeRole.LEGACY_ROLES::contains).collect(Collectors.toList());
out.writeVInt(rolesToWrite.size());
for (final DiscoveryNodeRole role : rolesToWrite) {
if (role == DiscoveryNodeRole.MASTER_ROLE) {
out.writeEnum(LegacyRole.MASTER);
} else if (role == DiscoveryNodeRole.DATA_ROLE) {
out.writeEnum(LegacyRole.DATA);
} else if (role == DiscoveryNodeRole.INGEST_ROLE) {
out.writeEnum(LegacyRole.INGEST);
}
}

However this code does not use DiscoveryNodeRole#getCompatibilityRole() so it does not map the data-tiers-related roles DATA_{HOT,WARM,COLD,FROZEN} onto the legacy data role.

Relates #60994 and #63581 so I'm labelling this similarly.

@elasticmachine elasticmachine added the Team:Data Management Meta label for data/management team label Apr 8, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-features (Team:Core/Features)

@andreidan
Copy link
Contributor

Closing as fixed by #71628

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>bug Team:Data Management Meta label for data/management team
Projects
None yet
Development

No branches or pull requests

3 participants