Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SPARK-27026][BUILD] Upgrade Docker image for release build to Ubuntu 18.04 LTS #23932

Closed
wants to merge 2 commits into from

Conversation

dbtsai
Copy link
Member

@dbtsai dbtsai commented Mar 1, 2019

What changes were proposed in this pull request?

Upgrade Docker image for release build to Ubuntu 18.04LTS

How was this patch tested?

Manually tested.

@dbtsai
Copy link
Member Author

dbtsai commented Mar 1, 2019

cc @dongjoon-hyun @vanzin

@vanzin
Copy link
Contributor

vanzin commented Mar 1, 2019

I'm fine with it but maybe @felixcheung has opinions about the R stuff.

@SparkQA
Copy link

SparkQA commented Mar 2, 2019

Test build #102929 has finished for PR 23932 at commit 9d34977.

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

# * Python/PyPandoc (2.7.12/3.5.2)
# * R-base/R-base-dev (3.3.2+)
# * Python/PyPandoc (2.7.15/3.6.7)
# * R-base/R-base-dev (3.5.0+)
Copy link
Member

Choose a reason for hiding this comment

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

I'm not sure - particularly whether that brings in packages that we haven't been testing with. I guess we will find out in the next RC

Copy link
Member

Choose a reason for hiding this comment

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

FYI, we don't run R 3.5+ in Jenkins or any where

Copy link
Member Author

Choose a reason for hiding this comment

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

In Ubuntu 18.04LTS, R 3.3 is no longer supported. Only R 3.5 is supported.

@@ -38,7 +42,8 @@ ARG PIP_PKGS="pyopenssl pypandoc numpy pygments sphinx"
#
# This is all in a single "RUN" command so that if anything changes, "apt update" is run to fetch
# the most current package versions (instead of potentially using old versions cached by docker).
RUN echo 'deb http://cran.cnr.Berkeley.edu/bin/linux/ubuntu xenial/' >> /etc/apt/sources.list && \
RUN apt-get update && $APT_INSTALL gnupg && \
Copy link
Member

Choose a reason for hiding this comment

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

18.04 doesn't have gpg? I can see why here but nit all other places we are apt-get clean after
(though maybe unnecessary in a build image)

Copy link
Member Author

Choose a reason for hiding this comment

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

gpg is not installed by default in 18.04, and I was surprised.

Copy link
Member

@srowen srowen left a comment

Choose a reason for hiding this comment

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

Looks reasonable to me

Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

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

LGTM. I know that @dbtsai is using this one. Yep. We can see the result at next RC vote.

@SparkQA
Copy link

SparkQA commented Mar 4, 2019

Test build #102966 has finished for PR 23932 at commit e09aa4e.

  • This patch fails due to an unknown error code, -9.
  • This patch merges cleanly.
  • This patch adds no public classes.

@dilipbiswal
Copy link
Contributor

retest this please

@SparkQA
Copy link

SparkQA commented Mar 4, 2019

Test build #102971 has finished for PR 23932 at commit e09aa4e.

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

@SparkQA
Copy link

SparkQA commented Mar 5, 2019

Test build #103035 has finished for PR 23932 at commit d0af942.

  • This patch fails due to an unknown error code, -9.
  • This patch merges cleanly.
  • This patch adds no public classes.

@dongjoon-hyun
Copy link
Member

Retest this please

@SparkQA
Copy link

SparkQA commented Mar 5, 2019

Test build #103062 has finished for PR 23932 at commit d0af942.

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

@dongjoon-hyun
Copy link
Member

Retest this please

@SparkQA
Copy link

SparkQA commented Mar 6, 2019

Test build #103068 has finished for PR 23932 at commit d0af942.

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

@dongjoon-hyun dongjoon-hyun changed the title [SPARK-27026] [Build] Upgrade Docker image for release build to Ubuntu 18.04LTS [SPARK-27026][BUILD] Upgrade Docker image for release build to Ubuntu 18.04 LTS Mar 6, 2019
@dongjoon-hyun
Copy link
Member

Merged to master.

@@ -84,7 +84,7 @@ GIT_REF=${GIT_REF:-master}

RELEASE_STAGING_LOCATION="https://dist.apache.org/repos/dist/dev/spark"

GPG="gpg -u $GPG_KEY --no-tty --batch"
GPG="gpg -u $GPG_KEY --no-tty --batch --pinentry-mode loopback"
Copy link
Member

Choose a reason for hiding this comment

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

I noticed the failures on Jenkins job spark-master-package: https://amplab.cs.berkeley.edu/jenkins/view/Spark%20Packaging/job/spark-master-package/2275/console

It seems to be related to this change.

gpg: invalid option "--pinentry-mode"
Deleting credential directory /home/jenkins/workspace/spark-master-package/spark-utils/new-release-scripts/jenkins/jenkins-credentials-tVKlQBOF
Build step 'Execute shell' marked build as failure
[WS-CLEANUP] Deleting project workspace...[WS-CLEANUP] done
Finished: FAILURE

cc @dongjoon-hyun @dbtsai

Copy link
Member

Choose a reason for hiding this comment

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

@dbtsai dbtsai deleted the ubuntu18.04 branch November 11, 2019 23:00
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.

8 participants