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 URL paths #40370

Closed
jasonrhodes opened this issue Nov 21, 2018 · 5 comments
Closed

APM URL paths #40370

jasonrhodes opened this issue Nov 21, 2018 · 5 comments
Labels
enhancement New value added to drive a business result Team:APM All issues that need APM UI Team support

Comments

@jasonrhodes
Copy link
Member

We currently assume the top-level urls in APM, if we don't recognize them, are service names.

e.g. {kibana root}/app/apm#/something --> we assume "something" is a service and redirect this URL to #/something/transactions.

Yesterday, after conversations with @sqren and @ogupte about the work done in #25906, we're proposing that for 7.0, we switch to /services/:serviceName and avoid any top-level wildcard matches.

For instance, right now if your service is named "services", the service redirect won't work because "/services" will load the list of services and won't redirect you to "/services/transactions" like it would for any other service name (except "traces" and "invalid-license"). As we add more top-level sections (service-maps perhaps? maybe even traces as their own thing, etc.), it'll probably be good to have reserved namespaces at the top level, so we think it would make sense to make this change for all service URLs in 7.0.

  • /:serviceName/(transactions|errors)/* becomes /services/:serviceName/(transactions|errors)/*
@sorenlouv
Copy link
Member

Since this is purely UI, shouldn't it be in Kibana?

@alvarolobato alvarolobato transferred this issue from elastic/apm Jul 4, 2019
@alvarolobato alvarolobato added the Team:APM All issues that need APM UI Team support label Jul 4, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/apm-ui

@alvarolobato
Copy link

Moved from APM repo.

@sorenlouv sorenlouv added [zube]: Inbox enhancement New value added to drive a business result and removed [zube]: Inbox labels Jul 10, 2019
@alvarolobato
Copy link

@roncohen suggested that a possible solution to this would be removing the service from the URL altogether as we did with transactions.

@sorenlouv
Copy link
Member

Closed in #41233

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New value added to drive a business result Team:APM All issues that need APM UI Team support
Projects
None yet
Development

No branches or pull requests

4 participants