Skip to content

Commit ec8fbf4

Browse files
committed
add more checks to ensure vars are set
1 parent 4173ef7 commit ec8fbf4

1 file changed

Lines changed: 22 additions & 0 deletions

File tree

mkdocs/docs/how-to-release.md

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -134,14 +134,20 @@ This action will generate two final artifacts:
134134
If `gh` is available, watch the GitHub Action progress using:
135135

136136
```bash
137+
: "${GIT_TAG:?ERROR: GIT_TAG is not set or is empty}"
138+
137139
RUN_ID=$(gh run list --repo apache/iceberg-python --workflow "Python Build Release Candidate" --branch "${GIT_TAG}" --event push --json databaseId -q '.[0].databaseId')
140+
: "${RUN_ID:?ERROR: RUN_ID could not be determined}"
141+
138142
echo "Waiting for workflow to complete, this will take several minutes..."
139143
gh run watch $RUN_ID --repo apache/iceberg-python
140144
```
141145

142146
and download the artifacts using:
143147

144148
```bash
149+
: "${RUN_ID:?ERROR: RUN_ID is not set or is empty}"
150+
145151
gh run download $RUN_ID --repo apache/iceberg-python
146152
```
147153

@@ -159,6 +165,9 @@ Navigate to the artifact directory. Generate signature and checksum files:
159165
* `.sha512` files: SHA-512 checksums for verifying file integrity.
160166

161167
```bash
168+
: "${VERSION:?ERROR: VERSION is not set or is empty}"
169+
: "${RC:?ERROR: RC is not set or is empty}"
170+
162171
(
163172
cd svn-release-candidate-${VERSION}rc${RC}
164173

@@ -177,6 +186,10 @@ The parentheses `()` create a subshell. Any changes to the directory (`cd`) are
177186
Now, upload the files from the same directory:
178187

179188
```bash
189+
: "${VERSION:?ERROR: VERSION is not set or is empty}"
190+
: "${VERSION_WITH_RC:?ERROR: VERSION_WITH_RC is not set or is empty}"
191+
: "${RC:?ERROR: RC is not set or is empty}"
192+
180193
export SVN_TMP_DIR=/tmp/iceberg-${VERSION}/
181194
svn checkout https://dist.apache.org/repos/dist/dev/iceberg $SVN_TMP_DIR
182195

@@ -215,6 +228,9 @@ Update the artifact directory to PyPi using `twine`. This **won't** bump the ver
215228
<!-- prettier-ignore-end -->
216229

217230
```bash
231+
: "${VERSION:?ERROR: VERSION is not set or is empty}"
232+
: "${RC:?ERROR: RC is not set or is empty}"
233+
218234
twine upload pypi-release-candidate-${VERSION}rc${RC}/*
219235
```
220236

@@ -227,6 +243,10 @@ Verify the artifact is uploaded to [PyPi](https://pypi.org/project/pyiceberg/#hi
227243
Final step is to generate the email to the dev mail list:
228244

229245
```bash
246+
: "${GIT_TAG:?ERROR: GIT_TAG is not set or is empty}"
247+
: "${VERSION:?ERROR: VERSION is not set or is empty}"
248+
: "${VERSION_WITH_RC:?ERROR: VERSION_WITH_RC is not set or is empty}"
249+
230250
export GIT_TAG_REF=$(git show-ref ${GIT_TAG})
231251
export GIT_TAG_HASH=${GIT_TAG_REF:0:40}
232252
export LAST_COMMIT_ID=$(git rev-list ${GIT_TAG} 2> /dev/null | head -n 1)
@@ -340,6 +360,8 @@ The latest version can be pushed to PyPi. Check out the Apache SVN and make sure
340360
<!-- prettier-ignore-end -->
341361

342362
```bash
363+
: "${VERSION:?ERROR: VERSION is not set or is empty}"
364+
343365
svn checkout https://dist.apache.org/repos/dist/release/iceberg/pyiceberg-${VERSION} /tmp/iceberg-dist-release/pyiceberg-${VERSION}
344366

345367
cd /tmp/iceberg-dist-release/pyiceberg-${VERSION}

0 commit comments

Comments
 (0)