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

[Reporting] csv report always includes frozen indices #152884

Closed
Dosant opened this issue Mar 8, 2023 · 2 comments · Fixed by #153359
Closed

[Reporting] csv report always includes frozen indices #152884

Dosant opened this issue Mar 8, 2023 · 2 comments · Fixed by #153359
Labels
bug Fixes for quality problems that affect the customer experience (Deprecated) Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience)

Comments

@Dosant
Copy link
Contributor

Dosant commented Mar 8, 2023

Kibana version: 8.6.0

This is an issue to track a bug Kibana where csv report always includes frozen indices. Even if search:includeFrozen is false.

The bug appeared when we switch to point-in-time API inside csv report code #144201.
It appears that elasticsearch always includes frozen indices when using PIT API elastic/elasticsearch#94377

If the issue is solved on the elasticsearch side, then in Kibana everything should work as expected. This issue can be closed when the fixed behavior is tested or if elasticsearch decides to not fix the issue.
@Dosant Dosant added bug Fixes for quality problems that affect the customer experience (Deprecated) Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience) labels Mar 8, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/appex-sharedux (Team:SharedUX)

@Dosant Dosant mentioned this issue Mar 8, 2023
Merged
@tsullivan
Copy link
Member

blocked by elastic/elasticsearch#94377

@Dosant Dosant mentioned this issue Mar 21, 2023
Merged
Dosant added a commit that referenced this issue Mar 23, 2023
@Dosant
[Reporting] Remove no longer relevant comment about frozen indices (#…
847e78e 
…153359)

## Summary

close #152884 since
elastic/elasticsearch#94377 was fixed. I tested
it, works well except for one thing I noticed deleting a PIT created for a frozen index
without `include_throttled=false` will result in a 404 error:

```
POST /test/_pit?keep_alive=1m // `test` index is frozen

// returns
{
  "id": "45XtAwAA"
}

DELETE /_pit
{
    "id" : "45XtAwAA"
}

// returns 404
{
  "succeeded": true,
  "num_freed": 0
}

```

Not sure if this expected
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Fixes for quality problems that affect the customer experience (Deprecated) Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience)
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Reporting] Remove no longer relevant comment about frozen indices Dosant/kibana
3 participants
@tsullivan @Dosant @elasticmachine