Skip to content

Commit 60c3358

Browse files
authored
ci: remove docker step on main (#951)
We no longer consume these images so lets stop building them. Additionally we recently rotated some credentials and this step broke. Rather than fixing it I think we should just remove it since we don't use the artifacts anymore. Test Plan: watch CI on main
1 parent 045b0e0 commit 60c3358

File tree

2 files changed

+0
-83
lines changed

2 files changed

+0
-83
lines changed

.github/workflows/ci.yml

Lines changed: 0 additions & 70 deletions
Original file line numberDiff line numberDiff line change
@@ -113,73 +113,3 @@ jobs:
113113
# Check if the generated code is up-to-date
114114
- run: .github/workflows/buf-generate-check.sh
115115

116-
# We build a shared docker image called "zoekt". This is not pushed, but is
117-
# used for creating the indexserver and webserver images.
118-
docker:
119-
if: github.ref == 'refs/heads/main'
120-
runs-on: ubuntu-latest
121-
needs:
122-
- "test"
123-
- "shellcheck"
124-
steps:
125-
- name: checkout
126-
uses: actions/checkout@v3
127-
128-
- name: version
129-
id: version
130-
run: .github/workflows/docker-version.sh
131-
132-
- name: docker-meta-webserver
133-
id: meta-webserver
134-
uses: docker/metadata-action@v3
135-
with:
136-
images: |
137-
sourcegraph/zoekt-webserver
138-
tags: |
139-
type=ref,event=branch
140-
type=ref,event=pr
141-
type=semver,pattern={{version}}
142-
type=sha
143-
- name: docker-meta-indexserver
144-
id: meta-indexserver
145-
uses: docker/metadata-action@v3
146-
with:
147-
images: |
148-
sourcegraph/zoekt-indexserver
149-
tags: |
150-
type=ref,event=branch
151-
type=ref,event=pr
152-
type=semver,pattern={{version}}
153-
type=sha
154-
155-
- name: build-zoekt
156-
uses: docker/build-push-action@v4
157-
with:
158-
context: .
159-
tags: "zoekt:latest"
160-
push: "false"
161-
build-args: VERSION=${{ steps.version.outputs.value }}
162-
163-
- name: Login to Docker Hub
164-
uses: docker/login-action@v2
165-
with:
166-
username: ${{ secrets.DOCKERHUB_USERNAME }}
167-
password: ${{ secrets.DOCKERHUB_TOKEN }}
168-
169-
- name: build-push-webserver
170-
uses: docker/build-push-action@v4
171-
with:
172-
context: .
173-
tags: sourcegraph/zoekt-webserver:${{ steps.version.outputs.value }}, ${{ steps.meta-webserver.outputs.tags }}, sourcegraph/zoekt-webserver:latest
174-
file: Dockerfile.webserver
175-
cache-from: sourcegraph/zoekt-webserver:latest
176-
push: true
177-
178-
- name: build-push-indexserver
179-
uses: docker/build-push-action@v4
180-
with:
181-
context: .
182-
tags: sourcegraph/zoekt-indexserver:${{ steps.version.outputs.value }}, ${{ steps.meta-indexserver.outputs.tags }}, sourcegraph/zoekt-indexserver:latest
183-
file: Dockerfile.indexserver
184-
cache-from: sourcegraph/zoekt-indexserver:latest
185-
push: true

.github/workflows/docker-version.sh

Lines changed: 0 additions & 13 deletions
This file was deleted.

0 commit comments

Comments
 (0)