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

[ZEPPELIN-6130] Write a Dockerfile for Elasticsearch interpreter image build #4876

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

ParkGyeongTae
Copy link
Contributor

@ParkGyeongTae ParkGyeongTae commented Oct 17, 2024

What is this PR for?

This PR adds a Dockerfile to build a elasticsearch interpreter.

What type of PR is it?

Feature

Todos

  • - README.md
  • - Dockerfile
  • - Dockerfile_elasticsearch

What is the Jira issue?

How should this be tested?

  • Build and run Elasticsearch with Docker
# Build
docker build -f ./elasticsearch/Dockerfile_elasticsearch -t elasticsearch .
# Run
docker run -d -p 9200:9200 -p 9300:9300 --name elasticsearch elasticsearch
  • Build and run the Elasticsearch interpreter with Docker
# Build
docker build -f ./elasticsearch/Dockerfile -t elasticsearch-interpreter .
# Run
docker run \
--name elasticsearch-interpreter \
-p 8087:8087 \
-e INTERPRETER_EVENT_SERVER_PORT=${__YOUR_INTERPRETER_EVENT_SERVER_PORT__} \
elasticsearch-interpreter
  • Edit interpreter.json
# Old
  "elasticsearch": {
      "...": "...",
      "option": {
        "remote": true,
        "port": -1,
        "perNote": "shared",
        "perUser": "shared",
        "isExistingProcess": false,
        "setPermission": false,
        "owners": [],
        "isUserImpersonate": false
      }
    }
# New
  "elasticsearch": {
      "...": "...",
      "option": {
        "remote": true,
        "host": "host.docker.internal",
        "port": ${__YOUR_INTERPRETER_EVENT_SERVER_PORT__},
        "perNote": "shared",
        "perUser": "shared",
        "isExistingProcess": true,
        "setPermission": false,
        "owners": [],
        "isUserImpersonate": false
      }
    }
  • Verify that the shell interpreter container communicates with the zeppelin server.

Screenshots (if appropriate)

Questions:

  • Does the license files need to update? Yes
  • Is there breaking changes for older versions? No
  • Does this needs documentation? Yes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant