Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Conditions to workflow status #2421

Closed
seaneagan opened this issue Mar 12, 2020 · 3 comments · Fixed by #2453
Closed

Add Conditions to workflow status #2421

seaneagan opened this issue Mar 12, 2020 · 3 comments · Fixed by #2453
Assignees
Labels
type/feature Feature request

Comments

@seaneagan
Copy link

seaneagan commented Mar 12, 2020

Summary

Add Conditions to workflow status.

Motivation

From the kubernetes api conventions:

Some resources in the v1 API contain fields called phase, and associated message, reason, and other status fields. The pattern of using phase is deprecated. Newer API types should use conditions instead.

This would facilitate using e.g. kubectl wait to wait for workflows:

kubectl wait --for=condition=complete workflow/myworkflow

The argo wait command UI is very nice, not sure if it's possible to get any similar experience with kubectl wait. But where kubectl wait is handy, is if you are trying to wait on a data drive list of resources, some of which may be workflows, some jobs, or any other type of resource for certain conditions, which our team is looking to do.

Proposal

Add a Conditions field to workflow status, which includes at least a completed or similar condition, and whatever other conditions may make sense. Deprecate the phase etc. fields in status which are superceded by Conditions.


Message from the maintainers:

If you wish to see this enhancement implemented please add a 👍 reaction to this issue! We often sort issues this way to know what to prioritize.

@seaneagan seaneagan added the type/feature Feature request label Mar 12, 2020
@alexec
Copy link
Contributor

alexec commented Mar 12, 2020

great idea! we put the status is a label currently, so you'd just need to figure out how to get that into the status

@NikeNano
Copy link
Contributor

I would be happy to help out with this issue @seaneagan , will start to look at it. How do I assign my self to this issue @alexec?

@alexec
Copy link
Contributor

alexec commented Mar 13, 2020

Assigned!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/feature Feature request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants