Closed theobarberbany closed 5 months ago
@theobarberbany: This pull request references Jira Issue OCPBUGS-23000, which is valid.
Requesting review from QA contact: /cc @sunzhaohua2
The bug has been updated to refer to the pull request using the external bug tracker.
/lgtm
From autoscaler side, we are doing this to prevent blocks when node critical pods get evicted prior to workloads
@theobarberbany: all tests passed!
Full PR test history. Your PR dashboard.
/lgtm
Thanks.
I always thought that --daemonset-eviction-for-empty-nodes
should default to true
instead of false
, and daemonset-eviction-for-occupied-nodes
to false
instead of true
upstream. It's more practical, realistic, and aligns better with the usage that admins may have for daemonsets. Even so, I have no issue with being explicit.
I don't know if CAO allows changing those args, but it can be useful for admins that don't want/can't continually track the daemonsets to make sure the right annotation is there.
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: JoelSpeed, machine424, theobarberbany
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/jira refresh
The requirements for Jira bugs have changed (Jira issues linked to PRs on main branch need to target different OCP), recalculating validity.
@openshift-bot: This pull request references Jira Issue OCPBUGS-23000, which is valid.
Requesting review from QA contact: /cc @sunzhaohua2
/jira refresh
The requirements for Jira bugs have changed (Jira issues linked to PRs on main branch need to target different OCP), recalculating validity.
@openshift-bot: This pull request references Jira Issue OCPBUGS-23000, which is invalid:
Comment /jira refresh
to re-evaluate validity if changes to the Jira bug are made, or edit the title of this pull request to link to a different bug.
/jira refresh
@simonpasquier: This pull request references Jira Issue OCPBUGS-23000, which is valid.
Requesting review from QA contact: /cc @sunzhaohua2
@theobarberbany: Jira Issue OCPBUGS-23000: Some pull requests linked via external trackers have merged:
The following pull requests linked via external trackers have not merged:
These pull request must merge or be unlinked from the Jira bug in order for it to move to the next state. Once unlinked, request a bug refresh with /jira refresh
.
Jira Issue OCPBUGS-23000 has not been moved to the MODIFIED state.
Adds the 'enable-ds-eviction' annotation to prevent the cluster autoscaler from removing the node-exporter daemonset during a scaling event.
The motivation for doing this to prevent blocks when node critical pods get evicted prior to workloads.