-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
Fix APM E2E #113297
Fix APM E2E #113297
Conversation
Add `?*` to the end of request aliases in instances table tests. Not sure why or when these started failing. It could be that the version of Cypress we're using changed and this behavior changed.
Pinging @elastic/apm-ui (Team:apm) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't understand why this fixes it (and why it was not needed previously) but lgtm
💛 Build succeeded, but was flaky
Metrics [docs]
History
To update your PR or re-run it, just comment with: |
Add `?*` to the end of request aliases in instances table tests. elastic#104301 upgraded Cypress to a newer version that required this, but the APM E2E tests were not run because there were no changes to any files in APM.
💚 Backport successful
This backport PR will be merged automatically after passing CI. |
Add `?*` to the end of request aliases in instances table tests. #104301 upgraded Cypress to a newer version that required this, but the APM E2E tests were not run because there were no changes to any files in APM. Co-authored-by: Nathan L Smith <[email protected]>
Add
?*
to the end of request aliases in instances table tests.#104301 upgraded Cypress to a newer version that required this, but the APM E2E tests were not run because there were no changes to any files in APM.