-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
📖 Small improvements to the v1beta2 status proposal #11460
📖 Small improvements to the v1beta2 status proposal #11460
Conversation
|
||
Most notably, the Cluster controller is going to read `ScalingUp`, `ScalingDown` conditions, if existing, and use | ||
them to compute a Cluster level `ScalingUp`, `ScalingDown` condition including all the scalable resources. | ||
Most notably, If `RollingOut`, `ScalingUp`, `ScalingDown` conditions are implemented, the the Cluster controller is going to read |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Most notably, If `RollingOut`, `ScalingUp`, `ScalingDown` conditions are implemented, the the Cluster controller is going to read | |
Most notably, If `RollingOut`, `ScalingUp`, `ScalingDown` conditions are implemented, the Cluster controller is going to read |
b087434
to
e04e63e
Compare
/lgtm |
LGTM label has been added. Git tree hash: 045a4e6fa028df71afa22fd17a2856615bae11de
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: chrischdi The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What this PR does / why we need it:
/area conditions