-
Notifications
You must be signed in to change notification settings - Fork 29k
[SPARK-24372][build] Add scripts to help with preparing releases. #21515
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
Changes from 4 commits
44454b6
1366f23
04f6371
48eda71
7fe133a
faab8ae
82ae00e
5771354
a39933e
b6d4c70
745342d
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,109 @@ | ||
| #!/usr/bin/env bash | ||
|
|
||
| # | ||
| # Licensed to the Apache Software Foundation (ASF) under one or more | ||
| # contributor license agreements. See the NOTICE file distributed with | ||
| # this work for additional information regarding copyright ownership. | ||
| # The ASF licenses this file to You under the Apache License, Version 2.0 | ||
| # (the "License"); you may not use this file except in compliance with | ||
| # the License. You may obtain a copy of the License at | ||
| # | ||
| # http://www.apache.org/licenses/LICENSE-2.0 | ||
| # | ||
| # Unless required by applicable law or agreed to in writing, software | ||
| # distributed under the License is distributed on an "AS IS" BASIS, | ||
| # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
| # See the License for the specific language governing permissions and | ||
| # limitations under the License. | ||
| # | ||
|
|
||
| set -e | ||
| SELF=$(cd $(dirname $0) && pwd) | ||
| . "$SELF/release-util.sh" | ||
|
|
||
| WORKDIR= | ||
| IMGTAG=latest | ||
| JAVA= | ||
| while getopts "d:j:nt:" opt; do | ||
| case $opt in | ||
| d) WORKDIR="$OPTARG" ;; | ||
| n) DRY_RUN=1 ;; | ||
| t) IMGTAG="$OPTARG" ;; | ||
| j) JAVA="$OPTARG" ;; | ||
| ?) error "Invalid option: $OPTARG" ;; | ||
| esac | ||
| done | ||
|
|
||
| if [ -z "$WORKDIR" ] || [ ! -d "$WORKDIR" ]; then | ||
| error "Work directory (-d) must be defined and exist." | ||
| fi | ||
|
|
||
| if [ -d "$WORKDIR/output" ]; then | ||
| read -p "Output directory already exists. Overwrite and continue? [y/n] " ANSWER | ||
| if [ "$ANSWER" != "y" ]; then | ||
| error "Exiting." | ||
| fi | ||
| fi | ||
|
|
||
| cd "$WORKDIR" | ||
| rm -rf "$WORKDIR/output" | ||
| mkdir "$WORKDIR/output" | ||
|
|
||
| get_release_info | ||
|
|
||
| # Place all RM scripts and necessary data in a local directory that must be defined in the command | ||
| # line. This directory is mounted into the image. | ||
| for f in "$SELF"/*; do | ||
| if [ -f "$f" ]; then | ||
| cp "$f" "$WORKDIR" | ||
| fi | ||
| done | ||
| GPG_KEY_FILE="$WORKDIR/gpg.key" | ||
| fcreate_secure "$GPG_KEY_FILE" | ||
| $GPG --export-secret-key --armor "$GPG_KEY" > "$GPG_KEY_FILE" | ||
|
|
||
| run_silent "Building spark-rm image with tag $IMGTAG..." "docker-build.log" \ | ||
| docker build -t "spark-rm:$IMGTAG" --build-arg UID=$UID "$SELF/spark-rm" | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. So we need to do
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. got it. This is a system variable. So we can't run this script with root user... |
||
|
|
||
| # Write the release information to a file with environment variables to be used when running the | ||
| # image. | ||
| ENVFILE="$WORKDIR/env.list" | ||
| fcreate_secure "$ENVFILE" | ||
|
|
||
| function cleanup { | ||
| rm -f "$ENVFILE" | ||
| rm -f "$GPG_KEY_FILE" | ||
| } | ||
|
|
||
| trap cleanup EXIT | ||
|
|
||
| cat > $ENVFILE <<EOF | ||
| DRY_RUN=$DRY_RUN | ||
| SKIP_TAG=$SKIP_TAG | ||
| RUNNING_IN_DOCKER=1 | ||
| GIT_BRANCH=$GIT_BRANCH | ||
| NEXT_VERSION=$NEXT_VERSION | ||
| RELEASE_VERSION=$RELEASE_VERSION | ||
| RELEASE_TAG=$RELEASE_TAG | ||
| GIT_REF=$GIT_REF | ||
| SPARK_PACKAGE_VERSION=$SPARK_PACKAGE_VERSION | ||
| ASF_USERNAME=$ASF_USERNAME | ||
| GIT_NAME=$GIT_NAME | ||
| GIT_EMAIL=$GIT_EMAIL | ||
| GPG_KEY=$GPG_KEY | ||
| ASF_PASSWORD=$ASF_PASSWORD | ||
| GPG_PASSPHRASE=$GPG_PASSPHRASE | ||
| EOF | ||
|
|
||
| JAVA_VOL= | ||
| if [ -n "$JAVA" ]; then | ||
| echo "JAVA_HOME=/opt/spark-java" >> $ENVFILE | ||
| JAVA_VOL="--volume $JAVA:/opt/spark-java" | ||
| fi | ||
|
|
||
| echo "Building $RELEASE_TAG; output will be at $WORKDIR/output" | ||
| docker run -ti \ | ||
| --env-file "$ENVFILE" \ | ||
| --volume "$WORKDIR:/opt/spark-rm" \ | ||
| $JAVA_VOL \ | ||
| "spark-rm:$IMGTAG" | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,59 @@ | ||
| #!/usr/bin/env bash | ||
|
|
||
| # | ||
| # Licensed to the Apache Software Foundation (ASF) under one or more | ||
| # contributor license agreements. See the NOTICE file distributed with | ||
| # this work for additional information regarding copyright ownership. | ||
| # The ASF licenses this file to You under the Apache License, Version 2.0 | ||
| # (the "License"); you may not use this file except in compliance with | ||
| # the License. You may obtain a copy of the License at | ||
| # | ||
| # http://www.apache.org/licenses/LICENSE-2.0 | ||
| # | ||
| # Unless required by applicable law or agreed to in writing, software | ||
| # distributed under the License is distributed on an "AS IS" BASIS, | ||
| # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
| # See the License for the specific language governing permissions and | ||
| # limitations under the License. | ||
| # | ||
|
|
||
| SELF=$(cd $(dirname $0) && pwd) | ||
| . "$SELF/release-util.sh" | ||
|
|
||
| while getopts "bn" opt; do | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. it would be nice to have a high level description in the script just saying this does a release which does things like tag, build, etc and pushes things to the asf spark repo. |
||
| case $opt in | ||
| b) GIT_BRANCH=$OPTARG ;; | ||
| n) DRY_RUN=1 ;; | ||
| ?) error "Invalid option: $OPTARG" ;; | ||
| esac | ||
| done | ||
|
|
||
| if [ "$RUNNING_IN_DOCKER" = "1" ]; then | ||
| # Inside docker, need to import the GPG key stored in the current directory. | ||
| echo $GPG_PASSPHRASE | $GPG --passphrase-fd 0 --import "$SELF/gpg.key" | ||
|
|
||
| # We may need to adjust the path since JAVA_HOME may be overridden by the driver script. | ||
| if [ -n "$JAVA_HOME" ]; then | ||
| export PATH="$JAVA_HOME/bin:$PATH" | ||
| else | ||
| # JAVA_HOME for the openjdk package. | ||
| export JAVA_HOME=/usr | ||
| fi | ||
| else | ||
| # Outside docker, need to ask for information about the release. | ||
| get_release_info | ||
| fi | ||
|
|
||
| if [ $SKIP_TAG = 0 ]; then | ||
| maybe_run "Creating release tag $RELEASE_TAG..." "tag.log" \ | ||
| "$SELF/release-tag.sh" | ||
| else | ||
| echo "Skipping tag creation for $RELEASE_TAG." | ||
| fi | ||
|
|
||
| run_silent "Building Spark..." "build.log" \ | ||
| "$SELF/release-build.sh" package | ||
| run_silent "Building documentation..." "docs.log" \ | ||
| "$SELF/release-build.sh" docs | ||
| maybe_run "Publishing release" "publish.log" \ | ||
| "$SELF/release-build.sh" publish-release | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
even if rat doesn't check, isn't vote.tmpl packaged into the source release this way?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for example,
.gitis removed from release here https://github.com/apache/spark/blob/master/dev/create-release/release-build.sh#L157There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are you saying this file should not be packaged in the source release? Not sure I see why that would be the case. There's a lot of stuff in
.rat-excludesthat is still packaged.