Skip to content

fix(rel): add migrator job to release process#351

Merged
jdpleiness merged 1 commit intomainfrom
10-02-fix_rel_add_migrator_job_to_release_process
Oct 3, 2025
Merged

fix(rel): add migrator job to release process#351
jdpleiness merged 1 commit intomainfrom
10-02-fix_rel_add_migrator_job_to_release_process

Conversation

@jdpleiness
Copy link
Copy Markdown
Contributor

@jdpleiness jdpleiness commented Oct 2, 2025

Description

The migrator job was missing from the release process, causing the image tag to no be updated during releases.

This adds the steps to update the image during the release process.

Checklist

Test plan

Tested actions locally:

sg release create --workdir=. --version=v6.8.2313 --inputs=server=v6.8.2313 --pretend
...
👉 [      step] Pretending to run step "sg ops (migrator)"
   [sg ops (migrator)] set -eu
   [sg ops (migrator)] sg ops update-images \
   [sg ops (migrator)]   --kind k8s \
   [sg ops (migrator)]   --registry us-docker.pkg.dev/sourcegraph-images/internal \
   [sg ops (migrator)]   --docker-username=$DOCKER_USERNAME \
   [sg ops (migrator)]   --docker-password=$DOCKER_PASSWORD \
   [sg ops (migrator)]   --pin-tag 6.8.2313 \
   [sg ops (migrator)]   components/utils/migrator/
   [sg ops (migrator)] 
...

Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@jdpleiness jdpleiness requested a review from a team October 2, 2025 18:54
@jdpleiness jdpleiness marked this pull request as ready for review October 2, 2025 18:54
@jdpleiness jdpleiness requested a review from gabtorre October 2, 2025 18:58
@jdpleiness jdpleiness merged commit 9e64aa1 into main Oct 3, 2025
13 of 14 checks passed
Copy link
Copy Markdown
Contributor Author

Merge activity

jdpleiness pushed a commit that referenced this pull request Oct 3, 2025
## Description

The migrator job was missing from the release process, causing the image tag to no be updated during releases.

This adds the steps to update the image during the release process.

## Checklist


- [ ] Update [CHANGELOG.md](https://github.com/sourcegraph/deploy-sourcegraph-k8s/blob/main/CHANGELOG.md)
- [ ] Update [K8s Upgrade notes](https://github.com/sourcegraph/sourcegraph/blob/main/doc/admin/updates/kubernetes.md)
- [x] Kustomiz-specific changes
- [ ] Update sister repository: [deploy-sourcegraph-helm](https://github.com/sourcegraph/deploy-sourcegraph-helm)
- [ ] Update sister repository: [deploy-sourcegraph-docker](https://github.com/sourcegraph/deploy-sourcegraph-docker)
- [ ] Verify all images have a valid tag and SHA256 sum

## Test plan

Tested actions locally:

```shell
sg release create --workdir=. --version=v6.8.2313 --inputs=server=v6.8.2313 --pretend
...
👉 [      step] Pretending to run step "sg ops (migrator)"
   [sg ops (migrator)] set -eu
   [sg ops (migrator)] sg ops update-images \
   [sg ops (migrator)]   --kind k8s \
   [sg ops (migrator)]   --registry us-docker.pkg.dev/sourcegraph-images/internal \
   [sg ops (migrator)]   --docker-username=$DOCKER_USERNAME \
   [sg ops (migrator)]   --docker-password=$DOCKER_PASSWORD \
   [sg ops (migrator)]   --pin-tag 6.8.2313 \
   [sg ops (migrator)]   components/utils/migrator/
   [sg ops (migrator)] 
...
```
 <br> Backport 9e64aa1 from #351
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants