You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Fix Docker install and require the 'stdio' CLI parameter (#97)
* Require the stdio CLI parameter when running as a container. This is to future-proof the addition of the http protocol that will require the http parameter.
* Refactor Docker file do make it build cache friendly
* Fix OTel configuration so that it doesn't pollute stdout that's used to communicate with the client
* Build a multi-arch (amd64 and arm64) using npm run build-docker-image
* Update README and CONTRIBUTING to include instructions for the Docker image
Copy file name to clipboardExpand all lines: docs/CONTRIBUTING.md
+8Lines changed: 8 additions & 0 deletions
Display the source diff
Display the rich diff
Original file line number
Diff line number
Diff line change
@@ -28,6 +28,14 @@ Please note that this project follows the [Elastic's Open Source Community Code
28
28
npm run build
29
29
```
30
30
31
+
To build the Docker image, run:
32
+
33
+
```sh
34
+
npm run build-docker-image
35
+
```
36
+
37
+
This builds a multi-architecture image for amd64 and arm64. If you don't have a configuration that allows multi-architecture builds, simply run `docker build -t mcp/elasticsearch` .
0 commit comments