Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

feat(appliance): self-update can update multiple deployments#64132

Merged
craigfurman merged 1 commit into
mainfrom
craig-rel-302-self-update-should-update-appliance-frontend-too
Jul 30, 2024
Merged

feat(appliance): self-update can update multiple deployments#64132
craigfurman merged 1 commit into
mainfrom
craig-rel-302-self-update-should-update-appliance-frontend-too

Conversation

@craigfurman

Copy link
Copy Markdown
Contributor

The helm chart will configure both the backend and frontend deployments to self-update: sourcegraph/deploy-sourcegraph-helm#513

Relates to https://linear.app/sourcegraph/issue/REL-302/self-update-should-update-appliance-frontend-too

Test plan

Integration test against a real kubernetes API server included. I also kicked the tires in conjunction with the companion Helm PR.

Changelog

@craigfurman craigfurman requested review from a team and Chickensoupwithrice and removed request for a team July 29, 2024 15:05
@cla-bot cla-bot Bot added the cla-signed label Jul 29, 2024
Comment thread internal/appliance/selfupdate/selfupdate.go Outdated
@craigfurman craigfurman force-pushed the craig-rel-302-self-update-should-update-appliance-frontend-too branch from 687df7e to 26b988c Compare July 30, 2024 08:24
@craigfurman craigfurman enabled auto-merge (squash) July 30, 2024 08:24
@craigfurman craigfurman merged commit d945f19 into main Jul 30, 2024
@craigfurman craigfurman deleted the craig-rel-302-self-update-should-update-appliance-frontend-too branch July 30, 2024 08:30
craigfurman pushed a commit that referenced this pull request Jul 31, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants