zeppelin icon indicating copy to clipboard operation
zeppelin copied to clipboard

[ZEPPELIN-5449] Zeppelin currently cannot do the CI under k8s environment

Open rickchengx opened this issue 3 years ago • 5 comments

What is this PR for?

Zeppelin uses GitHub Action to complete Continuous integration (CI). Specifically, it has three workflows: core.yaml, frontend.yaml and rat.yaml. However, It is currently not possible to verify the functions of zeppelin in the k8s environment in these workflows.

This PR utilizes Minikube as the k8s environment for CI testing. Specifically, it defines a new GitHub Action workflow: k8s_ci_tests.yaml. It contains following key steps:

  1. start Minikube (i.e., a single-node Kubernetes cluster) on the GitHub-hosted VM.
  2. build Zeppelin and output the distribution tar.gz
  3. download Spark and build image
  4. build zeppelin image from the distribution tar.gz (Dockerfile is in scripts/docker/zeppelin/build-from-distribution/Dockerfile)
  5. mvn testing
  • mvn test -DskipRat -Dtest=K8sMinikubeTestBasic,K8sMinikubeTestPySpark,K8sMinikubeTestSparkR -pl :launcher-k8s-standard -B It starts and tests multiple interpreters (shell, python, spark, pyspark, sparkr, sparksql) in the Minikube cluster.

What type of PR is it?

[Improvement]

Todos

  • [ ] - Task

What is the Jira issue?

How should this be tested?

  • CI passed and manually tested

Screenshots (if appropriate)

Questions:

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

rickchengx avatar Jul 28 '21 09:07 rickchengx

Hi, @zjffdu Thanks for the comments. I've made the following changes:

  1. Move the test classes into module zeppelin-interpreter-integration
  2. Reuse the org.apache.zeppelin.integration.MiniZeppelin but add a parameter to set the ZeppelinConfiguration.
  3. Reuse the properties file about log4j.

rickchengx avatar Jul 30 '21 06:07 rickchengx

ping

jongyoul avatar Sep 25 '21 02:09 jongyoul

@rickchengx Could you rebase this PR ?

zjffdu avatar Oct 02 '21 05:10 zjffdu

@zjffdu Sorry for the late response, I was busy with my school things. I have rebased my PR.

rickchengx avatar Nov 15 '21 09:11 rickchengx

Thanks @rickchengx

zjffdu avatar Nov 16 '21 03:11 zjffdu