Skip to content
This repository has been archived by the owner on Aug 19, 2024. It is now read-only.

Commit

Permalink
Stop publishing to the internal container registry (heroku#247)
Browse files Browse the repository at this point in the history
Currently the base images are published not only to Docker Hub, but also
an internal container registry.

However, the last remaining consumer of the base-images from the
internal registry is migrating away from these images as of:
heroku/kodon#483

As such, there is no longer any need to publish to this registry, which
simplifies the publishing process here, and means one less set of
credentials to manage/rotate.

GUS-W-15019312.
  • Loading branch information
edmorley authored Feb 12, 2024
1 parent 69c5a0f commit 7a0073e
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 17 deletions.
5 changes: 0 additions & 5 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,11 +29,6 @@ jobs:
STACK_VERSION: "${{ matrix.stack-version }}"
DOCKER_HUB_TOKEN: "${{ secrets.DOCKER_HUB_TOKEN }}"
DOCKER_HUB_USERNAME: "${{ secrets.DOCKER_HUB_USERNAME }}"
ID_SERVICE_PASSWORD: "${{ secrets.ID_SERVICE_PASSWORD }}"
ID_SERVICE_TOKEN_ENDPOINT: "${{ secrets.ID_SERVICE_TOKEN_ENDPOINT }}"
ID_SERVICE_USERNAME: "${{ secrets.ID_SERVICE_USERNAME }}"
INTERNAL_REGISTRY_HOST: "${{ secrets.INTERNAL_REGISTRY_HOST }}"
INTERNAL_REGISTRY_USERNAME: "${{ secrets.INTERNAL_REGISTRY_USERNAME }}"
MANIFEST_APP_TOKEN: "${{ secrets.MANIFEST_APP_TOKEN }}"
MANIFEST_APP_URL: "${{ secrets.MANIFEST_APP_URL }}"
strategy:
Expand Down
12 changes: 0 additions & 12 deletions bin/publish-to-registries.sh
Original file line number Diff line number Diff line change
Expand Up @@ -11,11 +11,6 @@ bin/build.sh "${STACK_VERSION}"

echo "Logging into Docker Hub..."
echo "${DOCKER_HUB_TOKEN}" | docker login -u "${DOCKER_HUB_USERNAME}" --password-stdin

echo "Logging into internal container registry..."
curl -sSf --retry 3 -X POST "$ID_SERVICE_TOKEN_ENDPOINT" -d "{\"username\":\"${ID_SERVICE_USERNAME}\",\"password\":\"${ID_SERVICE_PASSWORD}\"}" \
| jq -er ".raw_id_token" \
| docker login "$INTERNAL_REGISTRY_HOST" -u "$INTERNAL_REGISTRY_USERNAME" --password-stdin
)

push_group() {
Expand All @@ -30,7 +25,6 @@ push_group() {
}

publicTag="heroku/heroku:${STACK_VERSION}"
internalTag="${INTERNAL_REGISTRY_HOST}/s/${ID_SERVICE_USERNAME}/heroku:${STACK_VERSION}"

# Push nightly tags to Docker Hub (e.g. heroku/heroku:22.nightly)
push_group "${publicTag}" ".nightly"
Expand All @@ -39,12 +33,6 @@ if [ "$GITHUB_REF_TYPE" == 'tag' ]; then
# Push release tags to Docker Hub (e.g. heroku/heroku:22.v99)
push_group "${publicTag}" ".${GITHUB_REF_NAME}"

# Push release tags to internal registry
push_group "${internalTag}" ".${GITHUB_REF_NAME}"

# Push latest/no-suffix tags to Docker Hub (e.g. heroku/heroku:22)
push_group "${publicTag}" ""

# Push latest/no-suffix tags to internal registry
push_group "${internalTag}" ""
fi

0 comments on commit 7a0073e

Please sign in to comment.