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

[Maps] Maps SO for 'ecommerce' sample dataset is registered twice #116263

Closed
mshustov opened this issue Oct 26, 2021 · 3 comments · Fixed by #116295
Closed

[Maps] Maps SO for 'ecommerce' sample dataset is registered twice #116263

mshustov opened this issue Oct 26, 2021 · 3 comments · Fixed by #116295
Assignees
Labels
bug Fixes for quality problems that affect the customer experience [Deprecated-Use Team:Presentation]Team:Geo Former Team Label for Geo Team. Now use Team:Presentation

Comments

@mshustov
Copy link
Contributor

Kibana version: master
Describe the bug: ecommerce dataset cannot be removed due 409 Conflict error (see steps to reproduce section) due to duplicated map:2c9c1f60-1909-11e9-919b-ffe5949a18d2 SavedObject. Which is registered by home plugin as well.


The registration should be refactored. I didn't remove the SO registration from home plugin because Maps SO is referenced there

Steps to reproduce:

  1. uncomment 61b7f90
  2. navigate to Kibana
  3. install ecommerce sample dataset, remove it, install again, remove it again.

Expected behavior:
Dataset is removed on the second and later attempts.

@mshustov mshustov added bug Fixes for quality problems that affect the customer experience [Deprecated-Use Team:Presentation]Team:Geo Former Team Label for Geo Team. Now use Team:Presentation labels Oct 26, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-gis (Team:Geo)

@nreese
Copy link
Contributor

nreese commented Oct 26, 2021

Maps saved object moved to home plugin by #107746.

I can open a PR to move it back to Maps plugin

@mshustov
Copy link
Contributor Author

@nreese works for me as long as it doesn't break the sample data set. I was going to remove it from home but it's been referenced by a dashboard in the home plugin.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience [Deprecated-Use Team:Presentation]Team:Geo Former Team Label for Geo Team. Now use Team:Presentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants