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

Handling for system indices #161

Closed
s-vkropotko opened this issue Mar 31, 2021 · 2 comments
Closed

Handling for system indices #161

s-vkropotko opened this issue Mar 31, 2021 · 2 comments

Comments

@s-vkropotko
Copy link

ES-Operator doesn't separate system indices and ordinary users indices.
It's already added a deprecation alert about new default behavior for system indices, e.g.

Deprecation: this request accesses system indices: [.kibana_2], but in a future major version, direct access to system indices will be prevented by default

So it makes sense to exclude system indices from es-opeartor management.

otrosien added a commit that referenced this issue May 14, 2021
Closes #161

Signed-off-by: Oliver <oliver.trosien@zalando.de>
@otrosien
Copy link
Member

otrosien commented Jun 4, 2021

@s-vkropotko alternative to the current implementation: Should the ES-Operator move the system indices away (exclude allocation of the system index from the current EDS). They not only shouldn't be scaled up or down but they could also be harming an even balancing out of user shards. WDYT?

@s-vkropotko
Copy link
Author

s-vkropotko commented Jun 16, 2021

Hi!
According to this elastic/elasticsearch#50251
Smarter replication for system indices (continue to use auto expand 0-1 or maybe a new scheme?)
System indices suppose to use auto_expand_replicas option for now but this can be changed in the future.
Also, users can create indices with this option(auto_expand_replicas) then ES-Op cannot manage a number of replicas for these indices (as the direct update of number_of_replicas acceptable but not applicable).

>> curl http://localhost:9200/_cat/indices
green open test5 Ietp5gGtQcK0WBWsbnnh3g 1 1 0 0 416b 208b
>> curl -X PUT http://localhost:9200/test5/_settings --data '{"settings": {"number_of_replicas": 2}}' -H 'Content-type: application/json'
{"acknowledged":true}
>> curl "http://localhost:9200/test5/_settings?pretty=true"
{
  "test5" : {
    "settings" : {
      "index" : {
        "number_of_shards" : "1",
        "auto_expand_replicas" : "0-1",
        "provided_name" : "test",
...
        "number_of_replicas" : "1",
...

So maybe we can try to update the function GetIndices to return only manageable indices and an additional returns value of unmanageable total shards(primary+replicas), e.g. system indices and indices with auto_expand option ? Then we can use it for proper calculation of shards to nodes ratio.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants