-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Logs onboarding] Added Cypress to observability onboarding plugin #157147
Conversation
🤖 GitHub commentsExpand to view the GitHub comments
Just comment with:
|
💚 Build Succeeded
Metrics [docs]Unknown metric groupsESLint disabled in files
ESLint disabled line counts
Total ESLint disabled count
To update your PR or re-run it, just comment with: |
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.
.buildkite/
LGTM
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.
Thanks for adding this
@@ -0,0 +1,3 @@ | |||
xpack.observability_onboarding.ui.enabled: true | |||
|
|||
xpack.cloud.id: 'foo' |
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.
We should probably set the cloud id value to something that matches the expected format
Example: eastus2.azure.elastic-cloud.com:9243$59ef636c6917463db140321484d63cfa$a8b109c08adc43279ef48f29af1a3911
We may need to decode this id to get the ES deployment when we generate the elastic-agent.yml
}); | ||
|
||
process.exitCode = exitStatus; | ||
console.log(`Quitting with exit code ${exitStatus}`); |
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.
This script has caused some confusion in APM so I'm not sure it's the best approach. But if it works for you feel free to carry it over.
This Pr is a follow up of [PR#157008](#157008) and [PR#157147](#157147). Changes: 1. Replaced usage of `euiThemeVars` by `useEuiTheme` and `useEuiFontSize`. 2. e2e cloudId following [cloud plugin format](https://github.com/elastic/kibana/tree/main/x-pack/plugins/cloud#cloudid).
This Pr is a follow up of [PR#157008](#157008) and [PR#157147](#157147). Changes: 1. Replaced usage of `euiThemeVars` by `useEuiTheme` and `useEuiFontSize`. 2. e2e cloudId following [cloud plugin format](https://github.com/elastic/kibana/tree/main/x-pack/plugins/cloud#cloudid).
This PR enables cypress testing in observability onboarding plugin.