-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
PNG report doesn't create any report #29385
Comments
Pinging @elastic/kibana-app |
Did not attempt this, but if it's a legit error I think this is a 7.0 blocker and a regression. Labeled as such. |
Looking into this now |
Based on the error message, it looks like the EDIT: for reasoning, this might happening from deleting the .reporting index while Kibana is running, and then schedule a Report. The index might be getting auto-created with default mappings. |
Taking a look now |
When you start Elasticsearch, you might be seeing this in the logs:
This shows that the index is getting created from something indexing documents with the Bulk API, but there was no mapping. This looks related to https://github.com/elastic/kibana/pull/27094/files#r241225441 This: https://github.com/tsullivan/kibana/blob/7e20835/x-pack/plugins/reporting/server/lib/esqueue/helpers/create_index.js#L95 should check for the kind of error that it is ignoring :D |
When I allow an error to be thrown from that part of the code, I see this info in the error:
|
Sounds like it might be related to the Elastisearch root type breaking change, Chris Davies fixed up the tests for reporting here: #29184 Maybe the actual code and mappings needs to change as well. |
Interesting... I've got stuff lined up to work once I'm through my AM meetings. |
After merging #29184 locally, I'm seeing the Reporting index get created successfully when I put a job into the queue. I saw some changes go into ESQueue in that PR. However, our doc type is |
Clarification: Based on https://www.elastic.co/guide/en/elasticsearch/reference/6.x/removal-of-types.html#_schedule_for_removal_of_mapping_types, in ES 7.x:
I'm not sure how this is going to work if our doc type is not
While the bug seems fixed in master for now, I think we're not completely done |
I think we're good with the change Chris made to make Kibana ES7-compliant, might need to revisit this in the 8.x series. I'll verify this is working locally and close this issue out if so |
Verified that PNG and CSV reports are working on latest (7.x.x). Closing! |
Ah yea, probably this line: https://github.com/elastic/kibana/pull/29184/files#diff-857ec3bbdaea920d52c2d00c124c8b2bR91 is what fixed it. |
Kibana version: master
I started trial license and tried to export png from a dashboard, notification that the report is in progress appeared but nothing happened beyond that. The report doesn't doesn't show up in the reporting page and I got an error in Kibana
The text was updated successfully, but these errors were encountered: