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

[SPARK-36859][BUILD][K8S] Upgrade kubernetes-client to 5.8.0 #34109

Closed
wants to merge 1 commit into from
Closed

[SPARK-36859][BUILD][K8S] Upgrade kubernetes-client to 5.8.0 #34109

wants to merge 1 commit into from

Conversation

dongjoon-hyun
Copy link
Member

@dongjoon-hyun dongjoon-hyun commented Sep 26, 2021

What changes were proposed in this pull request?

This PR aims to upgrade kubernetes-client dependency from 5.7.3 to 5.8.0 for Apache Spark 3.3.

Why are the changes needed?

This will add K8s Model v1.22.1 for the developers who are using ExternalClusterManager in K8s environments.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Pass the CIs.

@github-actions github-actions bot added the BUILD label Sep 26, 2021
@viirya
Copy link
Member

viirya commented Sep 26, 2021

looks okay if CI can pass.

@SparkQA
Copy link

SparkQA commented Sep 26, 2021

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/48148/

@dongjoon-hyun
Copy link
Member Author

Thank you for review, @viirya !

On a local Minikube test, it passes all tests except a known issue failure Test decommissioning with dynamic allocation & shuffle cleanups. In Amplab environment, SparkR test is known to fail additionally due to the mismatch of SparkR version because Amplab machine has old R installations.

KubernetesSuite:
- Run SparkPi with no resources
- Run SparkPi with no resources & statefulset allocation
- Run SparkPi with a very long application name.
- Use SparkLauncher.NO_RESOURCE
- Run SparkPi with a master URL without a scheme.
- Run SparkPi with an argument.
- Run SparkPi with custom labels, annotations, and environment variables.
- All pods have the same service account by default
- Run extraJVMOptions check on driver
- Run SparkRemoteFileTest using a remote data file
- Verify logging configuration is picked from the provided SPARK_CONF_DIR/log4j.properties
- Run SparkPi with env and mount secrets.
- Run PySpark on simple pi.py example
- Run PySpark to test a pyfiles example
- Run PySpark with memory customization
- Run in client mode.
- Start pod creation from template
- PVs with local hostpath storage on statefulsets
- PVs with local hostpath and storageClass on statefulsets
- PVs with local storage
- Launcher client dependencies
- SPARK-33615: Launcher client archives
- SPARK-33748: Launcher python client respecting PYSPARK_PYTHON
- SPARK-33748: Launcher python client respecting spark.pyspark.python and spark.pyspark.driver.python
- Launcher python client dependencies using a zip file
- Test basic decommissioning
- Test basic decommissioning with shuffle cleanup
- Test decommissioning with dynamic allocation & shuffle cleanups *** FAILED ***
- Test decommissioning timeouts
- Run SparkR on simple dataframe.R example
Run completed in 18 minutes, 57 seconds.
Total number of tests run: 30
Suites: completed 2, aborted 0
Tests: succeeded 29, failed 1, canceled 0, ignored 0, pending 0
*** 1 TEST FAILED ***

@SparkQA
Copy link

SparkQA commented Sep 26, 2021

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/48148/

@SparkQA
Copy link

SparkQA commented Sep 26, 2021

Test build #143636 has finished for PR 34109 at commit 23b014f.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@dongjoon-hyun
Copy link
Member Author

Jenkins passed. Merged to master for Apache Spark 3.3!

@dongjoon-hyun dongjoon-hyun deleted the SPARK-36859 branch September 26, 2021 23:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants