Skip to content

Conversation

@ifilonenko
Copy link
Contributor

What changes were proposed in this pull request?

I added integration tests for PySpark ( + checking JVM options + RemoteFileTest) which wasn't properly merged in the initial integration test PR.

How was this patch tested?

I tested this with integration tests using:

dev/dev-run-integration-tests.sh --spark-tgz spark-2.4.0-SNAPSHOT-bin-2.7.3.tgz

@ifilonenko
Copy link
Contributor Author

ifilonenko commented Jun 18, 2018

@ssuchter @holdenk @mccheah @shaneknapp for review

@ifilonenko
Copy link
Contributor Author

I don't know if Jenkins build a distribution with --pip. TBD based on success. But locally, this worked when I ran:
dev/make-distribution.sh --pip --tgz -Phadoop-2.7 -Pkubernetes

@SparkQA
Copy link

SparkQA commented Jun 18, 2018

@SparkQA
Copy link

SparkQA commented Jun 18, 2018

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/265/

@SparkQA
Copy link

SparkQA commented Jun 18, 2018

Test build #92012 has finished for PR 21583 at commit 2707dee.

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

@ifilonenko
Copy link
Contributor Author

retest this please

1 similar comment
@ifilonenko
Copy link
Contributor Author

retest this please

@SparkQA
Copy link

SparkQA commented Jun 18, 2018

Test build #92013 has finished for PR 21583 at commit 2707dee.

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

@SparkQA
Copy link

SparkQA commented Jun 18, 2018

@SparkQA
Copy link

SparkQA commented Jun 18, 2018

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/268/

@SparkQA
Copy link

SparkQA commented Jun 18, 2018

Test build #92014 has finished for PR 21583 at commit 2707dee.

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

@ifilonenko
Copy link
Contributor Author

retest this please

@SparkQA
Copy link

SparkQA commented Jun 18, 2018

@SparkQA
Copy link

SparkQA commented Jun 18, 2018

Test build #92035 has finished for PR 21583 at commit 2707dee.

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

@SparkQA
Copy link

SparkQA commented Jun 18, 2018

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/283/

@ifilonenko
Copy link
Contributor Author

ifilonenko commented Jun 18, 2018

This PR is now just waiting on appropriate changes to bin/docker-image-tool.sh which are contained in the following PR: #21555 . Will wait until that is merged.

@ifilonenko
Copy link
Contributor Author

retest this please

@SparkQA
Copy link

SparkQA commented Jun 21, 2018

@SparkQA
Copy link

SparkQA commented Jun 21, 2018

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/385/

@SparkQA
Copy link

SparkQA commented Jun 21, 2018

Test build #92182 has finished for PR 21583 at commit 2707dee.

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

@ifilonenko
Copy link
Contributor Author

retest this please

@SparkQA
Copy link

SparkQA commented Jun 25, 2018

@SparkQA
Copy link

SparkQA commented Jun 25, 2018

Test build #92304 has finished for PR 21583 at commit 2707dee.

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

@SparkQA
Copy link

SparkQA commented Jun 25, 2018

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/461/

@foxish
Copy link
Contributor

foxish commented Jun 25, 2018

@ifilonenko, is this ready to go?

@SparkQA
Copy link

SparkQA commented Jun 29, 2018

@SparkQA
Copy link

SparkQA commented Jun 29, 2018

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/584/

@SparkQA
Copy link

SparkQA commented Jul 12, 2018

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/910/

@SparkQA
Copy link

SparkQA commented Jul 12, 2018

Test build #92952 has finished for PR 21583 at commit c0b5927.

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

@shaneknapp
Copy link
Contributor

test this please

@SparkQA
Copy link

SparkQA commented Jul 12, 2018

@SparkQA
Copy link

SparkQA commented Jul 12, 2018

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/912/

@SparkQA
Copy link

SparkQA commented Jul 12, 2018

Test build #92955 has finished for PR 21583 at commit c0b5927.

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

@shaneknapp
Copy link
Contributor

test this please

@SparkQA
Copy link

SparkQA commented Jul 12, 2018

@SparkQA
Copy link

SparkQA commented Jul 12, 2018

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/913/

@shaneknapp
Copy link
Contributor

test this please

@SparkQA
Copy link

SparkQA commented Jul 13, 2018

Test build #92956 has finished for PR 21583 at commit c0b5927.

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

@SparkQA
Copy link

SparkQA commented Jul 13, 2018

@SparkQA
Copy link

SparkQA commented Jul 13, 2018

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/914/

@SparkQA
Copy link

SparkQA commented Jul 13, 2018

Test build #92957 has finished for PR 21583 at commit c0b5927.

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

@shaneknapp
Copy link
Contributor

test this please

@SparkQA
Copy link

SparkQA commented Jul 13, 2018

Test build #92980 has finished for PR 21583 at commit c0b5927.

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

@SparkQA
Copy link

SparkQA commented Jul 13, 2018

@SparkQA
Copy link

SparkQA commented Jul 13, 2018

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/933/

@shaneknapp
Copy link
Contributor

test this please

@SparkQA
Copy link

SparkQA commented Jul 13, 2018

Test build #92983 has finished for PR 21583 at commit c0b5927.

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

@SparkQA
Copy link

SparkQA commented Jul 13, 2018

@SparkQA
Copy link

SparkQA commented Jul 13, 2018

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/935/

@shaneknapp
Copy link
Contributor

test this please

1 similar comment
@shaneknapp
Copy link
Contributor

test this please

@SparkQA
Copy link

SparkQA commented Jul 13, 2018

@SparkQA
Copy link

SparkQA commented Jul 13, 2018

Test build #92987 has finished for PR 21583 at commit c0b5927.

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

@SparkQA
Copy link

SparkQA commented Jul 13, 2018

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/938/

@ifilonenko
Copy link
Contributor Author

@mccheah
@foxish
Can we merge?

@mccheah
Copy link
Contributor

mccheah commented Jul 14, 2018

Yeah let's merge this - think there might be some work to clean this up a bit later.

@asfgit asfgit closed this in f1a99ad Jul 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants