Skip to content

Conversation

@eseliger
Copy link
Member

The service is no longer needed. Corresponding helm chart update: sourcegraph/deploy-sourcegraph-helm#671

Test plan: n/a

The service is no longer needed. Corresponding helm chart update: sourcegraph/deploy-sourcegraph-helm#671

Test plan: n/a
@vercel
Copy link

vercel bot commented Apr 30, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
sourcegraph-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 30, 2025 4:47pm

@eseliger eseliger marked this pull request as ready for review April 30, 2025 16:48
@eseliger eseliger requested review from a team, devdinu, filiphaftek and michaellzc April 30, 2025 16:48
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.
@DaedalusG
Copy link
Contributor

We probably need to also review some of the helper commands etc in our run docs and make sure we're not having people run commands that reference repo-updater there as well. I'll have a look 👀

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 eseliger merged commit ef5aded into main Apr 30, 2025
7 checks passed
@eseliger eseliger deleted the es/ru-rm branch April 30, 2025 21:12
eseliger added a commit to sourcegraph/deploy-sourcegraph-helm that referenced this pull request Apr 30, 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
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 to sourcegraph/deploy-sourcegraph-helm 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