Skip to content

Commit b3623ee

Browse files
authored
[VIRT] Add client parameter to descheduler resources (#5185)
##### What this PR does / why we need it: Add client parameter to descheduler resources <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Tests** * Enhanced test infrastructure for descheduler operator reconciliation testing to improve client parameter handling and isolation. <!-- end of auto-generated comment: release notes by coderabbit.ai --> Signed-off-by: Denys Shchedrivyi <dshchedr@redhat.com>
1 parent 9c6a871 commit b3623ee

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

tests/virt/node/descheduler/conftest.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@
3737

3838

3939
@pytest.fixture(scope="package")
40-
def descheduler_operator_reconciled():
40+
def descheduler_operator_reconciled(admin_client):
4141
"""Restart descheduler-operator deployment to trigger reconciliation.
4242
4343
Workaround for the issue when descheduler is installed before other OpenShift operators.
@@ -48,6 +48,7 @@ def descheduler_operator_reconciled():
4848
deployment = Deployment(
4949
name="descheduler-operator",
5050
namespace=NamespacesNames.OPENSHIFT_KUBE_DESCHEDULER_OPERATOR,
51+
client=admin_client,
5152
)
5253
initial_replicas = deployment.instance.spec.replicas
5354
deployment.scale_replicas(replica_count=0)
@@ -258,6 +259,7 @@ def utilization_imbalance(
258259
with PodDisruptionBudget(
259260
name=utilization_imbalance_deployment_name,
260261
namespace=namespace.name,
262+
client=admin_client,
261263
min_available=unallocated_pod_count,
262264
selector=evict_protected_pod_selector,
263265
):

0 commit comments

Comments
 (0)