Skip to content
This repository has been archived by the owner on Nov 1, 2022. It is now read-only.

Always list the status of a workload in fluxctl #2035

Merged
merged 1 commit into from
May 13, 2019

Conversation

hiddeco
Copy link
Member

@hiddeco hiddeco commented May 9, 2019

Before this change the fluxctl list-workloads command would only list the status of a workload if it had containers. But because of how we integrated HelmReleases into the Flux daemon, a HelmRelease only has 'containers' if we are able to detect images in the .spec.values of the resource.

After this change it will always list the status of the workload (and any policies), regardless of the amount of containers it (does not) have.

@hiddeco hiddeco added the UX In pursuit of a delightful user experience label May 9, 2019
@hiddeco hiddeco added this to the v1.12.3 milestone May 9, 2019
Copy link
Member

@squaremo squaremo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks Hidde

Before this change the `fluxctl list-workloads` command would only list
the status of a workload if it had containers. But because of how we
integrated HelmReleases into the Flux daemon, a HelmRelease only has
'containers' if we are able to detect images in the .spec.values of the
resource.

After this change it will always list the status of the workload (and
any policies), regardless of the amount of containers it (does not)
have.
@hiddeco hiddeco merged commit 1bd32aa into master May 13, 2019
@hiddeco hiddeco deleted the show-release-status-hr branch May 13, 2019 10:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
UX In pursuit of a delightful user experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants