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

[BUG] [Remote State] Remote state repositories are not being added to cluster state when remote data store is not enabled #16363

Open
soosinha opened this issue Oct 17, 2024 · 0 comments

Comments

@soosinha
Copy link
Member

Describe the bug

When remote cluster state and remote data store are both enabled, the remote state and remote routing table repos are added to cluster state and hence showing up in _cat/repositories API.
But when only remote cluster state is enabled and remote data store is disabled, the remote state and remote routing table repos are not being added to cluster state. However the remote state publication is working correctly since it uses the repos from RepositoriesService which get initialized from from RemoteStoreNodeService.createAndVerifyRepositories during bootstrap.
The repos get added to cluster state during the join flow by JoinTaskExecutor. JoinTaskExecutor calls remoteStoreNodeService.updateRepositoriesMetadata to fetch repos to be added to the cluster state. But this function executes only when joining node is a remote store node


This logic needs to be changed to check for nodes with only remote state enabled as well.
Similar logic is present here:
if (remoteDN.isEmpty() && node.isRemoteStoreNode()) {

Related component

Cluster Manager

To Reproduce

  1. Create a cluster with remote state enabled, remote publication enabled, remote state repo and remote routing table repo
  2. Invoke _cat/repositories API.
  3. Remote state and remote routing table repo should not be present in reponse.

Expected behavior

Remote cluster state and routing table repos should get added to cluster state and show up in _cat/repositories API.

Additional Details

Plugins
Please list all plugins currently enabled.

Screenshots
If applicable, add screenshots to help explain your problem.

Host/Environment (please complete the following information):

  • OS: [e.g. iOS]
  • Version [e.g. 22]

Additional context
Add any other context about the problem here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: ✅ Done
Development

No branches or pull requests

2 participants