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

[CCR] Edit follower index: advanced settings are not persisted #62943

Closed
sebelga opened this issue Apr 8, 2020 · 5 comments · Fixed by #63653
Closed

[CCR] Edit follower index: advanced settings are not persisted #62943

sebelga opened this issue Apr 8, 2020 · 5 comments · Fixed by #63653
Assignees
Labels
bug Fixes for quality problems that affect the customer experience Feature:CCR and Remote Clusters Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more

Comments

@sebelga
Copy link
Contributor

sebelga commented Apr 8, 2020

When editing a follower index and changing an advanced setting, the changes is not persisted.

Changing "Max read request operation count" to 2000
Screenshot 2020-04-08 at 15 55 29

Once saved, the value is still 5120

Screenshot 2020-04-08 at 15 55 46

@sebelga sebelga added bug Fixes for quality problems that affect the customer experience Feature:CCR and Remote Clusters Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more labels Apr 8, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/es-ui (Team:Elasticsearch UI)

@cjcenizal cjcenizal self-assigned this Apr 9, 2020
@cjcenizal
Copy link
Contributor

cjcenizal commented Apr 9, 2020

Looks like this bug is on the ES side: elastic/elasticsearch#54996. This manifests in the UI because creating a follower index performs a create request and then a fetch request in rapid succession. When the new follower index is fetched it's in a paused state (which ES later changes to an active state). When you then edit the new follower index it's still in this stale paused state, which means there's no advanced settings data to populate the form.

@sebelga
Copy link
Contributor Author

sebelga commented Apr 13, 2020

Thanks for investigating. I do think our UI should handle this so the user does not think there is a bug and the follower index hasn't been updated.
A solution is, after editing a follower index, have a request polling in the details flyout and wait until the follower index is active again to display the stats. WDYT?

@sebelga
Copy link
Contributor Author

sebelga commented Apr 14, 2020

I have been playing around and it is clearly a bug.

I have manually done all the steps in console and they work as expected, the value gets updated (here max_read_request_operation_count set to 2000)

Screenshot 2020-04-14 at 09 51 26

Screenshot 2020-04-14 at 09 53 35

Once we edit the follower index in the UI, don't touch anything and click "Update", we reverted the updated values to the default ones.

Screenshot 2020-04-14 at 09 53 59

@cjcenizal
Copy link
Contributor

Thanks for digging deeper @sebelga! I found the problem. When we migrated the server to NP we left out some route validation that strips the body from the requests (https://github.com/elastic/kibana/pull/60121/files#diff-8ae5a51616bdcb1706f0aaf1ab1581c9R173). I'll get a fix up shortly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience Feature:CCR and Remote Clusters Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants