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] Service map health indicators missing for services with camelcase names #68978

Closed
ogupte opened this issue Jun 12, 2020 · 1 comment · Fixed by #68979
Closed

[APM] Service map health indicators missing for services with camelcase names #68978

ogupte opened this issue Jun 12, 2020 · 1 comment · Fixed by #68979
Assignees
@ogupte
Labels
apm:service-maps Service Map feature in APM bug Fixes for quality problems that affect the customer experience Team:APM All issues that need APM UI Team support v7.8.0 v7.9.0

Comments

@ogupte
Copy link
Contributor

ogupte commented Jun 12, 2020

Service maps are not displaying health indicators for services which have ml jobs and anomalies. This is due too the group names being in the ML job being specifically encoded and lowercased during job creation and then failing to match the actual service name.
@ogupte ogupte added [zube]: In Progress bug Fixes for quality problems that affect the customer experience apm:service-maps Service Map feature in APM Team:APM All issues that need APM UI Team support v7.8.0 v7.9.0 labels Jun 12, 2020
@ogupte ogupte self-assigned this Jun 12, 2020
@elasticmachine
Copy link
Contributor

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

ogupte added a commit to ogupte/kibana that referenced this issue Jun 12, 2020
@ogupte
Closes elastic#68978 by encoding service names before comparing them …
f789386 
…to ML job group names.
@ogupte ogupte mentioned this issue Jun 12, 2020
Merged
ogupte added a commit that referenced this issue Jun 12, 2020
@ogupte @elasticmachine
[APM] Service map fix missing ML health indicators for camelCased ser…
fd740a6 
…vices (#68979)

* Closes #68978 by encoding service names before comparing them to ML job group names.

* Code tweak for readbility

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
ogupte added a commit to ogupte/kibana that referenced this issue Jun 12, 2020
@ogupte @elasticmachine
[APM] Service map fix missing ML health indicators for camelCased ser…
e2f0e87 
…vices (elastic#68979)

* Closes elastic#68978 by encoding service names before comparing them to ML job group names.

* Code tweak for readbility

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
ogupte added a commit to ogupte/kibana that referenced this issue Jun 12, 2020
@ogupte @elasticmachine
[APM] Service map fix missing ML health indicators for camelCased ser…
c672973 
…vices (elastic#68979)

* Closes elastic#68978 by encoding service names before comparing them to ML job group names.

* Code tweak for readbility

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
ogupte added a commit that referenced this issue Jun 12, 2020
@ogupte @elasticmachine
[APM] Service map fix missing ML health indicators for camelCased ser…
008eae4 
…vices (#68979) (#69023)

* Closes #68978 by encoding service names before comparing them to ML job group names.

* Code tweak for readbility

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
ogupte added a commit that referenced this issue Jun 12, 2020
@ogupte @elasticmachine
[APM] Service map fix missing ML health indicators for camelCased ser…
52f9583 
…vices (#68979) (#69022)

* Closes #68978 by encoding service names before comparing them to ML job group names.

* Code tweak for readbility

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>

Co-authored-by: Elastic Machine <elasticmachine@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:service-maps Service Map feature in APM bug Fixes for quality problems that affect the customer experience Team:APM All issues that need APM UI Team support v7.8.0 v7.9.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[APM] Service map fix missing ML health indicators for camelCased services ogupte/kibana
3 participants
@sorenlouv @ogupte @elasticmachine