Skip to content

Commit 5683e84

Browse files
authored
revert(template): Publish to quay.io (#584) (#593)
This reverts commit 96368df. This change is reverted because the current default Helm version used by the publish-helm-chart action errors out when trying to push Helm Charts to quay.io. This issue is decribed in the upstream issue: helm/helm#32144. The only option we currently would have had was to downgrade to Helm 3.17.x. We decided to wait a little longer for either a) a new, fixed Helm version, or b) to downgrade until a fixed version is released. This revert is needed to unblock other changes in this repo to be rolled out without breaking CI.
1 parent 96368df commit 5683e84

2 files changed

Lines changed: 4 additions & 46 deletions

File tree

template/.github/workflows/build.yaml.j2

Lines changed: 4 additions & 42 deletions
Original file line numberDiff line numberDiff line change
@@ -173,7 +173,7 @@ jobs:
173173
build-arguments: VERSION=${{ steps.version.outputs.OPERATOR_VERSION }}
174174
container-file: docker/Dockerfile
175175

176-
- name: Publish Container Image to oci.stackable.tech
176+
- name: Publish Container Image
177177
if: ${{ !github.event.pull_request.head.repo.fork }}
178178
uses: stackabletech/actions/publish-image@6a84fcfccf4ee37d85217407bc6a94742afbb45f # v0.14.2
179179
with:
@@ -184,17 +184,6 @@ jobs:
184184
image-manifest-tag: ${{ steps.build.outputs.image-manifest-tag }}
185185
source-image-uri: ${{ steps.build.outputs.image-manifest-uri }}
186186

187-
- name: Publish Container Image to quay.io
188-
if: ${{ !github.event.pull_request.head.repo.fork }}
189-
uses: stackabletech/actions/publish-image@6a84fcfccf4ee37d85217407bc6a94742afbb45f # v0.14.2
190-
with:
191-
image-registry-uri: quay.io
192-
image-registry-username: stackable+robot_sdp_github_action_build
193-
image-registry-password: ${{ secrets.QUAY_ROBOT_SDP_GITHUB_ACTION_BUILD_SECRET }}
194-
image-repository: stackable/sdp/${{ env.OPERATOR_NAME }}
195-
image-manifest-tag: ${{ steps.build.outputs.image-manifest-tag }}
196-
source-image-uri: ${{ steps.build.outputs.image-manifest-uri }}
197-
198187
publish-index-manifest:
199188
name: Publish/Sign ${{ needs.build-container-image.outputs.operator-version }} Index
200189
if: |
@@ -213,7 +202,7 @@ jobs:
213202
with:
214203
persist-credentials: false
215204

216-
- name: Publish and Sign Image Index to oci.stackable.tech
205+
- name: Publish and Sign Image Index
217206
uses: stackabletech/actions/publish-image-index-manifest@6a84fcfccf4ee37d85217407bc6a94742afbb45f # v0.14.2
218207
with:
219208
image-registry-uri: oci.stackable.tech
@@ -222,15 +211,6 @@ jobs:
222211
image-repository: sdp/${{ env.OPERATOR_NAME }}
223212
image-index-manifest-tag: ${{ needs.build-container-image.outputs.operator-version }}
224213

225-
- name: Publish and Sign Image Index to quay.io
226-
uses: stackabletech/actions/publish-image-index-manifest@6a84fcfccf4ee37d85217407bc6a94742afbb45f # v0.14.2
227-
with:
228-
image-registry-uri: quay.io
229-
image-registry-username: stackable+robot_sdp_github_action_build
230-
image-registry-password: ${{ secrets.QUAY_ROBOT_SDP_GITHUB_ACTION_BUILD_SECRET }}
231-
image-repository: stackable/sdp/${{ env.OPERATOR_NAME }}
232-
image-index-manifest-tag: ${{ needs.build-container-image.outputs.operator-version }}
233-
234214
publish-helm-chart:
235215
name: Package/Publish ${{ needs.build-container-image.outputs.operator-version }} Helm Chart
236216
if: |
@@ -249,7 +229,7 @@ jobs:
249229
persist-credentials: false
250230
submodules: recursive
251231

252-
- name: Package, Publish, and Sign Helm Chart to oci.stackable.tech
232+
- name: Package, Publish, and Sign Helm Chart
253233
uses: stackabletech/actions/publish-helm-chart@6a84fcfccf4ee37d85217407bc6a94742afbb45f # v0.14.2
254234
with:
255235
chart-registry-uri: oci.stackable.tech
@@ -261,18 +241,6 @@ jobs:
261241
app-version: ${{ needs.build-container-image.outputs.operator-version }}
262242
publish-and-sign: ${{ !github.event.pull_request.head.repo.fork }}
263243

264-
- name: Package, Publish, and Sign Helm Chart to quay.io
265-
uses: stackabletech/actions/publish-helm-chart@6a84fcfccf4ee37d85217407bc6a94742afbb45f # v0.14.2
266-
with:
267-
chart-registry-uri: quay.io
268-
chart-registry-username: stackable+robot_sdp_charts_github_action_build
269-
chart-registry-password: ${{ secrets.QUAY_ROBOT_SDP_CHARTS_GITHUB_ACTION_BUILD_SECRET }}
270-
chart-repository: stackable/sdp-charts
271-
chart-directory: deploy/helm/${{ env.OPERATOR_NAME }}
272-
chart-version: ${{ needs.build-container-image.outputs.operator-version }}
273-
app-version: ${{ needs.build-container-image.outputs.operator-version }}
274-
publish-and-sign: ${{ !github.event.pull_request.head.repo.fork }}
275-
276244
openshift-preflight-check:
277245
name: Run OpenShift Preflight Check for ${{ needs.build-container-image.outputs.operator-version }}-${{ matrix.arch }}
278246
if: |
@@ -291,18 +259,12 @@ jobs:
291259
- arm64
292260
runs-on: ubuntu-latest
293261
steps:
294-
- name: Run OpenShift Preflight Check for oci.stackable.tech
262+
- name: Run OpenShift Preflight Check
295263
uses: stackabletech/actions/run-openshift-preflight@6a84fcfccf4ee37d85217407bc6a94742afbb45f # v0.14.2
296264
with:
297265
image-index-uri: oci.stackable.tech/sdp/${{ env.OPERATOR_NAME }}:${{ needs.build-container-image.outputs.operator-version }}
298266
image-architecture: ${{ matrix.arch }}
299267

300-
- name: Run OpenShift Preflight Check for quay.io
301-
uses: stackabletech/actions/run-openshift-preflight@6a84fcfccf4ee37d85217407bc6a94742afbb45f # v0.14.2
302-
with:
303-
image-index-uri: quay.io/stackable/sdp/${{ env.OPERATOR_NAME }}:${{ needs.build-container-image.outputs.operator-version }}
304-
image-architecture: ${{ matrix.arch }}
305-
306268
# This job is a required check in GitHub Settings for this repository.
307269
# It saves us having to list many required jobs, or work around dynamically
308270
# named jobs (since there is no concept of required settings).

template/deploy/helm/[[operator]]/values/quay.io.yaml

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

0 commit comments

Comments
 (0)