Skip to content

Commit

Permalink
Remove DockerHub auth from workflow (apache#719)
Browse files Browse the repository at this point in the history
Temporarily remove DOCKERHUB credentials since they're not being
passed to workflows. Containers are public, so auth isn't
required. This unblocks dev work while we investigate the credentials
issue.
  • Loading branch information
edespino authored Nov 20, 2024
1 parent 22856f4 commit f92faf0
Showing 1 changed file with 0 additions and 16 deletions.
16 changes: 0 additions & 16 deletions .github/workflows/build-cloudberry.yml
Original file line number Diff line number Diff line change
Expand Up @@ -58,10 +58,6 @@
# - Aggregates status from all jobs
# - Provides failure notifications if needed
#
# Required Secrets:
# DOCKERHUB_USER: DockerHub username for container access
# DOCKERHUB_TOKEN: DockerHub access token
#
# Resource Requirements:
# - Disk: Minimum 20GB free space
# - Memory: Minimum 8GB RAM
Expand Down Expand Up @@ -107,7 +103,6 @@
# - Job execution reports (7 days retention)
#
# Dependencies:
# - Docker Hub credentials for container access
# - Cloudberry build scripts repository
# - GitHub Actions artifacts for job communication
#
Expand Down Expand Up @@ -140,8 +135,6 @@ permissions:
actions: write

env:
DOCKERHUB_USER: ${{ secrets.DOCKERHUB_USER }}
DOCKERHUB_TOKEN: ${{ secrets.DOCKERHUB_TOKEN }}
LOG_RETENTION_DAYS: 7

jobs:
Expand Down Expand Up @@ -186,9 +179,6 @@ jobs:

container:
image: apache/incubator-cloudberry:cbdb-build-rocky9-latest
credentials:
username: ${{ env.DOCKERHUB_USER }}
password: ${{ env.DOCKERHUB_TOKEN }}
options: >-
--user root
-h cdw
Expand Down Expand Up @@ -470,9 +460,6 @@ jobs:

container:
image: apache/incubator-cloudberry:cbdb-test-rocky9-latest
credentials:
username: ${{ env.DOCKERHUB_USER }}
password: ${{ env.DOCKERHUB_TOKEN }}
options: >-
--user root
-h cdw
Expand Down Expand Up @@ -653,9 +640,6 @@ jobs:

container:
image: apache/incubator-cloudberry:cbdb-build-rocky9-latest
credentials:
username: ${{ env.DOCKERHUB_USER }}
password: ${{ env.DOCKERHUB_TOKEN }}
options: >-
--user root
-h cdw
Expand Down

0 comments on commit f92faf0

Please sign in to comment.