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

Use the ReadyConditionBuilder for all custom resources that have ready state #3237

Closed
georgethebeatle opened this issue Apr 30, 2024 · 0 comments
Assignees
Labels

Comments

@georgethebeatle
Copy link
Member

Those resources are:

  • CFPackage
  • CFOrg
  • CFSpace
  • CFProcess
  • AppWorkload

Check whether there are more types that are just using strings instead of using the korifiv1alpha1.StatusConditionReady constant

@github-project-automation github-project-automation bot moved this to 🧊 Icebox in Korifi - Backlog Apr 30, 2024
@danail-branekov danail-branekov moved this from 🧊 Icebox to 🇪🇺 To do in Korifi - Backlog Apr 30, 2024
@danail-branekov danail-branekov moved this from 🇪🇺 To do to 🔄 In progress in Korifi - Backlog May 23, 2024
@danail-branekov danail-branekov self-assigned this May 23, 2024
danail-branekov added a commit that referenced this issue May 27, 2024
Korifi custom resources that have their `Ready` condition set by
controllers now utilise the ready condition builder pattern we recently
came up with. The afffected resources are:

* CFPackage
* CFOrg
* CFSpace
* CFProcess
* AppWorkload

While being here, take the opportunity to slightly improve their tests

fixes #3237

wip

* Address added TODOs

wip
danail-branekov added a commit that referenced this issue May 27, 2024
Korifi custom resources that have their `Ready` condition set by
controllers now utilise the ready condition builder pattern we recently
came up with. The afffected resources are:

* CFPackage
* CFOrg
* CFSpace
* CFProcess
* AppWorkload

While being here, take the opportunity to slightly improve their tests

fixes #3237

wip

* Address added TODOs

wip
danail-branekov added a commit that referenced this issue May 27, 2024
Korifi custom resources that have their `Ready` condition set by
controllers now utilise the ready condition builder pattern we recently
came up with. The afffected resources are:

* CFPackage
* CFOrg
* CFSpace
* CFProcess
* AppWorkload

While being here,
* take the opportunity to slightly improve their tests
* use the `korifiv1alpha1.StatusConditionReady` constant wherever it
  makes sense

fixes #3237

wip

* Address added TODOs

wip
danail-branekov added a commit that referenced this issue May 27, 2024
Korifi custom resources that have their `Ready` condition set by
controllers now utilise the ready condition builder pattern we recently
came up with. The afffected resources are:

* CFPackage
* CFOrg
* CFSpace
* CFProcess
* AppWorkload

While being here,
* take the opportunity to slightly improve their tests
* use the `korifiv1alpha1.StatusConditionReady` constant wherever it
  makes sense

fixes #3237
danail-branekov added a commit that referenced this issue May 27, 2024
Korifi custom resources that have their `Ready` condition set by
controllers now utilise the ready condition builder pattern we recently
came up with. The afffected resources are:

* CFPackage
* CFOrg
* CFSpace
* CFProcess
* AppWorkload

While being here,
* take the opportunity to slightly improve their tests
* use the `korifiv1alpha1.StatusConditionReady` constant wherever it
  makes sense

fixes #3237
danail-branekov added a commit that referenced this issue May 27, 2024
Korifi custom resources that have their `Ready` condition set by
controllers now utilise the ready condition builder pattern we recently
came up with. The afffected resources are:

* CFPackage
* CFOrg
* CFSpace
* CFProcess
* AppWorkload

While being here,
* take the opportunity to slightly improve their tests
* use the `korifiv1alpha1.StatusConditionReady` constant wherever it
  makes sense

fixes #3237
tcdowney added a commit that referenced this issue May 29, 2024
Use the ready condition builder in relevant controllers
@github-project-automation github-project-automation bot moved this from 🔄 In progress to ✅ Done in Korifi - Backlog May 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

No branches or pull requests

2 participants