Skip to content

Commit a14cbcb

Browse files
authored
sourcegraph: remove repo-updater deployment (#671)
We no longer need this service, starting in 6.4. It's fully incorporated into other services now and an empty service doing nothing. Test plan: This setup has been running on S2 and Dotcom for a couple of days now. ### Checklist - [ ] Follow the [manual testing process](https://github.com/sourcegraph/deploy-sourcegraph-helm/blob/main/TEST.md) - [x] Update [changelog](https://github.com/sourcegraph/deploy-sourcegraph-helm/blob/main/charts/sourcegraph/CHANGELOG.md) - [x] Update [Kubernetes update doc](https://docs.sourcegraph.com/admin/updates/kubernetes): sourcegraph/docs#1110
1 parent b65c4ae commit a14cbcb

File tree

8 files changed

+1
-205
lines changed

8 files changed

+1
-205
lines changed

charts/sourcegraph/CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ Use `**BREAKING**:` to denote a breaking change
1010

1111
- Fix Pod Disruption Budget for sourcegraph-frontend
1212
- Added a startup probe to the gitserver statefulset to give it time to run the on-disk migration from repo names to repo IDs
13+
- The repo-updater service is no longer needed and has been removed from the chart.
1314

1415
## 5.10.0
1516

charts/sourcegraph/README.md

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -289,14 +289,6 @@ In addition to the documented values, all services also support the following va
289289
| redisStore.serviceAccount.create | bool | `false` | Enable creation of ServiceAccount for `redis-store` |
290290
| redisStore.serviceAccount.name | string | `""` | Name of the ServiceAccount to be created or an existing ServiceAccount |
291291
| redisStore.storageSize | string | `"100Gi"` | PVC Storage Request for `redis-store` data volume |
292-
| repoUpdater.containerSecurityContext | object | `{"allowPrivilegeEscalation":false,"readOnlyRootFilesystem":true,"runAsGroup":101,"runAsUser":100}` | Security context for the `repo-updater` container, learn more from the [Kubernetes documentation](https://kubernetes.io/docs/tasks/configure-pod-container/security-context/#set-the-security-context-for-a-container) |
293-
| repoUpdater.image.defaultTag | string | `"6.0.0@sha256:238702dde17eaa41f9dc5b5f379c08a9e57940587128ceda6008d7f06e72cccc"` | Docker image tag for the `repo-updater` image |
294-
| repoUpdater.image.name | string | `"repo-updater"` | Docker image name for the `repo-updater` image |
295-
| repoUpdater.name | string | `"repo-updater"` | Name used by resources. Does not affect service names or PVCs. |
296-
| repoUpdater.podSecurityContext | object | `{}` | Security context for the `repo-updater` pod, learn more from the [Kubernetes documentation](https://kubernetes.io/docs/tasks/configure-pod-container/security-context/#set-the-security-context-for-a-pod) |
297-
| repoUpdater.resources | object | `{"limits":{"cpu":"1","memory":"2Gi"},"requests":{"cpu":"1","memory":"500Mi"}}` | Resource requests & limits for the `repo-updater` container, learn more from the [Kubernetes documentation](https://kubernetes.io/docs/concepts/configuration/manage-resources-containers/) |
298-
| repoUpdater.serviceAccount.create | bool | `false` | Enable creation of ServiceAccount for `repo-updater` |
299-
| repoUpdater.serviceAccount.name | string | `""` | Name of the ServiceAccount to be created or an existing ServiceAccount |
300292
| searcher.containerSecurityContext | object | `{"allowPrivilegeEscalation":false,"readOnlyRootFilesystem":true,"runAsGroup":101,"runAsUser":100}` | Security context for the `searcher` container, learn more from the [Kubernetes documentation](https://kubernetes.io/docs/tasks/configure-pod-container/security-context/#set-the-security-context-for-a-container) |
301293
| searcher.image.defaultTag | string | `"6.0.0@sha256:c7508abda2202d4a33400ce23a95dd8d59fe6220d85d7fbee6fb186c55931336"` | Docker image tag for the `searcher` image |
302294
| searcher.image.name | string | `"searcher"` | Docker image name for the `searcher` image |

charts/sourcegraph/examples/common-modifications/override.yaml

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -171,15 +171,6 @@ redisStore:
171171
memory: 7Gi
172172
storageSize: 100Gi
173173

174-
repoUpdater:
175-
resources:
176-
limits:
177-
cpu: "1"
178-
memory: 2Gi
179-
requests:
180-
cpu: "1"
181-
memory: 500Mi
182-
183174
searcher:
184175
replicaCount: 2
185176
resources:

charts/sourcegraph/examples/gcp/override.yaml

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -69,11 +69,6 @@ preciseCodeIntel:
6969
SRC_LOG_FORMAT:
7070
value: json_gcp
7171

72-
repoUpdater:
73-
env:
74-
SRC_LOG_FORMAT:
75-
value: json_gcp
76-
7772
searcher:
7873
env:
7974
SRC_LOG_FORMAT:

charts/sourcegraph/templates/repo-updater/repo-updater.Deployment.yaml

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

charts/sourcegraph/templates/repo-updater/repo-updater.Service.yaml

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

charts/sourcegraph/templates/repo-updater/repo-updater.ServiceAccount.yaml

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

charts/sourcegraph/values.yaml

Lines changed: 0 additions & 33 deletions
Original file line numberDiff line numberDiff line change
@@ -1035,39 +1035,6 @@ redisStore:
10351035
# -- PVC Storage Request for `redis-store` data volume
10361036
storageSize: 100Gi
10371037

1038-
repoUpdater:
1039-
image:
1040-
# -- Docker image tag for the `repo-updater` image
1041-
defaultTag: 6.0.0@sha256:238702dde17eaa41f9dc5b5f379c08a9e57940587128ceda6008d7f06e72cccc
1042-
# -- Docker image name for the `repo-updater` image
1043-
name: "repo-updater"
1044-
# -- Security context for the `repo-updater` container,
1045-
# learn more from the [Kubernetes documentation](https://kubernetes.io/docs/tasks/configure-pod-container/security-context/#set-the-security-context-for-a-container)
1046-
containerSecurityContext:
1047-
allowPrivilegeEscalation: false
1048-
runAsUser: 100
1049-
runAsGroup: 101
1050-
readOnlyRootFilesystem: true
1051-
# -- Name used by resources. Does not affect service names or PVCs.
1052-
name: "repo-updater"
1053-
# -- Security context for the `repo-updater` pod,
1054-
# learn more from the [Kubernetes documentation](https://kubernetes.io/docs/tasks/configure-pod-container/security-context/#set-the-security-context-for-a-pod)
1055-
podSecurityContext: {}
1056-
# -- Resource requests & limits for the `repo-updater` container,
1057-
# learn more from the [Kubernetes documentation](https://kubernetes.io/docs/concepts/configuration/manage-resources-containers/)
1058-
resources:
1059-
limits:
1060-
cpu: "1"
1061-
memory: 2Gi
1062-
requests:
1063-
cpu: "1"
1064-
memory: 500Mi
1065-
serviceAccount:
1066-
# -- Enable creation of ServiceAccount for `repo-updater`
1067-
create: false
1068-
# -- Name of the ServiceAccount to be created or an existing ServiceAccount
1069-
name: ""
1070-
10711038
searcher:
10721039
image:
10731040
# -- Docker image tag for the `searcher` image

0 commit comments

Comments
 (0)