Skip to content

Commit

Permalink
Fix script typos for validating docker licenses. (#30176)
Browse files Browse the repository at this point in the history
  • Loading branch information
lostluck committed Feb 1, 2024
1 parent 1a5466b commit 574e7bf
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions contributor-docs/release-guide.md
Original file line number Diff line number Diff line change
Expand Up @@ -564,11 +564,11 @@ The following should be confirmed:
At
[https://hub.docker.com/u/apache](https://hub.docker.com/search?q=apache%2Fbeam&type=image),
visit each repository and navigate to "tags" tab. Verify images are pushed
with tags: `${RELEASE_VERSION}rc{RC_NUM}`
with tags: `${RELEASE_VERSION}rc${RC_NUM}`

Verify that third party licenses are included in Docker. You can do this with a simple script:

RC_TAG=${RELEASE_VERSION}rc{RC_NUM}
RC_TAG=${RELEASE_VERSION}rc${RC_NUM}
for pyver in 3.8 3.9 3.10 3.11; do
docker run --rm --entrypoint sh \
apache/beam_python${pyver}_sdk:${RC_TAG} \
Expand All @@ -577,7 +577,7 @@ Verify that third party licenses are included in Docker. You can do this with a

for javaver in 8 11 17; do
docker run --rm --entrypoint sh \
apache/beam_java${pyver}_sdk:${RC_TAG} \
apache/beam_java${javaver}_sdk:${RC_TAG} \
-c 'ls -al /opt/apache/beam/third_party_licenses/ | wc -l'
done

Expand Down

0 comments on commit 574e7bf

Please sign in to comment.