Skip to content

[v1 API Review] Add Progressing status condition #1292

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

Closed
Tracked by #740
everettraven opened this issue Sep 20, 2024 · 0 comments · Fixed by #1302
Closed
Tracked by #740

[v1 API Review] Add Progressing status condition #1292

everettraven opened this issue Sep 20, 2024 · 0 comments · Fixed by #1302
Assignees
Labels
epic/v1-api v1.0 Issues related to the initial stable release of OLMv1

Comments

@everettraven
Copy link
Contributor

everettraven commented Sep 20, 2024

To better signal to a user when a ClusterExtension is transitioning to a new state we need to add a Progressing status condition. This condition's state does not affect the Installed condition state unless the ClusterExtension has not progressed successfully through the initial installation.

Some examples of when state transitions may occur:

  • Resolving bundle to be installed for the ClusterExtension
  • Unpacking bundle contents
  • Installing/Upgrading resources for the ClusterExtension

Proposed Status and Reason combinations are:

  • Status: False, Reason: Success - reconciliation has completed successfully, no further progression necessary
  • Status: False, Reason: Blocked - error that requires manual user intervention was encountered during reconciliation. We cannot or will not progress further. As an example, the bundle image reference we resolved from a catalog is invalid (unparseable).
  • Status: True, Reason: Retrying - error encountered during reconciliation that is recoverable without manual intervention. ClusterExtension is re-queued for reconciliation. Some examples of when this condition case will be populated:
    • Upgrade operation fails leaving the ClusterExtension in an intermediate state
    • Permission errors related to informer establishment

For more information, see the RFC

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
epic/v1-api v1.0 Issues related to the initial stable release of OLMv1
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant