Skip to content

Commit c7945da

Browse files
authored
fix/release: remove appliance image publication (#800)
I'm not sure how this hasn't failed on other releases but we don't publish appliance anymore and this crashed promotion in the 6.12.0 release: ``` 👉 [ step] Running step "sg ops:migrator" -- [sg ops:migrator] +++ 👉 using Docker Hub registry index.docker.io/sourcegraph [sg ops:migrator] +++ 👉 pinning images to tag "6.12.0" [sg ops:migrator] Updated migrator [sg ops:migrator] - docker.io/sourcegraph/migrator:6.0.0@sha256:ec295eb0b743da6bf56777ca6524972267a5c442b0288095e2fe12fce38ebacc [sg ops:migrator] + index.docker.io/sourcegraph/migrator:6.12.0@sha256:b12184766122a8f22cef78f8ecbfd5cd20837f9d480716cea0e031e9f505b1f1 [ step] Step "sg ops:migrator" succeeded 👉 [ step] Running step "sg ops:appliance" [sg ops:appliance] +++ 👉 using Docker Hub registry index.docker.io/sourcegraph [sg ops:appliance] +++ 👉 pinning images to tag "6.12.0" [sg ops:appliance] ^^^ +++ ⨯ couldn't read charts/sourcegraph-appliance/values.yaml: open charts/sourcegraph-appliance/values.yaml: no such file or directory [sg ops:appliance] Step failed: exit status 1 🚨 Error: The command exited with status 1 ``` https://buildkite.com/sourcegraph/deploy-sourcegraph-helm/builds/2441#019bb42a-f692-41cc-9c2e-d3f27c6c6d95 ### Checklist - [ ] Follow the [manual testing process](https://github.com/sourcegraph/deploy-sourcegraph-helm/blob/main/TEST.md) - [ ] Update [changelog](https://github.com/sourcegraph/deploy-sourcegraph-helm/blob/main/charts/sourcegraph/CHANGELOG.md) - [ ] Update [Kubernetes update doc](https://docs.sourcegraph.com/admin/updates/kubernetes) ### Test plan <!-- As part of SOC2/GN-104 and SOC2/GN-105 requirements, all pull requests are REQUIRED to provide a "test plan". A test plan is a loose explanation of what you have done or implemented to test this, as outlined in our Testing principles and guidelines: https://docs.sourcegraph.com/dev/background-information/testing_principles Write your test plan here after the "Test plan" header. -->
1 parent 5ff81c8 commit c7945da

File tree

1 file changed

+0
-14
lines changed

1 file changed

+0
-14
lines changed

release.yaml

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -575,34 +575,20 @@ promoteToPublic:
575575
--docker-password $DOCKER_PASSWORD \
576576
charts/sourcegraph-migrator/
577577
578-
- name: sg ops:appliance
579-
cmd: |
580-
set -eu
581-
registry=index.docker.io/sourcegraph
582-
sg ops update-images \
583-
-t {{inputs.server.tag}} \
584-
-k helm \
585-
--registry "${registry}" \
586-
--docker-username $DOCKER_USERNAME \
587-
--docker-password $DOCKER_PASSWORD \
588-
charts/sourcegraph-appliance/
589-
590578
- name: "chart:version"
591579
cmd: |
592580
comby -matcher ".generic" -in-place "version: \":[~\d+\.\d+\.\d+]\"" 'version: "{{inputs.server.tag}}"' -f charts/sourcegraph/Chart.yaml
593581
comby -matcher ".generic" -in-place "version: \":[~\d+\.\d+\.\d+]\"" 'version: "{{inputs.server.tag}}"' -f charts/sourcegraph/examples/subchart/Chart.yaml
594582
comby -matcher ".generic" -in-place "version: \":[~\d+\.\d+\.\d+]\"" 'version: "{{inputs.server.tag}}"' -f charts/sourcegraph-executor/dind/Chart.yaml
595583
comby -matcher ".generic" -in-place "version: \":[~\d+\.\d+\.\d+]\"" 'version: "{{inputs.server.tag}}"' -f charts/sourcegraph-executor/k8s/Chart.yaml
596584
comby -matcher ".generic" -in-place "version: \":[~\d+\.\d+\.\d+]\"" 'version: "{{inputs.server.tag}}"' -f charts/sourcegraph-migrator/Chart.yaml
597-
comby -matcher ".generic" -in-place "version: \":[~\d+\.\d+\.\d+]\"" 'version: "{{inputs.server.tag}}"' -f charts/sourcegraph-appliance/Chart.yaml
598585
599586
- name: "chart:appVersion"
600587
cmd: |
601588
comby -matcher ".generic" -in-place "appVersion: \":[~\d+\.\d+\.\d+]\"" 'appVersion: "{{inputs.server.tag}}"' -f charts/sourcegraph/Chart.yaml
602589
comby -matcher ".generic" -in-place "appVersion: \":[~\d+\.\d+\.\d+]\"" 'appVersion: "{{inputs.server.tag}}"' -f charts/sourcegraph-executor/dind/Chart.yaml
603590
comby -matcher ".generic" -in-place "appVersion: \":[~\d+\.\d+\.\d+]\"" 'appVersion: "{{inputs.server.tag}}"' -f charts/sourcegraph-executor/k8s/Chart.yaml
604591
comby -matcher ".generic" -in-place "appVersion: \":[~\d+\.\d+\.\d+]\"" 'appVersion: "{{inputs.server.tag}}"' -f charts/sourcegraph-migrator/Chart.yaml
605-
comby -matcher ".generic" -in-place "appVersion: \":[~\d+\.\d+\.\d+]\"" 'appVersion: "{{inputs.server.tag}}"' -f charts/sourcegraph-appliance/Chart.yaml
606592
607593
- name: "update helm docs"
608594
cmd: ./scripts/helm-docs.sh

0 commit comments

Comments
 (0)