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

[Canvas] Support IP data type #85087

Merged
merged 6 commits into from
Feb 17, 2021
Merged

Conversation

cqliu1
Copy link
Contributor

@cqliu1 cqliu1 commented Dec 5, 2020

Summary

Closes #83142.

The enhancement issue above noted that this customer is unable to use Canvas without support for the ip data type.

This adds support for the ip ES data type by treating it as a string in Canvas. This will prevent ES datasource functions from throwing errors when trying to retrieve indexes with ip type fields.

Question: Should we add any other data types currently not supported in Canvas? See https://www.elastic.co/guide/en/elasticsearch/reference/current/mapping-types.html.

For maintainers

@cqliu1 cqliu1 added release_note:enhancement enhancement New value added to drive a business result Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas loe:small Small Level of Effort v8.0.0 impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. v7.11.0 Feature:Canvas labels Dec 5, 2020
@cqliu1 cqliu1 requested a review from a team as a code owner December 5, 2020 02:05
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-presentation (Team:Presentation)

@cqliu1
Copy link
Contributor Author

cqliu1 commented Dec 7, 2020

@elasticmachine merge upstream

@cqliu1 cqliu1 added the review label Dec 10, 2020
@cqliu1
Copy link
Contributor Author

cqliu1 commented Dec 10, 2020

@elasticmachine merge upstream

@cqliu1
Copy link
Contributor Author

cqliu1 commented Dec 14, 2020

@elasticmachine merge upstream

@LeeDr
Copy link
Contributor

LeeDr commented Jan 7, 2021

Since we're past 7.11.0 FF, and this isn't a bug fix, should probably bump to 7.12.0.

Actually... maybe we could call it a bug fix? We don't merge enhancements after FF.

@cqliu1 cqliu1 added v7.12.0 and removed v7.11.0 labels Jan 7, 2021
Copy link
Contributor

@clintandrewhall clintandrewhall left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's try to get this in for 7.12.

@clintandrewhall
Copy link
Contributor

@elasticmachine merge upstream

@cqliu1
Copy link
Contributor Author

cqliu1 commented Feb 16, 2021

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@cqliu1 cqliu1 merged commit aa269e1 into elastic:master Feb 17, 2021
gmmorris added a commit to gmmorris/kibana that referenced this pull request Feb 17, 2021
* master: (157 commits)
  [DOCS] Adds machine learning to the security section of alerting (elastic#91501)
  [Uptime] Ping list step screenshot caption formatting (elastic#91403)
  [Vislib] Use timestamp on brush event instead of iso dates (elastic#91483)
  [Application Usage] Remove deprecated & unused legacy.appChanged API (elastic#91464)
  Migrate logstash, monitoring, url_drilldowns, xpack_legacy to ts projects (elastic#91194)
  [APM] Wrap Elasticsearch client errors (elastic#91125)
  [APM] Fix optimize-tsconfig script (elastic#91487)
  [Discover][docs] Add searchFieldsFromSource description (elastic#90980)
  Adds support for 'ip' data type (elastic#85087)
  [Detection Rules] Add updates from 7.11.2 rules (elastic#91553)
  [SECURITY SOLUTION] Eql in timeline (elastic#90816)
  [APM] Correlations Beta (elastic#86477) (elastic#89952)
  [Security Solutions][Detection Engine] Adds a warning banner when the alerts data has not been migrated yet. (elastic#90258)
  [Security Solution] [Timeline] Endpoint row renderers (2nd batch) (elastic#91446)
  skip flaky suite (elastic#91450)
  skip flaky suite (elastic#91592)
  [Security Solution][Endpoint][Admin] Endpoint Details UX Enhancements (elastic#90870)
  [ML] Add better UI support for runtime fields Transforms  (elastic#90363)
  [Security Solution] [Detections] Replace 'partial failure' with 'warning' for rule statuses (elastic#91167)
  [Security Solution][Detections] Adds Indicator path config for indicator match rules (elastic#91260)
  ...
cqliu1 added a commit that referenced this pull request Feb 17, 2021
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
@timroes timroes added release_note:feature Makes this part of the condensed release notes and removed release_note:enhancement labels Mar 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New value added to drive a business result Feature:Canvas impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. loe:small Small Level of Effort release_note:feature Makes this part of the condensed release notes review Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas v7.12.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Enhancement Request] NTT COMMUNICATIONS: Canvas support for Type IP
6 participants