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

Cherry-pick #10995 to 7.x: Include ip and boolean type when generating index pattern #11089

Merged
merged 1 commit into from
Mar 6, 2019

Conversation

cwurm
Copy link
Contributor

@cwurm cwurm commented Mar 5, 2019

Cherry-pick of PR #10995 to 7.x branch. Original message:

A number of fields in our fields.yml files now are type: ip (e.g. source.ip) and type: boolean, but the code generating index patterns does not know about these types yet and so does not add a type to the index pattern for those fields at all. This leads to errors in Kibana when looking at dashboards that contain references to those fields.

Output of ./auditbeat export index-pattern:

Before:

  {
    "aggregatable": true,
    "analyzed": false,
    "count": 0,
    "doc_values": true,
    "indexed": true,
    "name": "source.ip",
    "scripted": false,
    "searchable": true
  },

After:

  {
    "aggregatable": true,
    "analyzed": false,
    "count": 0,
    "doc_values": true,
    "indexed": true,
    "name": "source.ip",
    "scripted": false,
    "searchable": true,
    "type": "ip"
  },

@cwurm cwurm requested a review from a team as a code owner March 5, 2019 15:25
)

A number of fields in our `fields.yml` files now are `type: ip` (e.g. `source.ip`) and `type: boolean`, but the code generating index patterns does not know about these types yet and so does not add a `type` to the index pattern for those fields at all. This leads to errors in Kibana when looking at dashboards that contain references to those fields.

(cherry picked from commit e98026e)
@cwurm cwurm merged commit 9e01d75 into elastic:7.x Mar 6, 2019
@cwurm cwurm deleted the backport_10995_7.x branch March 6, 2019 22:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants