Skip to content

Conversation

@sarutak
Copy link
Member

@sarutak sarutak commented Dec 11, 2020

What changes were proposed in this pull request?

This PR adds kubernetes-integration-tests to GitHub Actions for Scala 2.13 build.

Why are the changes needed?

Now that the build pass with kubernetes-integration-tests and Scala 2.13, it's better to keep it build-able.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Should be done by GitHub Actions.
I also confirmed that the build passes with the following command.

$ build/sbt -Pscala-2.13 -Pkubernetes -Pkubernetes-integration-tests compile test:compile

@github-actions github-actions bot added the INFRA label Dec 11, 2020
@SparkQA
Copy link

SparkQA commented Dec 11, 2020

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

@SparkQA
Copy link

SparkQA commented Dec 11, 2020

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

@SparkQA
Copy link

SparkQA commented Dec 11, 2020

Test build #132651 has finished for PR 30731 at commit e850a0b.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

HyukjinKwon pushed a commit that referenced this pull request Dec 11, 2020
… Scala 2.13 build

### What changes were proposed in this pull request?

This PR adds `kubernetes-integration-tests` to GitHub Actions for Scala 2.13 build.

### Why are the changes needed?

Now that the build pass with `kubernetes-integration-tests` and Scala 2.13, it's better to keep it build-able.

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

Should be done by GitHub Actions.
I also confirmed that the build passes with the following command.
```
$ build/sbt -Pscala-2.13 -Pkubernetes -Pkubernetes-integration-tests compile test:compile
```

Closes #30731 from sarutak/github-actions-k8s.

Authored-by: Kousuke Saruta <[email protected]>
Signed-off-by: HyukjinKwon <[email protected]>
(cherry picked from commit 29cc5b3)
Signed-off-by: HyukjinKwon <[email protected]>
@HyukjinKwon
Copy link
Member

Merged to master and branch-3.1.

@dongjoon-hyun
Copy link
Member

Thank. you, @sarutak and @HyukjinKwon .

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants