Skip to content

Add Ready condition#51

Open
albertompe wants to merge 13 commits intomainfrom
feature/ready-condition
Open

Add Ready condition#51
albertompe wants to merge 13 commits intomainfrom
feature/ready-condition

Conversation

@albertompe
Copy link
Copy Markdown
Contributor

@albertompe albertompe commented Apr 3, 2026

Closes #50

The Ready condition has been added, created from the first reconciliation cycle (when the Redkey cluster enters the Initializing status). This condition will be True as long as the Redkey cluster is ready (fully initialized and configured, with no scaling or upgrade operations in progress). Otherwise, it will be False.

An event is generated when the Ready condition changes to True.

Some bugs were found during testing and have been fixed:

  • LastTransitionTime was not updating on updates for all conditions. This bug has been fixed.
  • When performing fast scalings, the corresponding condition remained with the value True, when it should have changed to False.

Additionally, Helm charts have been added for deploying Redkey Operator and Redkey cluster instances.

These charts were used for deployment testing to verify the behavior of the new Ready condition. It was deemed beneficial to include them in the project, so they are included in this Pull Request, along with the other changes.

The documentation has also been updated, including a Quick Start using these new Helm charts.

@albertompe albertompe requested a review from a team as a code owner April 3, 2026 23:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature] Add Ready condition

1 participant