Skip to content

Conversation

eseliger
Copy link
Member

@eseliger eseliger commented Apr 24, 2025

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

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: TBD.
eseliger added a commit to sourcegraph/docs that referenced this pull request Apr 30, 2025
The service is no longer needed. Corresponding helm chart update: sourcegraph/deploy-sourcegraph-helm#671

Test plan: n/a
@eseliger eseliger marked this pull request as ready for review April 30, 2025 16:48
@eseliger eseliger requested review from a team April 30, 2025 16:48
@michaellzc michaellzc changed the title Remove repo-updater from helm chart sourcegraph: remove repo-updater deployment Apr 30, 2025
@michaellzc
Copy link
Member

let's also wait for Release's stamp

eseliger added a commit to sourcegraph/deploy-sourcegraph-k8s that referenced this pull request Apr 30, 2025
Sister PR of sourcegraph/deploy-sourcegraph-helm#671.

Upgrade info alert is at sourcegraph/docs#1110.

The repo-updater service is no longer required starting with Sourcegraph 6.4.

Test plan: Repo-updater-less instances have been running for a while now.
eseliger added a commit to sourcegraph/deploy-sourcegraph-docker that referenced this pull request Apr 30, 2025
Sister PR of sourcegraph/deploy-sourcegraph-helm#671.

Upgrade info alert is at sourcegraph/docs#1110.

The repo-updater service is no longer required starting with Sourcegraph 6.4.

Test plan: Repo-updater-less instances have been running for a while now.
eseliger added a commit to sourcegraph/deploy-sourcegraph-docker that referenced this pull request Apr 30, 2025
Sister PR of sourcegraph/deploy-sourcegraph-helm#671.

Upgrade info alert is at sourcegraph/docs#1110.

The repo-updater service is no longer required starting with Sourcegraph 6.4.

Test plan: Repo-updater-less instances have been running for a while now.
eseliger added a commit to sourcegraph/deploy-sourcegraph that referenced this pull request Apr 30, 2025
Sister PR of sourcegraph/deploy-sourcegraph-helm#671.

Upgrade info alert is at sourcegraph/docs#1110.

The repo-updater service is no longer required starting with Sourcegraph 6.4.

Test plan: Repo-updater-less instances have been running for a while now.
eseliger added a commit to sourcegraph/deploy-sourcegraph-docker-customer-replica-1 that referenced this pull request Apr 30, 2025
Sister PR of sourcegraph/deploy-sourcegraph-helm#671.

Upgrade info alert is at sourcegraph/docs#1110.

The repo-updater service is no longer required starting with Sourcegraph 6.4.

Test plan: Repo-updater-less instances have been running for a while now.
eseliger added a commit to sourcegraph/deploy-sourcegraph that referenced this pull request Apr 30, 2025
Sister PR of sourcegraph/deploy-sourcegraph-helm#671.

Upgrade info alert is at sourcegraph/docs#1110.

The repo-updater service is no longer required starting with Sourcegraph
6.4.

Test plan: Repo-updater-less instances have been running for a while
now.

### Checklist

- [ ]
[CHANGELOG.md](https://github.com/sourcegraph/sourcegraph/blob/main/CHANGELOG.md)
updated
- [x] [K8s Upgrade notes
updated](https://github.com/sourcegraph/sourcegraph/blob/main/doc/admin/updates/kubernetes.md)
- [x] Sister
[deploy-sourcegraph-k8s](https://github.com/sourcegraph/deploy-sourcegraph-docker)
change: sourcegraph/deploy-sourcegraph-k8s#269
- [x] Sister
[deploy-sourcegraph-docker](https://github.com/sourcegraph/deploy-sourcegraph-docker)
change:
sourcegraph/deploy-sourcegraph-docker#1202
- [ ] All images have a valid tag and SHA256 sum
- [x] I acknowledge that
[deploy-sourcegraph-k8s](https://github.com/sourcegraph/deploy-sourcegraph-k8s)
is now the preferred Kubernetes deployment repository
eseliger added a commit to sourcegraph/deploy-sourcegraph-k8s that referenced this pull request Apr 30, 2025
Sister PR of
sourcegraph/deploy-sourcegraph-helm#671.

Upgrade info alert is at sourcegraph/docs#1110.

The repo-updater service is no longer required starting with Sourcegraph
6.4.

Test plan: Repo-updater-less instances have been running for a while
now.


---

## Checklist

- [ ] Update
[CHANGELOG.md](https://github.com/sourcegraph/deploy-sourcegraph-k8s/blob/main/CHANGELOG.md)
- [x] Update [K8s Upgrade
notes](https://github.com/sourcegraph/sourcegraph/blob/main/doc/admin/updates/kubernetes.md)
- [x] Kustomiz-specific changes
- [x] Update sister repository:
[deploy-sourcegraph-helm](https://github.com/sourcegraph/deploy-sourcegraph-helm)
- [x] Update sister repository:
[deploy-sourcegraph-docker](https://github.com/sourcegraph/deploy-sourcegraph-docker)
sourcegraph/deploy-sourcegraph-docker#1202
- [ ] Verify all images have a valid tag and SHA256 sum
eseliger added a commit to sourcegraph/docs that referenced this pull request Apr 30, 2025
The service is no longer needed. Corresponding helm chart update:
sourcegraph/deploy-sourcegraph-helm#671

Test plan: n/a
@eseliger eseliger merged commit a14cbcb into main Apr 30, 2025
17 checks passed
@eseliger eseliger deleted the es/repoupdater branch April 30, 2025 21:12
eseliger added a commit to sourcegraph/deploy-sourcegraph-docker that referenced this pull request May 5, 2025
Sister PR of sourcegraph/deploy-sourcegraph-helm#671.

Upgrade info alert is at sourcegraph/docs#1110.

The repo-updater service is no longer required starting with Sourcegraph
6.4.

Test plan: Repo-updater-less instances have been running for a while
now.


### Checklist

<!--
Kubernetes and Docker Compose MUST be kept in sync. You should not merge
a change here
without a corresponding change in the other repository, unless it truly
is specific to
this repository. If uneeded, add link or explanation of why it is not
needed here.
-->
* [x] Sister
[deploy-sourcegraph](https://github.com/sourcegraph/deploy-sourcegraph)
change: sourcegraph/deploy-sourcegraph#4468
* [x] Sister
[customer-replica](https://github.com/sourcegraph/deploy-sourcegraph-docker-customer-replica-1)
change (if necessary, for any changes affecting pure-docker or
configuration):
sourcegraph/deploy-sourcegraph-docker-customer-replica-1#79
* [ ] All images have a valid tag and SHA256 sum
enriquegh pushed a commit that referenced this pull request Jul 10, 2025
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants