Skip to content

Commit

Permalink
[release] dont rely on release.sh being executable (#14445)
Browse files Browse the repository at this point in the history
Co-authored-by: Daniel Goldstein <[email protected]>
  • Loading branch information
ehigham and daniel-goldstein committed Apr 4, 2024
1 parent e7c152a commit 41126be
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 4 deletions.
6 changes: 5 additions & 1 deletion build.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3257,6 +3257,8 @@ steps:
name: test_dataproc-37
image:
valueFrom: ci_utils_image.image
resources:
preemptible: False
script: |
set -ex
Expand Down Expand Up @@ -3298,6 +3300,8 @@ steps:
name: test_dataproc-38
image:
valueFrom: ci_utils_image.image
resources:
preemptible: False
script: |
set -ex
Expand Down Expand Up @@ -3383,7 +3387,7 @@ steps:
HAIL_GENETICS_VEP_GRCH38_95_IMAGE=docker://{{ hailgenetics_vep_grch38_95_image.image }} \
AZURE_WHEEL=/io/azure-wheel/hail-*-py3-none-any.whl \
WEBSITE_TAR=/io/www.tar.gz \
./scripts/release.sh
bash scripts/release.sh
echo '1' > /io/release-hail-flag
inputs:
Expand Down
2 changes: 1 addition & 1 deletion hail/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -444,7 +444,7 @@ release: test-dataproc $(WHEEL)
HAIL_GENETICS_VEP_GRCH38_95_IMAGE=$(HAIL_GENETICS_VEP_GRCH38_95_IMAGE) \
AZURE_WHEEL=$(AZURE_WHEEL) \
WEBSITE_TAR=$(WEBSITE_TAR) \
./scripts/release.sh
bash scripts/release.sh

python/hail/docs/change_log.rst: python/hail/docs/change_log.md
sed -E "s/\(hail\#([0-9]+)\)/(\[#\1](https:\/\/github.com\/hail-is\/hail\/pull\/\1))/g" \
Expand Down
4 changes: 2 additions & 2 deletions hail/scripts/release.sh
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ SCRIPT_DIR=$( cd -- "$( dirname -- "${BASH_SOURCE[0]}" )" &> /dev/null && pwd )

usage() {
cat << EOF
usage: ./$(basename "$0")
usage: $(basename "$0")
All arguments are specified by environment variables. For example:
Expand All @@ -24,7 +24,7 @@ usage: ./$(basename "$0")
HAIL_GENETICS_VEP_GRCH38_95_IMAGE=docker://us-docker.pkg.dev/hail-vdc/hail/hailgenetics/vep-grch38-95:deploy-123abc
AZURE_WHEEL=/path/to/wheel/for/azure
WEBSITE_TAR=/path/to/www.tar.gz
./$(basename "$0")
bash $(basename "$0")
EOF
}

Expand Down

0 comments on commit 41126be

Please sign in to comment.