Skip to content

Add support for Docker and Serverless to kbn/es#161927

Merged
Ikuni17 merged 85 commits intoelastic:mainfrom
Ikuni17:feature/159260/kbn-es-docker
Jul 26, 2023
Merged

Add support for Docker and Serverless to kbn/es#161927
Ikuni17 merged 85 commits intoelastic:mainfrom
Ikuni17:feature/159260/kbn-es-docker

Conversation

@Ikuni17
Copy link
Copy Markdown
Contributor

@Ikuni17 Ikuni17 commented Jul 14, 2023

Closes #159260

Summary

Adds support for running ES through Docker and Serverless in @kbn/es

Checklist

Delete any items that are not applicable to this PR.

@Ikuni17 Ikuni17 marked this pull request as ready for review July 22, 2023 02:45
@Ikuni17 Ikuni17 requested a review from a team as a code owner July 22, 2023 02:45
@Ikuni17 Ikuni17 requested a review from jbudz July 22, 2023 02:45
@Ikuni17 Ikuni17 requested a review from jbudz July 25, 2023 19:13
@Ikuni17 Ikuni17 removed the WIP Work in progress label Jul 25, 2023
@kibana-ci
Copy link
Copy Markdown

kibana-ci commented Jul 25, 2023

💔 Build Failed

Failed CI Steps

Test Failures

  • [job] [logs] Serverless Observability Tests / serverless observability UI navigation active sidenav section is auto opened on load

Metrics [docs]

Unknown metric groups

ESLint disabled line counts

id before after diff
@kbn/es 1 2 +1

Total ESLint disabled count

id before after diff
@kbn/es 1 2 +1

History

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

cc @Ikuni17


['discovery.type', 'single-node'],

['xpack.security.enabled', 'false'],
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Was this needed for the docker command too? I know we want it for serverless, but it seems like we can run the default version normally.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@jbudz I was having SSL issues running Kibana local dev against Docker ES. ES was denying HTTP request and the socket was hanging. Should we follow up with another ticket for proper SSL support?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍 Works for me

@Ikuni17 Ikuni17 merged commit 7d1b1be into elastic:main Jul 26, 2023
@Ikuni17 Ikuni17 deleted the feature/159260/kbn-es-docker branch July 26, 2023 17:34
ThomThomson pushed a commit to ThomThomson/kibana that referenced this pull request Aug 1, 2023
Closes elastic#159260

## Summary

Adds support for running ES through Docker and Serverless in `@kbn/es`


### Checklist

Delete any items that are not applicable to this PR.

- [x]
[Documentation](https://www.elastic.co/guide/en/kibana/master/development-documentation.html)
was added for features that require explanation or tutorials
- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Co-authored-by: Tiago Costa <tiago.costa@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes v8.10.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[scripts/es] Add support for running elasticsearch from image

5 participants