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.

Sign up for GitHub

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

Remove 'cluster_manager' role attachment when using 'node.master' deprecated setting #6331

Merged
merged 12 commits into from
Mar 28, 2023
Merged

Remove 'cluster_manager' role attachment when using 'node.master' deprecated setting #6331

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
c82bc5f
Allow only cluster_manager role when leagcy setting node.master is pr…
sandeshkr419 Mar 2, 2023
c8dff61
test modificaiton
sandeshkr419 Mar 9, 2023
7b93d8c
Add Locale.Root in String format()
sandeshkr419 Mar 9, 2023
e9bad81
./gradlew :server:spotlessApply code formatting
sandeshkr419 Mar 9, 2023
ec35774
modify settings filtering by using getRolesFromSettings()
sandeshkr419 Mar 23, 2023
ca821e0
Edit Changelog
sandeshkr419 Mar 23, 2023
a94f6e1
Add test with deprecated master and cluster manager role
sandeshkr419 Mar 24, 2023
53d9d2e
Fix role filtering from legacy settings
sandeshkr419 Mar 24, 2023
a8fc151
Override isEnabledByDefault() for cluster manager role
sandeshkr419 Mar 24, 2023
b7cda75
minor refactoring with boolean condition
sandeshkr419 Mar 28, 2023
6dffd09
Remove test flakiness
sandeshkr419 Mar 28, 2023
35327e1
Test reduce flakiness
sandeshkr419 Mar 28, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix role filtering from legacy settings 
Signed-off-by: Sandesh Kumar <sandeshkr419@gmail.com>
  • Loading branch information
@sandeshkr419
sandeshkr419 committed Mar 28, 2023
commit 53d9d2e0b6ed555bf767205fe8511b497d7ef87e
9 changes: 8 additions & 1 deletion server/src/main/java/org/opensearch/cluster/node/DiscoveryNode.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -285,7 +285,14 @@ public static Set<DiscoveryNodeRole> getRolesFromSettings(final Settings setting
validateLegacySettings(settings, roleMap);
return Collections.unmodifiableSet(new HashSet<>(NODE_ROLES_SETTING.get(settings)));
} else {
return roleMap.values().stream().filter(s -> s.isEnabledByDefault(settings)).collect(Collectors.toSet());
return roleMap.values()
.stream()
.filter(
s -> !s.roleName().equals(DiscoveryNodeRole.CLUSTER_MANAGER_ROLE.roleName())
&& s.legacySetting() != null
&& s.legacySetting().get(settings)
)
.collect(Collectors.toSet());
}
}

Expand Down