Skip to content

Update build.sh to tag CKAN 2.10 and 2.9 images correctly#117

Merged
kowh-ai merged 1 commit intomainfrom
build.sh-to-tag-python-images-correctly
Apr 13, 2025
Merged

Update build.sh to tag CKAN 2.10 and 2.9 images correctly#117
kowh-ai merged 1 commit intomainfrom
build.sh-to-tag-python-images-correctly

Conversation

@kowh-ai
Copy link
Copy Markdown
Contributor

@kowh-ai kowh-ai commented Apr 11, 2025

Fixes #116

For CKAN 2.10 images

When the build.sh script runs, it initially builds the python:3.10-slim-bookworm based image and tags it correctly. However, when it proceeds to build the alpine:3.17 based image, it mistakenly tags it with the same python:3.10-slim-bookworm tag, therefore the tag points to the wrong image.

As a result, only the Alpine-based image is tagged.

CKAN 2.9 images has a similar problem

@kowh-ai kowh-ai requested a review from amercader April 11, 2025 13:10
@kowh-ai
Copy link
Copy Markdown
Contributor Author

kowh-ai commented Apr 11, 2025

@amercader - when you get a spare mo could you take a look at this please...I should be able to fix the 2.9 failed test over the weekend

@amercader
Copy link
Copy Markdown
Member

Makes sense @kowh-ai that was a nasty one

@kowh-ai kowh-ai self-assigned this Apr 11, 2025
@kowh-ai
Copy link
Copy Markdown
Contributor Author

kowh-ai commented Apr 13, 2025

CKAN 2.9 failure was just an anomaly - re-ran and finished just fine

@kowh-ai kowh-ai merged commit 03602c0 into main Apr 13, 2025
6 of 7 checks passed
@kowh-ai kowh-ai deleted the build.sh-to-tag-python-images-correctly branch April 13, 2025 13:33
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.

build.sh tags a CKAN 2.10 "alpine" image with a "python:3.10-slim-bookworm" tag

2 participants