Skip to content

[7.x] Add service map icon for rum-js agent type (#62721)#62796

Merged
smith merged 1 commit intoelastic:7.xfrom
smith:backport/7.x/pr-62721
Apr 7, 2020
Merged

[7.x] Add service map icon for rum-js agent type (#62721)#62796
smith merged 1 commit intoelastic:7.xfrom
smith:backport/7.x/pr-62721

Conversation

@smith
Copy link
Contributor

@smith smith commented Apr 7, 2020

Backports the following commits to 7.x:

* Add service map icon for rum-js agent type

We previously were only checking for "js-base" as the RUM `agent.name` but it can report either "js-base" or "rum-js", so make that work.

I saw an "opbeans-react" service on a map generated by apm-integration-testing that Gil had running last week. Not sure if that's a newer or older version because I don't see it on apm.elstc.co.

Also clean up the icons storybook a little.

Fixes elastic#62336.

* use isRumAgentName

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
@smith smith added the backport This PR is a backport of another PR label Apr 7, 2020
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

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

@smith smith merged commit 118c743 into elastic:7.x Apr 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants