Skip to content

[APM] unskipping e2e tests #177949

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
Merged
merged 7 commits into from
Mar 6, 2024
Merged

[APM] unskipping e2e tests #177949

merged 7 commits into from
Mar 6, 2024

Conversation

@cauemarcondes
Copy link
Contributor

@cauemarcondes cauemarcondes commented Mar 4, 2024 •
edited
Loading

@ghost
Copy link

ghost commented Mar 4, 2024

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

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

@cauemarcondes cauemarcondes changed the title Test [APM] unskipping e2e tests Mar 4, 2024
@cauemarcondes cauemarcondes added release_note:skip Skip the PR/issue when compiling release notes v8.14.0 labels Mar 4, 2024
@cauemarcondes cauemarcondes marked this pull request as ready for review March 4, 2024 16:25
@cauemarcondes cauemarcondes requested a review from a team as a code owner March 4, 2024 16:25
@cauemarcondes cauemarcondes requested a review from a team as a code owner March 5, 2024 14:11
@botelastic botelastic bot added the Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team label Mar 5, 2024
jennypavlova
jennypavlova approved these changes Mar 5, 2024
View reviewed changes
Copy link
Member

@jennypavlova jennypavlova left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks for the fixes!

MiriamAparicio
MiriamAparicio reviewed Mar 5, 2024
View reviewed changes
@MiriamAparicio
Copy link
Contributor

MiriamAparicio commented Mar 5, 2024

Might we also want to check and change the path for infra script, what do you think?

@kibana-ci
Copy link

kibana-ci commented Mar 6, 2024

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

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

delanni
delanni reviewed Mar 6, 2024
View reviewed changes
@delanni delanni mentioned this pull request Mar 6, 2024
Merged
2 tasks
@cauemarcondes cauemarcondes merged commit f327c2d into elastic:main Mar 6, 2024
@kibanamachine kibanamachine added the backport:skip This PR does not require backporting label Mar 6, 2024
@cauemarcondes cauemarcondes deleted the apm-e2e-tests branch March 6, 2024 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@delanni delanni delanni approved these changes

@jennypavlova jennypavlova jennypavlova approved these changes

@MiriamAparicio MiriamAparicio MiriamAparicio approved these changes

Assignees

No one assigned

Labels

backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team v8.14.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@cauemarcondes @MiriamAparicio @kibana-ci @delanni @jennypavlova @kibanamachine