Skip to content

Commit

Permalink
Adapt HelmRelease revision to API v2
Browse files Browse the repository at this point in the history
Signed-off-by: Stefan Prodan <[email protected]>
  • Loading branch information
stefanprodan committed May 9, 2024
1 parent b85d346 commit 714c5ea
Show file tree
Hide file tree
Showing 4 changed files with 12 additions and 8 deletions.
2 changes: 1 addition & 1 deletion cmd/flux/create_helmrelease.go
Original file line number Diff line number Diff line change
Expand Up @@ -309,7 +309,7 @@ func createHelmReleaseCmdRun(cmd *cobra.Command, args []string) error {
}
logger.Successf("HelmRelease %s is ready", name)

logger.Successf("applied revision %s", helmRelease.Status.LastAppliedRevision)
logger.Successf("applied revision %s", getHelmReleaseRevision(helmRelease))
return nil
}

Expand Down
9 changes: 8 additions & 1 deletion cmd/flux/get_helmrelease.go
Original file line number Diff line number Diff line change
Expand Up @@ -72,9 +72,16 @@ func init() {
getCmd.AddCommand(getHelmReleaseCmd)
}

func getHelmReleaseRevision(helmRelease helmv2.HelmRelease) string {
if helmRelease.Status.History != nil && len(helmRelease.Status.History) > 0 {
return string(rune(helmRelease.Status.History[0].Version))
}
return helmRelease.Status.LastAttemptedRevision
}

func (a helmReleaseListAdapter) summariseItem(i int, includeNamespace bool, includeKind bool) []string {
item := a.Items[i]
revision := item.Status.LastAppliedRevision
revision := getHelmReleaseRevision(item)
status, msg := statusAndMessage(item.Status.Conditions)
return append(nameColumns(&item, includeNamespace, includeKind),
revision, cases.Title(language.English).String(strconv.FormatBool(item.Spec.Suspend)), status, msg)
Expand Down
7 changes: 2 additions & 5 deletions cmd/flux/testdata/trace/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -59,11 +59,9 @@ status:
status: "True"
type: Ready
helmChart: {{ .fluxns }}/podinfo-podinfo
lastAppliedRevision: 6.3.5
lastAttemptedRevision: 6.3.5
lastAttemptedValuesChecksum: c31db75d05b7515eba2eef47bd71038c74b2e531
---
apiVersion: source.toolkit.fluxcd.io/v1beta2
apiVersion: source.toolkit.fluxcd.io/v1
kind: HelmChart
metadata:
name: podinfo-podinfo
Expand All @@ -90,7 +88,7 @@ status:
type: Ready

---
apiVersion: source.toolkit.fluxcd.io/v1beta2
apiVersion: source.toolkit.fluxcd.io/v1
kind: HelmRepository
metadata:
labels:
Expand Down Expand Up @@ -126,7 +124,6 @@ spec:
sourceRef:
kind: GitRepository
name: flux-system
validation: client
interval: 5m
prune: true
status:
Expand Down
2 changes: 1 addition & 1 deletion cmd/flux/trace.go
Original file line number Diff line number Diff line change
Expand Up @@ -448,7 +448,7 @@ Namespace: {{.HelmRelease.Namespace}}
{{- if .HelmRelease.Spec.TargetNamespace }}
Target: {{.HelmRelease.Spec.TargetNamespace}}
{{- end }}
Revision: {{.HelmRelease.Status.LastAppliedRevision}}
Revision: {{.HelmRelease.Status.LastAttemptedRevision}}
{{- if .HelmReleaseReady }}
Status: Last reconciled at {{.HelmReleaseReady.LastTransitionTime}}
Message: {{.HelmReleaseReady.Message}}
Expand Down

0 comments on commit 714c5ea

Please sign in to comment.