jackc / jackc/tern

rollback using helm ends with an error if "last" is used

Open
#96 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Go
Stars
1.3k
Forks
86
PR merge metrics
No merged PRs in 30d

Description

Hi! Thanks for the great tool. We're started using tern with a k8s job that is recreated with each new helm release. That also means that a previous version of the job is created with each helm rollback.
For simplicity we run tern with "last" destination. That greatly simplifies experience.

This piece of code is causing problems with rollback. https://github.com/jackc/tern/blob/22a33fd044cac3cde941b109580223c073c6302b/migrate/migrate.go#L303-L306

I suggest having a CLI argument to turn this control off if flag is set. WDYT?

Contributor guide

No contributing guide indexed for this repository

Research direction

Start in migrate/migrate.go around lines 303-306 and trace the rollback path when the destination is "last". Determine how the Helm rollback scenario reaches the error and how a CLI option should control that check. Done means the selected option avoids the rollback error without changing the default behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
go, helm
Domain
cli, database
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.