Skip to content

[WIP][SPARK-31902][K8S][TEST] Fix the K8s integration test Jenkins mockito errors #28722

Closed
holdenk wants to merge 1 commit into
apache:masterfrom
holdenk:SPARK-31902-k8s-integration-tests-are-failing-with-mockito-errors
Closed

[WIP][SPARK-31902][K8S][TEST] Fix the K8s integration test Jenkins mockito errors #28722
holdenk wants to merge 1 commit into
apache:masterfrom
holdenk:SPARK-31902-k8s-integration-tests-are-failing-with-mockito-errors

Conversation

@holdenk

@holdenk holdenk commented Jun 3, 2020

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Try adding a specific compatable bytebuddy as an explicit dependency to see if we can fix the mockito errors

Why are the changes needed?

The Jenkin K8s PR builder is consistently failing with Mockito errors running through the core suite.

Does this PR introduce any user-facing change?

No

How was this patch tested?

I will run it through Jenkins and see if it fixes the issues.

@HyukjinKwon

Copy link
Copy Markdown
Member

@holdenk, how does it relate to #28585? It was reverted to recover the Jenkins jobs.

@holdenk

holdenk commented Jun 4, 2020

Copy link
Copy Markdown
Contributor Author

Ah gotcha that makes sense. I was debugging the same issue but I was only seeing it in the K8s PRB. I'll close this if that revert has it covered :)

@holdenk holdenk closed this Jun 4, 2020
@SparkQA

SparkQA commented Jun 4, 2020

Copy link
Copy Markdown

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

@SparkQA

SparkQA commented Jun 4, 2020

Copy link
Copy Markdown

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

@HyukjinKwon

Copy link
Copy Markdown
Member

Ah, okay. I wasn't sure if it's same thing :-)

@SparkQA

SparkQA commented Jun 4, 2020

Copy link
Copy Markdown

Test build #123513 has finished for PR 28722 at commit 3f8bab5.

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

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.

3 participants