Skip to content

tests/system: update user-agent pipline test#4018

Merged
axw merged 1 commit intoelastic:masterfrom
axw:test-pipeline-useragent-mac
Jul 23, 2020
Merged

tests/system: update user-agent pipline test#4018
axw merged 1 commit intoelastic:masterfrom
axw:test-pipeline-useragent-mac

Conversation

@axw
Copy link
Copy Markdown
Member

@axw axw commented Jul 23, 2020

Motivation/summary

Since the uap-core regexes were updated in
elastic/elasticsearch#59697,
MacOS user-agents are now parsed differently.
Update our test to match the change.

Checklist

I have considered changes for:
- [ ] documentation
- [ ] logging (add log lines, choose appropriate log selector, etc.)
- [ ] metrics and monitoring (create issue for Kibana team to add metrics to visualizations, e.g. Kibana#44001)

How to test these changes

make docker-system-tests

Related issues

elastic/elasticsearch#59697

Since the uap-core regexes were updated in
elastic/elasticsearch#59697,
MacOS user-agents are now parsed differently.
Update our test to match the change.
@axw axw merged commit c1dff66 into elastic:master Jul 23, 2020
@axw axw deleted the test-pipeline-useragent-mac branch July 23, 2020 08:37
axw added a commit to axw/apm-server that referenced this pull request Jul 23, 2020
Since the uap-core regexes were updated in
elastic/elasticsearch#59697,
MacOS user-agents are now parsed differently.
Update our test to match the change.
axw added a commit to axw/apm-server that referenced this pull request Jul 23, 2020
Since the uap-core regexes were updated in
elastic/elasticsearch#59697,
MacOS user-agents are now parsed differently.
Update our test to match the change.
axw added a commit that referenced this pull request Jul 23, 2020
Since the uap-core regexes were updated in
elastic/elasticsearch#59697,
MacOS user-agents are now parsed differently.
Update our test to match the change.
axw added a commit that referenced this pull request Jul 23, 2020
Since the uap-core regexes were updated in
elastic/elasticsearch#59697,
MacOS user-agents are now parsed differently.
Update our test to match the change.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant