Skip to content

[K8s] Fix K8s integration test#29480

Closed
agrawaldevesh wants to merge 1 commit into
apache:masterfrom
agrawaldevesh:k8s_fix
Closed

[K8s] Fix K8s integration test#29480
agrawaldevesh wants to merge 1 commit into
apache:masterfrom
agrawaldevesh:k8s_fix

Conversation

@agrawaldevesh

Copy link
Copy Markdown

What changes were proposed in this pull request?

Apparently #29466 broke the K8s integration test:

[ERROR] [Error] /home/jenkins/workspace/SparkPullRequestBuilder-K8s/resource-managers/kubernetes/integration-tests/src/test/scala/org/apache/spark/deploy/k8s/integrationtest/DecommissionSuite.scala:29: value WORKER_DECOMMISSION_ENABLED is not a member of object org.apache.spark.internal.config.Worker

This PR fixes that.

Why are the changes needed?

Enables running K8s integration tests again.

Does this PR introduce any user-facing change?

No

How was this patch tested?

K8s testing from Github

@agrawaldevesh

Copy link
Copy Markdown
Author

cc: @holdenk and @Ngone51 for review please.

This is to unblock #29452

@SparkQA

SparkQA commented Aug 19, 2020

Copy link
Copy Markdown

Test build #127665 has finished for PR 29480 at commit 50afa71.

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

@holdenk

holdenk commented Aug 19, 2020

Copy link
Copy Markdown
Contributor

There’s actually two failures in that test and I’ve made a PR for that.

@holdenk

holdenk commented Aug 19, 2020

Copy link
Copy Markdown
Contributor

See #29479

@agrawaldevesh

Copy link
Copy Markdown
Author

Abandoning in favor of #29479

@SparkQA

SparkQA commented Aug 20, 2020

Copy link
Copy Markdown

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

@SparkQA

SparkQA commented Aug 20, 2020

Copy link
Copy Markdown

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants