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

[Zone Decommission] Cluster Health local API should throw decommissioned exception for local node if decommissioned #4528

Closed
imRishN opened this issue Sep 15, 2022 · 1 comment
Labels
distributed framework enhancement Enhancement or improvement to existing feature or request

Comments

@imRishN
Copy link
Member

imRishN commented Sep 15, 2022

Currently, a local healthcheck request on local decommissioned node would return cluster health response from its local cluster state saying cluster manager not discovered. But currently transport calls on a local decommissioned node won't be able to identify if it is decommissioned or not. There might be a use case to get that info from a local decommissioned node directly on local transport calls

Describe the solution you'd like
We can add a param in cluster health call to identify a node is decommissioned or not on a local node

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Additional context
Add any other context or screenshots about the feature request here.

@imRishN imRishN added enhancement Enhancement or improvement to existing feature or request untriaged labels Sep 15, 2022
@imRishN imRishN changed the title Show decommission status in Cluster Health API [RFC] Show decommission status in Cluster Health API Sep 15, 2022
@imRishN imRishN changed the title [RFC] Show decommission status in Cluster Health API [Zone Decommission] [RFC] Show decommission status in Cluster Health API Sep 15, 2022
@imRishN imRishN changed the title [Zone Decommission] [RFC] Show decommission status in Cluster Health API [Zone Decommission] Cluster Health local API should throw decommissioned exception for local node if decommissioned Jan 25, 2023
@imRishN
Copy link
Member Author

imRishN commented Mar 7, 2023

Closing the issue as PR has been merged

@imRishN imRishN closed this as completed Mar 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
distributed framework enhancement Enhancement or improvement to existing feature or request
Projects
None yet
Development

No branches or pull requests

2 participants