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] Run cypress record on merge #161068

Merged
merged 3 commits into from
Jul 10, 2023
Merged

Conversation

kpatticha
Copy link
Contributor

@kpatticha kpatticha commented Jul 3, 2023

Summary

closes #154551

Run cypress record on merge when Team:APM label in applied

I need to run add_gh_labels_to_bk_metadata script because labels are not available on merge pipeline.

Tested here: https://buildkite.com/elastic/kibana-pull-request/builds/141233#01893fd7-e2f9-432b-86a3-f80515f343c4

@apmmachine
Copy link
Contributor

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
  • run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

@kpatticha kpatticha added the apm:cypress-record cypress test runs will be recorded to the cypress dashboard label Jul 3, 2023
@kpatticha
Copy link
Contributor Author

@elasticmachine merge upstream

@kpatticha kpatticha changed the title Run add_gh_labels_to_bk_metadata [APM] Run cypress on merge Jul 10, 2023
@kpatticha kpatticha changed the title [APM] Run cypress on merge [APM] Run cypress record on merge Jul 10, 2023
@kpatticha kpatticha marked this pull request as ready for review July 10, 2023 13:33
@kpatticha kpatticha requested a review from a team as a code owner July 10, 2023 13:33
@elasticmachine
Copy link
Contributor

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

@kpatticha kpatticha added the release_note:skip Skip the PR/issue when compiling release notes label Jul 10, 2023
@kibana-ci
Copy link
Collaborator

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #10 / InfraOps App Metrics UI Hosts View #Single host Flyout "after each" hook for "should navigate to APM services after click"
  • [job] [logs] FTR Configs #10 / InfraOps App Metrics UI Hosts View #Single host Flyout should navigate to APM services after click

Metrics [docs]

Unknown metric groups

ESLint disabled line counts

id before after diff
enterpriseSearch 14 16 +2
securitySolution 408 412 +4
total +6

Total ESLint disabled count

id before after diff
enterpriseSearch 15 17 +2
securitySolution 487 491 +4
total +6

History

  • 💔 Build #141212 failed 6f5d128a3f10c0997473dc5d23bf85a7082b7e64
  • 💔 Build #141201 failed bac91f861098fce6954d3bed5a25018b1ab6fe36
  • 💔 Build #141196 failed ec9d86ecfecdcb43dbbdeb81d192512d8b85883b
  • 💚 Build #141166 succeeded 79769104297cb86b21d42438d28919a2d5b4c79d
  • 💔 Build #141160 failed 7c45bed53434aed14a26f9e50151b2b43c612f83

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

@kpatticha kpatticha removed the apm:cypress-record cypress test runs will be recorded to the cypress dashboard label Jul 10, 2023
@kpatticha kpatticha merged commit bfaf5e0 into elastic:main Jul 10, 2023
@kibanamachine kibanamachine added the backport:skip This commit does not require backporting label Jul 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:APM All issues that need APM UI Team support v8.10.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[APM] Record APM Cypress tests runs on merge
6 participants