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

ci(jenkins): simplify the kibana setup for the e2e tests #52729

Merged

Conversation

v1v
Copy link
Member

@v1v v1v commented Dec 11, 2019

Summary

Let's simplify the configuration for the kibana instance when running the e2e tests for the kibana-ui.

Checklist

Use strikethroughs to remove checklist items you don't feel are applicable to this PR.

- [ ] This was checked for cross-browser compatibility, including a check against IE11
- [ ] Any text added follows EUI's writing guidelines, uses sentence case text and includes [i18n support](https://github.com/elastic/kibana/blob/master/packages/kbn-i18n/README.md)
- [ ] Documentation was added for features that require explanation or tutorials
- [ ] Unit or functional tests were updated or added to match the most common scenarios
- [ ] This was checked for keyboard-only and screenreader accessibility

For maintainers

@v1v v1v self-assigned this Dec 11, 2019
@v1v v1v added the automation label Dec 11, 2019
@v1v
Copy link
Member Author

v1v commented Dec 11, 2019

  • Let's discuss how we can make this faster by disabling everything but the apm plugin.

@v1v v1v requested a review from sorenlouv December 11, 2019 12:55
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

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

@sorenlouv sorenlouv marked this pull request as ready for review December 11, 2019 15:56
@sorenlouv sorenlouv requested a review from a team as a code owner December 11, 2019 15:56
@sorenlouv sorenlouv merged commit 1644762 into elastic:master Dec 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants