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

Provide details about standard conditions used by Crossplane #750

Open
jbw976 opened this issue Apr 12, 2024 · 2 comments · May be fixed by #752
Open

Provide details about standard conditions used by Crossplane #750

jbw976 opened this issue Apr 12, 2024 · 2 comments · May be fixed by #752
Assignees
Labels
bug Something isn't working pending-reporter

Comments

@jbw976
Copy link
Member

jbw976 commented Apr 12, 2024

URL

https://docs.crossplane.io/latest/concepts/managed-resources/#conditions

What's Wrong?

Discussed a bit in this PR: crossplane/crossplane#5453 (comment)

Our docs focus on the possible specific reasons for each condition, but don't seem to focus on the high level conditions themselves, e.g. Ready and Synced. What are their intent and semantics, what scenarios do they cover?

@negz quick write-up that is useful:

Today we have the following status conditions:

  • Ready - The external resource is ready to use, (where applicable) according to its API.
  • Synced - The managed resource's desired state has been successfully applied to the external resource.
@jbw976 jbw976 added the bug Something isn't working label Apr 12, 2024
@plumbis plumbis linked a pull request Apr 12, 2024 that will close this issue
@plumbis
Copy link
Collaborator

plumbis commented Apr 12, 2024

@jbw976 and @TerjeLafton I've added more details in a draft PR.

Preview:
https://deploy-preview-752--crossplane.netlify.app/master/concepts/managed-resources/#conditions

If you're good with this I'll add the same content to the other versions as well.

@plumbis plumbis self-assigned this Apr 12, 2024
@TerjeLafton
Copy link

@jbw976 and @TerjeLafton I've added more details in a draft PR.

Preview: https://deploy-preview-752--crossplane.netlify.app/master/concepts/managed-resources/#conditions

If you're good with this I'll add the same content to the other versions as well.

It looks good to me at least!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working pending-reporter
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants