Skip to content

Commit

Permalink
fix: aggregate "*-dev-*" versions into single "dev" version
Browse files Browse the repository at this point in the history
This will prevent the upgrade responder from having too many tags which
can be a performance issue

longhorn-9522

Signed-off-by: Phan Le <phan.le@suse.com>
  • Loading branch information
PhanLe1010 authored and innobead committed Sep 25, 2024
1 parent 3ac6b93 commit 8bc7575
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion controller/setting_controller.go
Original file line number Diff line number Diff line change
Expand Up @@ -1400,8 +1400,13 @@ func (sc *SettingController) CheckLatestAndStableLonghornVersions() (string, str
if err != nil {
return "", "", errors.Wrap(err, "failed to get extra info for upgrade checker")
}

version := sc.version
if strings.Contains(version, "dev") {
version = "dev"
}
req := &CheckUpgradeRequest{
AppVersion: sc.version,
AppVersion: version,
ExtraTagInfo: extraTagInfo,
ExtraFieldInfo: extraFieldInfo,
}
Expand Down

0 comments on commit 8bc7575

Please sign in to comment.