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

Jump to bottom

Adding support for data_hot role on data_node detection #1621

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Adding support for data_hot role on data_node detection #1621

wants to merge 1 commit into from

Conversation

gnobironts
Copy link

Fixes #1620

Proposed Changes

  • Added data_hot role to _data_node check function
  • I havn't added the data_warm role to the list as these nodes are less wanted/efficient to run shrink operations

@cla-checker-service
Copy link

cla-checker-service bot commented Oct 6, 2021 •
edited

💚 CLA has been signed

@gnobironts
Copy link
Author

❌ Author of the following commits did not sign a Contributor Agreement: bfa388f

Please, read and sign the above mentioned agreement if you want to contribute to this project

Just signed the CLA. How can I trigger a new cla-check ?

@gnobironts
Copy link
Author

❌ Author of the following commits did not sign a Contributor Agreement: bfa388f
Please, read and sign the above mentioned agreement if you want to contribute to this project

Just signed the CLA. How can I trigger a new cla-check ?

OK, I think I have the answer

@untergeek
Copy link
Member

Thank you for this. I will also have to test this with 7.15 (both the ES version and the Python Client version) before merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Shrink action incompatible with new data tier roles
2 participants
@gnobironts @untergeek