Skip to content

Fix vega config issue#59737

Merged
flash1293 merged 1 commit intoelastic:masterfrom
flash1293:fix-vega-config
Mar 12, 2020
Merged

Fix vega config issue#59737
flash1293 merged 1 commit intoelastic:masterfrom
flash1293:fix-vega-config

Conversation

@flash1293
Copy link
Contributor

Fixes #59622

The vega root key got renamed from vega to vis_type_vega. However as the legacy plugin didn't pick up the new root key settings, it was possible to get into a situation with an enabled legacy vega plugin and a disabled NP vega plugin.

This PR makes sure both react in the same way to config keys provided in kibana.yml.

@flash1293 flash1293 added the Feature:Vega Vega visualizations label Mar 10, 2020
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

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

@flash1293 flash1293 marked this pull request as ready for review March 10, 2020 14:22
@flash1293 flash1293 added the Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// label Mar 10, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app (Team:KibanaApp)

@flash1293 flash1293 added release_note:skip Skip the PR/issue when compiling release notes v7.7.0 v8.0.0 labels Mar 10, 2020
@flash1293 flash1293 merged commit 0ecf8d1 into elastic:master Mar 12, 2020
flash1293 added a commit to flash1293/kibana that referenced this pull request Mar 12, 2020
flash1293 added a commit that referenced this pull request Mar 12, 2020
gmmorris added a commit to gmmorris/kibana that referenced this pull request Mar 12, 2020
* master: (45 commits)
  skip flaky suite (elastic#59717)
  UI Metrics use findAll to retrieve all Saved Objects (elastic#59891)
  [Discover] Migrate Context mocha tests to use Jest (elastic#59658)
  [Maps] Move redux reducers and store logic to NP (elastic#58294)
  rebalance x-pack groups (elastic#58930)
  [Discover] Reimplement $route.reload when index pattern changes (elastic#59877)
  [Upgrade Assistant Meta] Breaking changes issue template (elastic#59745)
  Skip CI based on changes in PR (elastic#59939)
  [ML] Transforms: Replace KqlFilterBar with QueryStringInput. (elastic#59723)
  [ML] Functional tests - stabilize date_nanos test (elastic#59986)
  [ML] Typescripting client side endpoint functions (elastic#59928)
  a11y tests on adding columns to discover table (elastic#59375)
  fix graph plugin config path (elastic#59540)
  fix vega config issues (elastic#59737)
  [Upgrade Assistant] Open And Close Slight Refactor (elastic#59890)
  [ML] Adding shared services to ml setup contract (elastic#59730)
  [Visualize] Fix linked search behavior (elastic#59690)
  [ML] Register NP ML plugin for Kibana management section. (elastic#59762)
  [Lens] Adds using queries/filters for field existence endpoint (elastic#59033)
  Delete FilterStateManager and QueryFilter :-D (elastic#59872)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature:Vega Vega visualizations release_note:skip Skip the PR/issue when compiling release notes Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// v7.7.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Disabling vis_type_vega causes an unhandled exception during runtime

4 participants