Skip to content

Conversation

@zsxwing
Copy link
Member

@zsxwing zsxwing commented Jul 20, 2018

What changes were proposed in this pull request?

The correct group id should be org.apache.spark. This is causing the nightly build failure: https://amplab.cs.berkeley.edu/jenkins/job/spark-master-maven-snapshots/2295/console

[ERROR] Failed to execute goal org.apache.maven.plugins:maven-deploy-plugin:2.8.2:deploy (default-deploy) on project spark-kubernetes-integration-tests_2.11: Failed to deploy artifacts: Could not transfer artifact spark-kubernetes-integration-tests:spark-kubernetes-integration-tests_2.11:jar:2.4.0-20180720.101629-1 from/to apache.snapshots.https (https://repository.apache.org/content/repositories/snapshots): Access denied to: https://repository.apache.org/content/repositories/snapshots/spark-kubernetes-integration-tests/spark-kubernetes-integration-tests_2.11/2.4.0-SNAPSHOT/spark-kubernetes-integration-tests_2.11-2.4.0-20180720.101629-1.jar, ReasonPhrase: Forbidden. -> [Help 1] [ERROR]

How was this patch tested?

Jenkins.

@zsxwing
Copy link
Member Author

zsxwing commented Jul 20, 2018

cc @gatorsmile

@SparkQA
Copy link

SparkQA commented Jul 20, 2018

@SparkQA
Copy link

SparkQA commented Jul 20, 2018

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

@gatorsmile
Copy link
Member

cc @mccheah @ssuchter

</parent>

<artifactId>spark-kubernetes-integration-tests_2.11</artifactId>
<groupId>spark-kubernetes-integration-tests</groupId>
Copy link
Member

Choose a reason for hiding this comment

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

I am wondering if we need groupId ?

Copy link
Member Author

Choose a reason for hiding this comment

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

I am wondering if we need groupId?

Yes. Each project must have a groupId and artifactId.

Copy link
Member Author

Choose a reason for hiding this comment

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

Discussed offline. groupId can be ignored and it will inherit from the parent module. I just removed it.

@gatorsmile
Copy link
Member

LGTM

@SparkQA
Copy link

SparkQA commented Jul 20, 2018

@SparkQA
Copy link

SparkQA commented Jul 20, 2018

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

@gatorsmile
Copy link
Member

Thanks! Merged to master.

@asfgit asfgit closed this in f765bb7 Jul 20, 2018
@zsxwing zsxwing deleted the fix-k8s-test branch July 20, 2018 22:33
@SparkQA
Copy link

SparkQA commented Jul 20, 2018

Test build #93362 has finished for PR 21831 at commit 4345139.

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

@SparkQA
Copy link

SparkQA commented Jul 21, 2018

Test build #93368 has finished for PR 21831 at commit 980d30c.

  • 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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants