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

APM on Cloud setup instructions contains a bad link #40477

Closed
osmanis opened this issue Jul 8, 2019 · 5 comments · Fixed by #40711
Closed

APM on Cloud setup instructions contains a bad link #40477

osmanis opened this issue Jul 8, 2019 · 5 comments · Fixed by #40711
Assignees
Labels
bug Fixes for quality problems that affect the customer experience Team:APM All issues that need APM UI Team support

Comments

@osmanis
Copy link

osmanis commented Jul 8, 2019

Kibana version: 7.2.0

Describe the bug: The APM setup instructions for Cloud include a link to the ESS console that does not redirect the user to Cloud but instead opens a new Kibana tab.

Steps to reproduce:

  1. Create a deployment in ESS that does not have APM enabled.
  2. Launch Kibana
  3. Open the APM setup instructions page and the broken link will be visible

Expected behavior:

  • The link should take the back to Cloud. Preferably the APM page: https://cloud.elastic.co/region/<region>/deployment/<deploymentId>/apm
  • The sentence should also be updated to: To enable the APM Server go to the Elastic Cloud console and enable APM in the deployment settings. Once enabled, refresh this page. This will help avoid confusion for those people that doesn't know what ESS means.

Screenshots (if relevant):
apm

Related: #32660

@osmanis osmanis added the bug Fixes for quality problems that affect the customer experience label Jul 8, 2019
@osmanis
Copy link
Author

osmanis commented Jul 8, 2019

CC @tbragin

@tbragin
Copy link
Contributor

tbragin commented Jul 8, 2019

@nehaduggal @graphaelli FYI. How do we go about triaging this?

@graphaelli
Copy link
Member

I will take care of it

@graphaelli graphaelli added the Team:APM All issues that need APM UI Team support label Jul 8, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/apm-ui

@graphaelli
Copy link
Member

This happens here:

function getApmServerInstructionSet(config) {
const apmUiUrl = getIfExists(config, 'xpack.cloud.apm.ui.url');
return {
title: i18n.translate('kbn.server.tutorials.apm.apmServer.title', {
defaultMessage: 'APM Server',
}),
instructionVariants: [
{
id: INSTRUCTION_VARIANT.ESC,
instructions: [
{
title: 'Enable the APM Server in the ESS console',
textPre: i18n.translate('kbn.server.tutorials.apm.elasticCloud.textPre', {
defaultMessage:
'To enable the APM Server go to [the ESS console]({essConsoleLink}). Once enabled, refresh this page.',
values: {
essConsoleLink: apmUiUrl,
},
}),
},
],
},
],
};
}
. The information needed to link directly to the deployment is not available yet so I propose we link to https://cloud.elastic.co/deployments?q=$deployment, where the deployment id can be parsed out of the cloud id (if otherwise unavailable)

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 Team:APM All issues that need APM UI Team support
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants