Skip to content

Conversation

@sarutak
Copy link
Member

@sarutak sarutak commented May 28, 2021

What changes were proposed in this pull request?

This PR proposes to add docker-integratin-tests to run-tests.py and GA.
Once #32631 was merged but there was a lack of consideration.

Diff between this change and 692d95d merged in #32631 is as follows.

       if: github.repository != 'apache/spark'
       id: sync-branch
       run: |
+        apache_spark_ref=`git rev-parse HEAD`
         git fetch https://github.com/$GITHUB_REPOSITORY.git ${GITHUB_REF#refs/heads/}
         git -c user.name='Apache Spark Test Account' -c user.email='sparktestacc@gmail.com' merge --no-commit --progress --squash FETCH_HEAD
         git -c user.name='Apache Spark Test Account' -c user.email='sparktestacc@gmail.com' commit -m "Merged commit"
+        echo "::set-output name=APACHE_SPARK_REF::$apache_spark_ref"
     - name: Cache Scala, SBT and Maven
       uses: actions/cache@v2
       with:

Why are the changes needed?

CI for docker-integration-tests is absent for now.

Does this PR introduce any user-facing change?

GA.

How was this patch tested?

@sarutak sarutak changed the title Docker integration test ga take2 [SPARK-35483][INFRA] Add docker-integration-tests to run-tests.py and GA May 28, 2021
@sarutak
Copy link
Member Author

sarutak commented May 28, 2021

cc: @HyukjinKwon
Thank you for letting me know about the issue.
I believe this change can fix it.

Copy link
Member

@HyukjinKwon HyukjinKwon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@SparkQA
Copy link

SparkQA commented May 28, 2021

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

@HyukjinKwon
Copy link
Member

Merged to master.

@SparkQA
Copy link

SparkQA commented May 28, 2021

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

@SparkQA
Copy link

SparkQA commented May 28, 2021

Test build #139049 has finished for PR 32691 at commit bdbfea5.

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

@sarutak sarutak deleted the docker-integration-test-ga-take2 branch June 4, 2021 20:40
@sarutak sarutak restored the docker-integration-test-ga-take2 branch June 4, 2021 20:40
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