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.

Sign up for GitHub

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

Jump to bottom

[APM] Show Correlations button and flyout for Basic license users with license prompt #90766

Closed
formgeist opened this issue Feb 9, 2021 · 1 comment · Fixed by #89952
Closed

[APM] Show Correlations button and flyout for Basic license users with license prompt #90766

formgeist opened this issue Feb 9, 2021 · 1 comment · Fixed by #89952
Assignees
@ogupte
Labels
apm:correlations Team:APM All issues that need APM UI Team support v7.12.0

Comments

@formgeist
Copy link
Contributor

formgeist commented Feb 9, 2021 •
edited
Loading

Summary

We initially implemented the Correlations to be hidden while experimental, but it makes sense to enable users on all license tiers to discover the Correlations feature and enable trial or upgrade via the standard license prompt for most Gold+ licenses.

Solution

  • Display the Correlations button and enable the flyout for all licenses where the option would normally appear.
  • Display license prompt enabling users to trial or upgrade their license via License management in Kibana.
  • Update the license prompt copy to read about the Correlations feature (@nehaduggal thoughts on the introduction?)

Correlations - License prompt

Example of license prompt
@formgeist formgeist added Team:APM All issues that need APM UI Team support v7.12.0 labels Feb 9, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/apm-ui (Team:apm)

@ogupte ogupte self-assigned this Feb 10, 2021
ogupte added a commit to ogupte/kibana that referenced this issue Feb 16, 2021
@ogupte
[APM] Adds license check for correlations (elastic#90766)
b5f69ab
@ogupte ogupte mentioned this issue Feb 16, 2021
Merged
ogupte added a commit that referenced this issue Feb 17, 2021
@ogupte @kibanamachine
[APM] Correlations Beta (#86477) (#89952)
d7d2b15 
* [APM] Correlations GA (#86477)

* polish and improvements to correlations UI

* more improvements and polish

* added impact bar

* added descriptions

* make custom field persistence be unique per service

* make custom threshold unique per service in latency correlations

* adds telemetry for apm correlations feature. Events:
 - 'show_correlations_flyout'
 - 'customize_correlations_fields'
 - 'select_significant_term'

* adds more telemetry for correlations (#90622)

* removes the raw score column

* replaces experiemental callout with beta badge

* replaces threshold number input with percentile option selector

* improvements to latency correlations scoring and percentage reporting

* removes the 'apm:enableCorrelations' UI setting

* - rename useFieldNames.ts -> use_field_names.ts
- filter out fields that are not type 'keyword'
- feedback improvements

* Fixes casing issue for the 'correlations' dir

* [APM] Moves correlations button to service details tabslist row (#91080)

* [APM] Adds license check for correlations (#90766)

* [APM] Adds metrics tracking for correlations views and license prompts (#90622)

* Updated the API integration tests to check for new default fields and 15 buckets

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
ogupte added a commit to ogupte/kibana that referenced this issue Feb 17, 2021
@ogupte @kibanamachine
[APM] Correlations Beta (elastic#86477) (elastic#89952)
01bc820 
* [APM] Correlations GA (elastic#86477)

* polish and improvements to correlations UI

* more improvements and polish

* added impact bar

* added descriptions

* make custom field persistence be unique per service

* make custom threshold unique per service in latency correlations

* adds telemetry for apm correlations feature. Events:
 - 'show_correlations_flyout'
 - 'customize_correlations_fields'
 - 'select_significant_term'

* adds more telemetry for correlations (elastic#90622)

* removes the raw score column

* replaces experiemental callout with beta badge

* replaces threshold number input with percentile option selector

* improvements to latency correlations scoring and percentage reporting

* removes the 'apm:enableCorrelations' UI setting

* - rename useFieldNames.ts -> use_field_names.ts
- filter out fields that are not type 'keyword'
- feedback improvements

* Fixes casing issue for the 'correlations' dir

* [APM] Moves correlations button to service details tabslist row (elastic#91080)

* [APM] Adds license check for correlations (elastic#90766)

* [APM] Adds metrics tracking for correlations views and license prompts (elastic#90622)

* Updated the API integration tests to check for new default fields and 15 buckets

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
ogupte added a commit that referenced this issue Feb 17, 2021
@ogupte @kibanamachine
[APM] Correlations Beta (#86477) (#89952) (#91603)
afaf615 
* [APM] Correlations GA (#86477)

* polish and improvements to correlations UI

* more improvements and polish

* added impact bar

* added descriptions

* make custom field persistence be unique per service

* make custom threshold unique per service in latency correlations

* adds telemetry for apm correlations feature. Events:
 - 'show_correlations_flyout'
 - 'customize_correlations_fields'
 - 'select_significant_term'

* adds more telemetry for correlations (#90622)

* removes the raw score column

* replaces experiemental callout with beta badge

* replaces threshold number input with percentile option selector

* improvements to latency correlations scoring and percentage reporting

* removes the 'apm:enableCorrelations' UI setting

* - rename useFieldNames.ts -> use_field_names.ts
- filter out fields that are not type 'keyword'
- feedback improvements

* Fixes casing issue for the 'correlations' dir

* [APM] Moves correlations button to service details tabslist row (#91080)

* [APM] Adds license check for correlations (#90766)

* [APM] Adds metrics tracking for correlations views and license prompts (#90622)

* Updated the API integration tests to check for new default fields and 15 buckets

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ogupte ogupte

Labels
apm:correlations Team:APM All issues that need APM UI Team support v7.12.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[APM] Correlations Beta (#86477) ogupte/kibana
4 participants
@graphaelli @ogupte @formgeist @elasticmachine