Skip to content

[APM] Fix duplicate index patterns#64883

Merged
sorenlouv merged 2 commits intoelastic:masterfrom
sorenlouv:fix-duplicate-index-patterns
May 4, 2020
Merged

[APM] Fix duplicate index patterns#64883
sorenlouv merged 2 commits intoelastic:masterfrom
sorenlouv:fix-duplicate-index-patterns

Conversation

@sorenlouv
Copy link
Copy Markdown
Contributor

Closes #61378
Closes #64765

An index pattern with the id “apm_static_index_pattern_id” is automatically created when opening the APM app. It is also possible to manually create the index pattern by clicking “Load Kibana Objects” in the Getting Started Guide, however this will create an index pattern with a different id (“apm-*).

This fixes it by making sure the index pattern id is always “apm_static_index_pattern_id”.

image

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

why haven't you imported the one in x-pack/plugins/apm/common/index_pattern_constants.ts?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good point 👍
(I was still in the old mindset that this code lived outside the APM app)

sorenlouv added 2 commits May 4, 2020 13:51
An index pattern with the id “apm_static_index_pattern_id” is automatically created when opening the APM app. It is also possible to manually create the index pattern by clicking “Load Kibana Objects” in the Getting Started Guide, however this will create an index pattern with a different id (“apm-*).

This fixes it by making sure the index pattern id is always “apm_static_index_pattern_id”.
@sorenlouv sorenlouv force-pushed the fix-duplicate-index-patterns branch from 289d1b6 to 7002fa8 Compare May 4, 2020 11:53
Copy link
Copy Markdown
Contributor

@cauemarcondes cauemarcondes left a comment

Choose a reason for hiding this comment

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

LGTM

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

History

  • 💚 Build #44392 succeeded 289d1b651514b9ecd198f0570b843c45b2576c1e

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

@sorenlouv sorenlouv merged commit a532a91 into elastic:master May 4, 2020
@sorenlouv sorenlouv deleted the fix-duplicate-index-patterns branch May 4, 2020 14:15
sorenlouv added a commit to sorenlouv/kibana that referenced this pull request May 4, 2020
gmmorris added a commit to gmmorris/kibana that referenced this pull request May 4, 2020
…ana into alerting/np-tests-migration

* 'alerting/np-tests-migration' of github.com:gmmorris/kibana:
  [APM] Agent remote config: validation for Java agent configs (elastic#63956)
  [APM] Fix duplicate index patterns (elastic#64883)
  Drilldowns (elastic#61219)
  [Alerting] fix labels and links in PagerDuty action ui and docs (elastic#64032)
  [Event Log] Ensure sorting tests are less flaky (elastic#64781)
  update endpoint to restrict removing with datasources (elastic#64978)
  [Logs UI] [Alerting] Alerts management page enhancements (elastic#64654)
  Adjust kibana app owning files (elastic#65064)
  Migrate tutorial resources (elastic#64298)
  [Logs UI] Tweak copy in log alerts dialog (elastic#64645)
  [Logs UI] [Alerting] Documentation (elastic#64886)
  [Logs UI] Add dataset filter to ML module setup screen (elastic#64470)
  [TSVB] Fixing memory leak (elastic#64918)
  Bump backport to 5.4.1 (elastic#65041)
@cauemarcondes cauemarcondes self-assigned this May 14, 2020
@cauemarcondes
Copy link
Copy Markdown
Contributor

Tested:

  • Chrome ✅
  • FF ✅
  • Safari ✅

@cauemarcondes cauemarcondes added the apm:test-plan-done Pull request that was successfully tested during the test plan label May 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

apm:test-plan-done Pull request that was successfully tested during the test plan release_note:fix v7.8.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[APM] Remove "Load Kibana objects" from APM getting started instructions APM Load Kibana Objects creates two indices

3 participants